Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduced the whitespace above the main heading on the about screen #194

Closed
wants to merge 1 commit into from

Conversation

UtiluMark
Copy link
Contributor

Part of #188

@UtiluMark
Copy link
Contributor Author

@whimboo Please review and merge.

@xabolcs
Copy link
Collaborator

xabolcs commented Oct 6, 2015

@UtiluMark, I don't see the difference with latest Nightly. Could you post some before-after screenshots?

@UtiluMark
Copy link
Contributor Author

@xabolcs
about:nightly before reducing the margin above "Nightly Tester Tools":
ntt pr 194 about nightly before
about:nightly after reducing the margin above "Nightly Tester Tools":
ntt pr 194 about nightly after

@whimboo
Copy link
Contributor

whimboo commented Oct 6, 2015

Makes sense to me, so r=me. I will land it in a moment.

@whimboo
Copy link
Contributor

whimboo commented Oct 6, 2015

Merged the PR as 9e83e43. Thank you for the patch!

@whimboo whimboo closed this Oct 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants