Closed Bug 997920 Opened 7 years ago Closed 7 years ago
Deactivate Rocketbar with cancel button
The spec shows a cancel button in the focused state of the Rocketbar which should take the Rocketbar out of the focused state and go back to previous state.
WIP patch https://github.com/benfrancis/gaia/commit/e180b60707bd8ed4943bf4a2578752dd969f16e8 needs tests and needs to wait for bug 993346 to land to work.
Fixed merge conflicts.
Cheers, I was waiting on 99346 to land before reviewing, will do now
Comment on attachment 8410482 [details] [review] https://github.com/mozilla-b2g/gaia/pull/18539 I believe from looking at the specs the divider line is supposed to have top and bottom padding not take up the full line-height (although its hard to see, the spec is blurry), and the right and left margins seem off (left is smaller than the right margin, should be equal) However we need to do a pass on exact visuals after user testing, this works great, so would be nice if you could get quick fixes for those in, if not its still best to get it in for testing
Attachment #8410482 - Flags: review?(dale) → review+
Thanks for the review Dale, I could have sworn the visuals I saw didn't have padding there, but I have seen several different versions now. We definitely need to make another pass on Rocketbar visuals in general as the specs have changed several times during implementation and many of the dimensions are now wrong anyway. This is green on Travis and I think is probably ready to land as soon as the tree re-opens.
Merged into master https://github.com/mozilla-b2g/gaia/commit/ceb0871838e45d4b813775e584d52d1edae4c1bb
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.