-
Notifications
You must be signed in to change notification settings - Fork 69
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
Documentation layout refresh #455
Comments
@molant something like this. "Have a suggestion" box is stickied to the page on desktop but on mobile we can keep the current implementation with the buttons below the title. Though on mobile it might make more sense to have them at the bottom of the article. Thoughts? |
This looks like a good starting point. Some comments:
Re about shrinking the header I was thinking of the following: The edit/signal issue links could be in the top (same height they are right now, next to the title?) and/or bottom of the document. |
When you're talking about the header you're referring to the menu/nav blue bars correct? |
Most of the pages we have are documentation or have content to the left only (governance, faq, etc.). The ones that are more unique are the homepage and the scanner (and we have to review the scanner anyways). Here's what other projects do (2 and 3 column examples) to take inspiration: |
I like what MS does. Especially with the "in this article" which could be useful for documentation since some pages have many subsections. I don't want to shrink the width of overall container of the site body. I get that most of our pages are documentation pages, but the ones that aren't will suffer from a lack of space and the changes will domino effect down to changing font sizing and margins between content modules etc. I think time is better spent ajusting the documentation pages. |
Can you propose something for sonarwhal? |
I just want to point out that for some odd reason the website wasn't loading the Source Sans Pro font for the paragraphs. If you look at the screenshots above, the table of contents is loading the correct Source Sans Pro font, but the documentation body copy is larger and different. This is the fallback font. This doesn't change anything with what I'm doing but I just wanted to make a note since there's no reason the body font should load for only the TOC and not the rest of the paragraphs. Will keep an eye on this. But the fallback font Verdana is very different from Source Sans Pro, so I will probably update it to Arial. |
Looks like Source Sans Pro was removed at some point from the font list so that would be why it's not loading. I just downloaded it locally which is why it's now rendering properly. |
Working on mockup now.
The text was updated successfully, but these errors were encountered: