Closed
Bug 1162315
Opened 10 years ago
Closed 10 years ago
Save password prompt string overlaps button text
Categories
(Firefox for iOS :: General, defect)
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| fennec | + | --- |
People
(Reporter: sleroux, Unassigned)
References
Details
Attachments
(2 files)
Looks like the snack bar is in the incorrect place or doesn't account for the placement of the buttons underneath
| Reporter | ||
Comment 1•10 years ago
|
||
Also, I found it pretty meta that I encountered a bug while trying to file a bug...
Updated•10 years ago
|
tracking-fennec: --- → ?
Comment 3•10 years ago
|
||
This is some cleanup for these constraints (I reworked them so that the buttons are always below the text/image). I also gave these a min-width on tablets here, and tweaked the colors slightly.
Attachment #8603578 -
Flags: review?(sleroux)
| Reporter | ||
Comment 4•10 years ago
|
||
Comment on attachment 8603578 [details] [review]
PR https://github.com/mozilla/firefox-ios/pull/441
Works well. Left a couple a couple of comments regarding the way we're sizing the SnackBar widget.
Attachment #8603578 -
Flags: feedback+
Comment 5•10 years ago
|
||
Updated this. I remember having issues if I didn't explicitly set a width originally here, but constraints seems smart enough to find a tight-wrapping height for me automatically now. I musta been missing one it needed originally. Changes are in their own commit.
| Reporter | ||
Updated•10 years ago
|
Attachment #8603578 -
Flags: review?(sleroux) → review+
| Reporter | ||
Comment 6•10 years ago
|
||
Looks good - just a minor nit comment and a question
Updated•10 years ago
|
tracking-fennec: ? → +
You need to log in
before you can comment on or make changes to this bug.
Description
•