Vector: Restore collapsibleNav design improvements
authorTimo Tijhof <krinklemail@gmail.com>
Sun, 4 May 2014 20:10:33 +0000 (22:10 +0200)
committerKrinkle <krinklemail@gmail.com>
Sun, 4 May 2014 20:13:46 +0000 (20:13 +0000)
commit761cd667445d3dab79e451a945c69936c99198db
tree9ecf89ab27748961067351463ec7564eb0a1cc73
parentc3d5444126197c5b215e49f5fc61d199d98e0cc4
Vector: Restore collapsibleNav design improvements

Follows-up db13c65 and c3d544412, which removed the
collapsibleNav module. However it also undid various design
changes that really should not have been part of the
collapsibleNav module but were significant, I've restored those.

Basically, this merges the old collapsibleNav.less into
navigation.less. I won't list all chnages, but here's a few:

* Increase spacing between heading and side of portal (via margin and
  padding on the portal, not on the heading itself).
  The headings were almost against the browser edge.

* Change color of heading slightly (#4d4d4d instead of #444).

* Move separator line from below each heading to between sidebar
  sections.

* Change spacing between list items from 0.5 on the bottom to
  0.25em on top and bottom (effectively the same, except aligns
  the first and last item better in relation to the heading and
  the separator line).

* (js-only enhancement) Mark first section with class "first",
  and remove the separator line, heading and offset from that one.

Also:

* Clean up 'h3' padding/margin situation.
  They were inheriting 0.3em of bottom margin from the global
  'h3' selector for content. This was a bit of a fragile
  dependency and was just a coincidence. Reset the margin against
  the global selector and added it to the local padding instead.
  No visual change.

* Remove unused less variables.

Change-Id: I979179a82c1de9b1b7d66310ebd5daf8e6aa1802
skins/vector/components/navigation.less
skins/vector/variables.less
skins/vector/vector.js