Measure two-step delete, with sliding buttons #75
Conversation
| .fa { | ||
| color: @brand-primary; | ||
| } | ||
| } |
There was a problem hiding this comment.
At a quick glance the arbitrary padding sizes and the explicit browser support for transitions (rather than using a mixin) could be cleaned up... can you work with Liam to get this CSS cleaned up, he'll likely have good approaches.
|
Noted one issue: if you click the gear, and then click the minus, clicking the gear doesn't hide the resulting buttons; it should hide the buttons and reset them. |
| } | ||
| } | ||
| .measure-settings-expanded { | ||
| width: 16%; |
There was a problem hiding this comment.
Setting a percentage-based width makes me pretty nervous... this might be worth talking through in person. PS, http://demosthenes.info/blog/775/Which-CSS-Measurements-To-Use-When is a good resource (even if Bootstrap breaks some of these standards when it comes to ems vs pixels).
|
Updated to use better styles and template, closing and will re-open with rebased branch |
Story
Notes
Images