On 03/30/2016 10:19 AM, Andrea Bolognani wrote:
Basically, the navigation menu is *way* too big, and takes
the focus away from the actual content. Plus it's very easy
for longish words to overflow it (see "Authentication" in
the screenshots below).
The headers are probably bigger than needed as well, so
I've scaled them down. Using Wikipedia for comparison,
you can see the new size is more comparable to what they
use.
Some other minor cleanups have been included as well.
Before[1] and after[2] screenshots for your convenience.
Cheers.
[1]
http://i.imgur.com/MZchAWD.png
[2]
http://i.imgur.com/BXepLTh.png
Andrea Bolognani (7):
docs: Adjust vertical whitespace in CSS
docs: Remove empty CSS rule
docs: Don't use <strong> in headers
docs: Make menu entries smaller
docs: Don't use bold text for menu entries
docs: Use bold text for all headers
docs: Make most headers a bit smaller
docs/apps.html.in | 2 +-
docs/generic.css | 15 +++++++++------
docs/libvirt.css | 30 ++++++------------------------
3 files changed, 16 insertions(+), 31 deletions(-)
Lot's of style churn here lately... Ironically the view I get from
libvirt.org has always had slightly bolder and larger text as opposed
the view I get when viewing changes in a local branch. Personally, I
like the larger, bolder text since it's easier to read than some font
style that's meant for a mobile phone. As long as we don't start adding
graphics and flash, we're good ;-)!
Anyway just a note that patch 3 removes the h1 font-weight bold, but
patch 6 claims all headers except h1 were already bold... Not a big deal
due to the end result, but I assume they should be together.
My only other comment would be regarding <dt> elements - I would prefer
to see them use a slightly bolder text so that they stand out from the
rest of the text. If I read the tea leaves correctly that would be an
adjustment to the <dt> to have "font-weight: bold;".
Other than that you have an ACK from me... and it should be safe for
freeze...
John