Bookmark Popup arrowbox should point up in tablet mode

VERIFIED FIXED in Firefox 9

Status

Firefox for Android Graveyard
General
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: mfinkle, Assigned: wesj)

Tracking

(Blocks: 3 bugs)

Trunk
Firefox 9
Dependency tree / graph

Details

(Whiteboard: [inbound])

Attachments

(2 attachments)

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
(Assignee)

Comment 1

7 years ago
Created attachment 557638 [details] [diff] [review]
Patch v1

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+
(Assignee)

Updated

7 years ago
Blocks: 684277, 684278
Blocks: 684397
http://hg.mozilla.org/mozilla-central/rev/9ca2aaaae7bf
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 9
Created attachment 563006 [details]
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).

Comment 8

7 years ago
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.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.