enyojs/moonstone — pull request #1567 by zhaopeterson (2014-09-02 19:36:55+00:00)
Title: BHV-12044-brookepeterson Refactor Header.less and Fix a few issues in HeaderSample. Enyo-DCO-1.1-...

...Signed-off-by: Brooke Peterson

###Refactor/Optimize Header.less
The key is to move the moon-header-text, moon-sub-header-text etc classes directly onto the controls, so no need to redefine them in moon-header-title or moon-sub-header-title classes. This way also rid of the need to write their enyo-locale-non-latin counter parts, 3k reduce in css size.
(follow title-above case in the existing code {name: 'titleAbove', classes: 'moon-super-header-text moon-header-title-above'},)
Removed, combined a quite a few rule/classes.

###also fix a few issues in HeaderSample:
1. The Small Header Height should be 120px according to the Visual Design Redlines instead of 112px.
2. Small Header subtitle enyo-locale-non-latin should be "LG display".
3. Small Header title and subtitle should reverse position in RTL mode.
4. in full-bleed mode, the small header header client now have 20px margin on the right.

###Note:
I think it is a UX design flaw to concat the small header title and subtile and to marquee together. In the case of a very long title, it takes quite a few seconds for the users to see the subtitle, if they see at all. It is better off to have both title and subtitle in separate blocks (set a max width on each), and synchronize the marquee if both texts are long.