Closed Bug 677673 Opened 8 years ago Closed 8 years ago

Bookmark Popup arrowbox should point up in tablet mode


(Firefox for Android Graveyard :: General, defect)

Not set


(Not tracked)

Firefox 9


(Reporter: mfinkle, Assigned: wesj)



(Whiteboard: [inbound])


(2 files)

The popup that appears when you "star" a page needs to anchor to the star button in the toolbar when in tablet mode. It is still trying to anchor to the sidebar button.
OS: Linux → All
Hardware: x86 → All
Attached patch Patch v1Splinter Review
This modifies our arrowbox anchorTo method to take a separate parameter giving the position of the popup. When it is passed in, the arrow position is forced, and the popup is moved to an appropriate position. Works well here!

I attempted to always force popups to be inbounds, which works with left, but not always with top. I assume this is because the first time these are shown, the height of the arrowbox is basically window-height.
Assignee: nobody → wjohnston
Attachment #557638 - Flags: review?(mark.finkle)
Comment on attachment 557638 [details] [diff] [review]
Patch v1

FYI: I recently added simpler methods to force position of the arrow. See pointLeftAt in arrowbox.xml. I guess you should replace pointLeftAt with your new stuff?
Comment on attachment 557638 [details] [diff] [review]
Patch v1

These changes look good. I like the code reduction. We should update the code Lucas added to use this new approach too. Also, tests? See what desktop does and maybe we could steal the approach.

I'm ok with landing this, but I do want the followup for Lucas' changes and some simple tests.
Attachment #557638 - Flags: review?(mark.finkle) → review+
Blocks: 684277, 684278
Blocks: 684397
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 9
Attached image screenshot
It seems that the arrow is missing from the Bookmark Popup on the latest Nightly build. Is this expected?

Mozilla/5.0 (Android;Linux armv7l;rv:9.0a1)Gecko/20110927
Firefox/9.0a1 Fennec/9.0a1
Device: Acer ICONIA A500
OS: Android 3.1
Yes, I think so, since they removed it intentionally from the search menu (bug 688432).
Build ID: Mozilla/5.0 (Android;Linux armv7l;rv:9.0a2)Gecko/20111004 Firefox/9.0a2 Fennec/9.0a2
Device: Acer ICONIA A500
OS: Android 3.1

There is no arrow pointing from Bookmark Popup. Popup is displayed in the top right side of the screen.
You need to log in before you can comment on or make changes to this bug.