search

PolymerElements / paper-toolbar

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

Proposed fix for #82 #89

Closed lauweijie closed 8 years ago

lauweijie commented 8 years ago

Reorder css rules to prevent margin-left from being ignored by --paper-toolbar-title mixin.

Fixes #82.

bicknellr commented 8 years ago

Comment; otherwise LGTM.

bicknellr commented 8 years ago

Oh, one last thing, could you move the .toolbar-tools > ::content paper-icon-button[icon=menu] rule below all the .title ones? That way they'll be a bit more grouped by what they're selecting now that they've been split up a bit more.

lauweijie commented 8 years ago

While I'm at it, shall I also remove the refactor TODO comment since we've discussed and looked through the selectors?

bicknellr commented 8 years ago

Sure, I don't think it's applicable anymore.