-
Notifications
You must be signed in to change notification settings - Fork 112
Add extra space for snippets, fix various CSS stuff #3047
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just that one comment about the typo and looks good!
margin-top: $topic-margin-top; | ||
line-height: 1.6; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
was this meant to stay here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
like, do you need both the multiplier and also the 16px? i remember the 16px came from an average number on both windows and osx looking pretty good, i wonder if adding the multiplier will change that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added the multiplier for the small sizes so that the text looks a little better wrapped. The 16px is for the full size
There was a commit referring to issue 3000. Did you mean a different one? |
I'm assuming the 3000 reference should have been 3001. |
Fix #3001. I also added a couple of commits to fix some responsive styling for the topics part of the Top Stories section, and fixed the vertical spacing being uneven