Skip to content

Address Melissa Amos's feedback.#1

Merged
nfagerlund merged 3 commits intopuppetlabs:masterfrom
gguillotte:ma-fixes
Apr 19, 2016
Merged

Address Melissa Amos's feedback.#1
nfagerlund merged 3 commits intopuppetlabs:masterfrom
gguillotte:ma-fixes

Conversation

@gguillotte
Copy link

@gguillotte gguillotte commented Apr 18, 2016

  • Increase the h1 size to match the main site's (~3.2rem, ~51px).
  • Realign ordered list item numerals to match unordered list item bullets.
  • Color <strong> text in headings.

@gguillotte
Copy link
Author

@nfagerlund @melamos


@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
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@melamos
Copy link

melamos commented Apr 19, 2016

Just one comment; otherwise, looks great. No worries about the period in steps 2. The extra padding should help.

@nfagerlund
Copy link

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.

@nfagerlund nfagerlund merged commit 4f24893 into puppetlabs:master Apr 19, 2016
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