Closed
Bug 867193
Opened 11 years ago
Closed 11 years ago
Form autocomplete popup is positioned too high when the dynamic toolbar is visible
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox20 unaffected, firefox21 unaffected, firefox22 unaffected, firefox23+ verified, firefox24+ verified, fennec23+)
VERIFIED
FIXED
Firefox 23
Tracking | Status | |
---|---|---|
firefox20 | --- | unaffected |
firefox21 | --- | unaffected |
firefox22 | --- | unaffected |
firefox23 | + | verified |
firefox24 | + | verified |
fennec | 23+ | --- |
People
(Reporter: yoanna_crisan, Assigned: cwiiis)
References
Details
(Keywords: regression, reproducible)
Attachments
(2 files)
60.91 KB,
image/png
|
Details | |
1.99 KB,
patch
|
Margaret
:
review+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Build: Firefox for Android 23.0a1(2013-04-30) Device: Samsung Galaxy R OS: Android 2.3.4 Steps to reproduce: 1.Go to gmail.com 2.Insert an username and press the "Sign in" button 3.Repeat step 2 for a new username 4.Delete the last inserted username Expected results: - The prefilled usernames are displayed bellow the username field Actual results: - The prefilled usernames overlaps the username field and a little bit of the "Username" label
Comment 1•11 years ago
|
||
This is also reproducible using the latest Nightly on Nexus 4-(4.2.2) and Samsung Galaxy SII (2.3.4)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Hardware: x86_64 → ARM
Updated•11 years ago
|
tracking-fennec: --- → ?
status-firefox20:
--- → unaffected
status-firefox21:
--- → unaffected
status-firefox22:
--- → unaffected
status-firefox23:
--- → affected
Updated•11 years ago
|
Keywords: regression
Comment 2•11 years ago
|
||
Yoanna would you get a regression range using inbound builds if possible. http://ftp.mozilla.org/pub/mobile/tinderbox-builds/mozilla-inbound-android/
Keywords: regressionwindow-wanted
Reporter | ||
Updated•11 years ago
|
Summary: Prefilled usernames overlaps the username field on gmail.com → Prefilled usernames overlap the username field on gmail.com
Updated•11 years ago
|
Keywords: reproducible
Reporter | ||
Comment 3•11 years ago
|
||
The regression window for this issue is: 1. mozilla central good build: 24.04.2013 bad build: 25.04.2013 pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=fef5f202b2dc&tochange=690b5e0f6562 2. inbound good build: 1366819129 bad build: 1366819610 pushlog: http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=93f79ae43c2a&tochange=684a5ca2efb7
Keywords: reproducible
Updated•11 years ago
|
Blocks: 858969
Keywords: regressionwindow-wanted → reproducible
Comment 4•11 years ago
|
||
Confirmed that this works with browser.chrome.dynamictoolbar; false
Updated•11 years ago
|
tracking-fennec: ? → 23+
Updated•11 years ago
|
status-firefox24:
--- → affected
Updated•11 years ago
|
tracking-firefox23:
--- → ?
tracking-firefox24:
--- → ?
Updated•11 years ago
|
Comment 6•11 years ago
|
||
In bug 874247, I mentioned fixing this in FormAssistPopup, but now I'm wondering if this is instead an issue with the return value of ElementTouchHelper.getBoundingContentRect(). There have been other issues with coordinates being incorrect in content (e.g. bug 872961), so maybe this fix needs to be closer to content.
Comment 7•11 years ago
|
||
Updating the summary to better reflect the exact issue.
Summary: Prefilled usernames overlap the username field on gmail.com → Form autocomplete popup is positioned too high when the dynamic toolbar is visible
Comment 8•11 years ago
|
||
Confirming that I only see this with the navigation bar visible on screen. On gmail, the moment I scroll the navigation bar off-screen the popup will drop correctly below the username field.
Assignee | ||
Comment 9•11 years ago
|
||
This takes the margin offset into account in FormAssistPopup. This works, but if you make the toolbar hide/show while a popup is visible and don't cause it to re-show/hide, it will end up in the wrong position. Perhaps a better way of fixing this would be to attach the layout that the popup is in to the bottom of the dynamic toolbar - now that I've thought of this, I prefer this method. I'll try to get to this, but feel free to take it.
Attachment #755344 -
Flags: review?(margaret.leibovic)
Comment 10•11 years ago
|
||
(In reply to Chris Lord [:cwiiis] from comment #9) > Perhaps a better way of fixing this would be to attach the layout that the > popup is in to the bottom of the dynamic toolbar - now that I've thought of > this, I prefer this method. I'll try to get to this, but feel free to take > it. I don't know if this will work well because the FormAssistPopup layout is in shared_ui_components, which is designed to be used in web apps as well.
Comment 11•11 years ago
|
||
Comment on attachment 755344 [details] [diff] [review] Take margin offset into account when positioning pop-up This seems like a simple approach, although it would be good to write a follow-up patch to reposition the popup when the toolbar is shown/hidden. Is there a way to listen for that?
Attachment #755344 -
Flags: review?(margaret.leibovic) → review+
Assignee | ||
Comment 12•11 years ago
|
||
Pushed to inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/f9424be46f38 Yes, we can listen to the metrics changing with onMetricsChanged (I think that's it - check GeckoLayerClient and BrowserApp, it listens for margin changes to move the toolbar)
Comment 13•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f9424be46f38
Assignee: nobody → chrislord.net
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 24
Comment 14•11 years ago
|
||
(In reply to Chris Lord [:cwiiis] from comment #12) > Pushed to inbound: > > https://hg.mozilla.org/integration/mozilla-inbound/rev/f9424be46f38 > > Yes, we can listen to the metrics changing with onMetricsChanged (I think > that's it - check GeckoLayerClient and BrowserApp, it listens for margin > changes to move the toolbar) Thanks for the pointer, I filed bug 877782 to implement this.
Comment 15•11 years ago
|
||
Verified fixed on: Build: Firefox for Android 24.0a1 (2013-06-09) Device: Samsung Galaxy R OS: Android 2.3.4
Assignee | ||
Comment 16•11 years ago
|
||
Comment on attachment 755344 [details] [diff] [review] Take margin offset into account when positioning pop-up [Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 867193 User impact if declined: Auto-complete pop-ups obscure user input Testing completed (on m-c, etc.): Been on m-c for a couple of weeks without complaint (afaik) Risk to taking this patch (and alternatives if risky): Low risk String or IDL/UUID changes made by this patch: None
Attachment #755344 -
Flags: approval-mozilla-aurora?
Updated•11 years ago
|
Attachment #755344 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 17•11 years ago
|
||
Pushed to aurora: https://hg.mozilla.org/releases/mozilla-aurora/rev/2158896cb923
Target Milestone: Firefox 24 → Firefox 23
Comment 18•11 years ago
|
||
Firefox for Android 23.0b2 (2013-07-02) Device: LG Optimus 4x OS:Android 4.1.2
Status: RESOLVED → VERIFIED
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•