1180597 - Wrong display for more than two doohangers

Status: RESOLVED INCOMPLETE

(Firefox for Android Graveyard :: General, defect)

Description

[Teodora Vermesan (:TeoVermesan)]

Attachment: Screenshot_2015-07-06-11-07-55.png

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

Comment 1

[:Margaret Leibovic]

Chenxia, can you look into this?

Comment 2

[:Margaret Leibovic]

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.

Comment 3

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

We should be fixing and scrolling within rather than pushing the doorhanger to cover the toolbar

Comment 4

[Chenxia Liu [:liuche]]

Yup, I'm looking at this now - setHeight with a MeasureSpec apparently just fails to work, so investigating other options.

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

Chenxia - You re-nomed this but never mentioned why this should not be tracking 41. Can you give some context?

Comment 6

[Chenxia Liu [: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.

Comment 7

[:Margaret Leibovic]

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.

Comment 8

[Chenxia Liu [:liuche]]

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.

Comment 9

[kimsaehun]

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?

Comment 10

[BMO Automation]

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.