Closed Bug 818229 Opened 12 years ago Closed 12 years ago

A GeckoPopupMenu similar to Android's PopupMenu

Categories

(Firefox for Android Graveyard :: Theme and Visual Design, defect)

ARM
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 20

People

(Reporter: sriram, Assigned: sriram)

References

Details

Attachments

(2 files, 1 obsolete file)

We have our own custom menu that is shown in a Popup on phones with s/w button. This can be re-used for the new tabs-menu -- following the Android's pattern of PopupMenu. http://developer.android.com/reference/android/widget/PopupMenu.html
Blocks: 817675
Blocks: 818238
Attached patch Patch (1/2): Move everything out (obsolete) (deleted) — Splinter Review
This is a refactor of our custom menu. 1. MenuPanel and MenuPopup will be reused -- hence moved out. 2. MenuPopup is enhanced to change the arrow position based on the anchor position and where the PopupWindow will be displayed. 3. There used to be a MenuPresenter inside GeckoApp that took care of showing the menu. As this would be needed for GeckoPopupMenu, and that it ties our custom menu to an activity, I've moved it as an interface inside GeckoMenu. Thereby, anyone can register themselves as a MenuPresenter for a particular menu, and they will take care of showing/hiding the menu.
Attachment #688435 - Flags: review?(mark.finkle)
This is the right patch.
Attachment #688435 - Attachment is obsolete: true
Attachment #688435 - Flags: review?(mark.finkle)
Attachment #688436 - Flags: review?(mark.finkle)
Attached patch Patch (2/2): Our PopupMenu (deleted) — Splinter Review
Our version of PopupMenu that closely shadows Android's PopupMenu. Why? - To support it on GB with our awesome custom menu that can show icons and also have a small little arrow pointing to the anchor. How? - Reuse and Recyle. Go green.
Attachment #688437 - Flags: review?(mark.finkle)
Comment on attachment 688436 [details] [diff] [review] Patch (1/2): Move everything out looks like a good refactor
Attachment #688436 - Flags: review?(mark.finkle) → review+
Attachment #688437 - Flags: review?(mark.finkle) → review+
Assignee: nobody → sriram
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: