Address Melissa Amos's feedback.#1
Address Melissa Amos's feedback.#1nfagerlund merged 3 commits intopuppetlabs:masterfrom gguillotte:ma-fixes
Conversation
|
|
||
| @font-size-h1: (@font-size-base * 2.6); // ~46px | ||
| @font-size-h1: (@font-size-base * 2.9); // ~51px | ||
| @font-size-h2: (@font-size-base * 2.15); // ~38px |
There was a problem hiding this comment.
Is it possible to make H2-H4 smaller, and more differentiated, rather than increasing the size of the H1? These fonts are all rather huge. With a longish title, the H2s extend to two or sometimes three lines, like here: https://docs.puppet.com/pe/latest/release_notes_known_issues_console.html. It's also difficult to distinguish between H2 and H3, like here: https://docs.puppet.com/pe/latest/install_agents.html. You have to really be paying attention to font size to recognize that the H3s are children. Because we don't use indenting to indicate document structure, it seems important that the heading levels are very clearly distinguishable.
|
Just one comment; otherwise, looks great. No worries about the period in steps 2. The extra padding should help. |
|
The ordered list spacing and the coloration both look good to me. I think I agree about the h2 and h3 being a little too close together, but I'm fine with merging this and iterating on that a second time. |
Uh oh!
There was an error while loading. Please reload this page.