PolymerElements / paper-toolbar

A Material Design toolbar/appbar
https://www.webcomponents.org/element/PolymerElements/paper-toolbar
40 stars 41 forks source link

Question: at 640px media breakpoint 64px -> 56px?? #44

Closed clintwood closed 9 years ago

clintwood commented 9 years ago

Is there any sense in this 8px difference as a default behavior at the 640px media breakpoint? IMHO there is no UX benefit and actually looks like buggy UI.

I'd suggest that the default :host value should just align with the media query value (or vice versa) as a default thus still allowing the default resize at the 640px media breakpoint.

I have tried the obvious override this within my app-theme.html:

--paper-toolbar {
  height: 56px;
};

but since there is no override for .toolbar-tools {height: 64px; ...} this doesn't seem to resolve the jump of the toolbar tools at the breakpoint.

There's a chance to fix it in PR #40!