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)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 20
People
(Reporter: sriram, Assigned: sriram)
References
Details
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
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
Assignee | ||
Comment 1•12 years ago
|
||
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)
Assignee | ||
Comment 2•12 years ago
|
||
This is the right patch.
Attachment #688435 -
Attachment is obsolete: true
Attachment #688435 -
Flags: review?(mark.finkle)
Attachment #688436 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 3•12 years ago
|
||
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 4•12 years ago
|
||
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+
Updated•12 years ago
|
Attachment #688437 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 5•12 years ago
|
||
Comment 6•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/7cefbc286acb
https://hg.mozilla.org/mozilla-central/rev/62352bbc0547
Assignee: nobody → sriram
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•