Closed
Bug 895457
Opened 11 years ago
Closed 11 years ago
Defect - "Remember password" buttons are not visible when in snapped view
Categories
(Firefox for Metro Graveyard :: Theme, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 26
People
(Reporter: TimAbraldes, Assigned: sfoster)
References
Details
(Whiteboard: feature=defect c=Content_features u=metro_firefox_user p=2)
Attachments
(3 files)
STR:
1. Put Firefox metro in snapped view
2. Go to a site requiring a login and log in
3. Notice that the buttons ("remember," "not now") are partially visible or completely obscured
Updated•11 years ago
|
Blocks: metrov1defect&change
Summary: "Remember password" buttons are not visible when in snapped view → Defect - "Remember password" buttons are not visible when in snapped view
Whiteboard: feature=defect c=tbd u=tbd p=0
Updated•11 years ago
|
Priority: -- → P2
Comment 1•11 years ago
|
||
While getting everything perfect in snapped view is probably a P2 and not a P1, we should make sure our content-interaction-triggered toolbars, like the passwordd toolbar, function in Snap or otherwise narrow views.
Blocks: 831977
Whiteboard: feature=defect c=tbd u=tbd p=0 → feature=defect c=Content_features u=tbd p=0
Updated•11 years ago
|
Whiteboard: feature=defect c=Content_features u=tbd p=0 → feature=defect c=Content_features u=metro_firefox_user p=0
Updated•11 years ago
|
Assignee: nobody → sfoster
Status: NEW → ASSIGNED
QA Contact: jbecerra
Whiteboard: feature=defect c=Content_features u=metro_firefox_user p=0 → feature=defect c=Content_features u=metro_firefox_user p=2
Updated•11 years ago
|
Assignee | ||
Comment 2•11 years ago
|
||
Adds a @media query for snapped view (although we might tweak the threshold as we test in 8.1) to override the single-row hbox layout to drop each of the action buttons unto their own line.
Attachment #789763 -
Flags: review?(mbrubeck)
Assignee | ||
Comment 3•11 years ago
|
||
Screenshot of the snapped-view notification in action.
To confirm:
(with a fresh profile)
Open twitter.com (or other site that uses a username/password which the browser will prompt to remember)
Enter login details
You'll see the notification bar asking to remember the password
Drag to snapped-view
The layout should change to vertically stack the "Remember password" and "Not for this site" buttons, which should all be visible, not-clipped and operable.
Assignee | ||
Comment 4•11 years ago
|
||
Let me know if you think subverting the flex behavior in CSS like this is reasonable, or if we should change the binding's markup to better fit how we need it to work (we would need to define our own notification binding)
Comment 5•11 years ago
|
||
Comment on attachment 789763 [details] [diff] [review]
snapped-notifications
Review of attachment 789763 [details] [diff] [review]:
-----------------------------------------------------------------
Rearranging things in CSS without changing markup makes me happy. Lots of fond memories of the CSS Zen Garden. :)
::: browser/metro/theme/platform.css
@@ +498,5 @@
> -moz-box-flex: 9999;
> }
>
> +/* ensure the notification children pick up snapped-view box property overrides */
> +.notification-inner .box-inherit {
Is this specifically for the buttons? If so, please use ".notification-button > .box-inherit" to make this more self-documenting and possibly faster.
Attachment #789763 -
Flags: review?(mbrubeck) → review+
Assignee | ||
Comment 6•11 years ago
|
||
.notification-button.box-inherit worked out and I think meets the review-comment goal.
Landed on fx-team: https://hg.mozilla.org/integration/fx-team/rev/347a01e705df
Comment 7•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
Comment 8•11 years ago
|
||
Mozilla/5.0 (Windows NT 6.2; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0
Build ID: 20130827030201
While testing this using latest Nightly for iteration #13, I got the result shown in the screenshot. It seems to me that this issue is still reproducible.
Does anyone have any suggestions/thoughts?
Flags: needinfo?
Assignee | ||
Comment 9•11 years ago
|
||
I dont follow - the 2 buttons ("Remember password", "Not for this site") are clearly visible in that screenshot?
Flags: needinfo?
Comment 10•11 years ago
|
||
> I dont follow - the 2 buttons ("Remember password", "Not for this site") are
> clearly visible in that screenshot?
To me, the 2 buttons ("Remember password", "Not for this site") from my screenshot seem to look pretty much the same as the ones from the screenshot attached in comment 3. I'm sorry for the confusion.
Marking this as verified, per comment 8 and comment 9. Thanks Sam!
Status: RESOLVED → VERIFIED
Comment 11•11 years ago
|
||
Went through the following "Defect" for iteration #14 without any issues. Used the following build:
http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-09-15-03-02-08-mozilla-central/
- Went through the original test case(s) added in comment #0 without any issues
- Logged into Twitter and received the "Remember password" & "Not for this site" prompt. Moved the screen into snapped view and it appeared correctly (nothing being cut off)
- Ensured that moving the screen between full, filled and snapped views worked without any issues (nothing being cut off)
- Ensured that pressing the "X" would dismiss the prompt but display the "Remember" password confirmation the next time around
- Ensured that "Remember password" worked in snapped view without any issues (another story in an earlier iteration also tests this)
- Ensured that "Not for this site" worked in snapped view without any issues (another story in an earlier iteration also tests this)
- Went through all of the test cases using both the left and right side of the screen in snapped view
Updated•10 years ago
|
OS: Windows 8 Metro → Windows 8.1
You need to log in
before you can comment on or make changes to this bug.
Description
•