Closed Bug 1180597 Opened 7 years ago Closed 1 year ago

Wrong display for more than two doohangers

Categories

(Firefox for Android Graveyard :: General, defect)

41 Branch
ARM
Android
defect
Not set
normal

Tracking

(firefox40 unaffected, firefox41 affected, firefox42 affected, fennec-)

RESOLVED INCOMPLETE
Tracking Status
firefox40 --- unaffected
firefox41 --- affected
firefox42 --- affected
fennec - ---

People

(Reporter: TeoVermesan, Unassigned, Mentored, NeedInfo)

Details

(Whiteboard: [good next bug][lang=java])

Attachments

(1 file)

Tested with:
Device: LG Nexus 4 (Android 5.1.1)

Steps to reproduce:
1. Go to https://lunch.softvision.ro
2. Enter username and password
3. Login and remember the login
4. Tap on the Mixed Content Shield icon

Expected result:
- Doorhangers are displayed correctly under the URL Bar 

Actual result:
- Doorhangers overlap the URL Bar
Chenxia, can you look into this?
tracking-fennec: --- → ?
Flags: needinfo?(liuche)
Assignee: nobody → liuche
tracking-fennec: ? → 41+
Looking at the screenshot, maybe this is the expected behavior for multiple doorhangers like this? Otherwise the user would just have to scroll even more.
We should be fixing and scrolling within rather than pushing the doorhanger to cover the toolbar
Yup, I'm looking at this now - setHeight with a MeasureSpec apparently just fails to work, so investigating other options.
Flags: needinfo?(liuche)
tracking-fennec: 41+ → ?
tracking-fennec: ? → 41+
Chenxia - You re-nomed this but never mentioned why this should not be tracking 41. Can you give some context?
tracking-fennec: 41+ → ?
Flags: needinfo?(liuche)
I'll take another look at this this week, and see if I can find how to anchor this better.

This has been a low priority because I'd consider this an annoying visual problem when a keyboard pops up, but doesn't affect the usage of the doorhanger.
Flags: needinfo?(liuche)
I don't think this is important enough to need to track, but it could be a good polish fix. Chenxia, feel free to make this a mentor bug if you think it could be a good one.
tracking-fennec: ? → -
This bug will likely take some digging and experimentation to figure out what is going wrong, and why setHeight or setAnchor are not being respected when the keyboard comes up and compresses the space.
Assignee: liuche → nobody
Mentor: liuche
Whiteboard: [good next bug][lang=java]
Hi, I'd like to get started on this but I have no idea where to start. May I have some guidance on fixing this bug?
Flags: needinfo?(liuche)
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.