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

Fix reputation wrap issue on mobile #842 #1016

Conversation

robert9g
Copy link
Contributor

@robert9g robert9g commented Nov 4, 2017

Fixes #842 .

Changes:

  • made .UserHeader__user to have margin: 0 !important; on small screens. This margin has no relevance on small screens, since the text is center aligned and has no elements on the sides.
  • decreased font size for .UserHeader__user__username to 28px on small screens

Observations:
The !important was necessary because some media queries were defined before normal style in UserHeader.less. Normally media queries are defined after the normal style(non-media). In the present case the normal style would override the media queries, therefore !important was needed.

I wanted to move the media queries at the bottom of the file, so that the !important will not be needed, but wasn't sure if the reason for this would be clear enough.

In case this is wanted, it can be easily changed.

@robert9g robert9g changed the title Fixes reputation wrap issue on mobile #842 Fix reputation wrap issue on mobile #842 Nov 4, 2017
@bonustrack bonustrack temporarily deployed to busy-nd-pr-1016 November 5, 2017 09:11 Inactive
@Sekhmet Sekhmet added the style label Nov 5, 2017
@Sekhmet Sekhmet merged commit 4b6265f into busyorg:new-design Nov 5, 2017
@robert9g robert9g deleted the fix-for-reputation-and-name-wrap-issue-842 branch November 5, 2017 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants