Closed
Bug 606565
Opened 14 years ago
Closed 14 years ago
create android style menu-button menu
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(fennec-)
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
fennec | - | --- |
People
(Reporter: madhava, Assigned: mbrubeck)
References
Details
Attachments
(3 files, 2 obsolete files)
When the user presses the Android hardware menu button, we can put up a native-style menu, that will link to bits of the Fennec UI, as in this mockup: http://www.flickr.com/photos/madhava_work/5105182033/sizes/l/
Reporter | ||
Comment 1•14 years ago
|
||
Well need android-style versions of our icons for this. CCing Sean.
Reporter | ||
Updated•14 years ago
|
tracking-fennec: --- → ?
Assignee | ||
Updated•14 years ago
|
Hardware: x86 → All
Updated•14 years ago
|
Assignee: nobody → mbrubeck
Depends on: 608594
Comment 3•14 years ago
|
||
I like having a direct shortcuts associated to this button. We can have 2 actions associated with this: - normal tap: show the proposed menu - long tap: directly show the site menu I've opened a platform bug (bug 608594) and a fix for allowing 2 actions. Btw, we can also have 2 actions for the back button (the stock browser does that) but I'm not sure which one to assign to the long tap.
Depends on: 608262
Reporter | ||
Comment 4•14 years ago
|
||
Per discussion with mbrubeck in IRC - let's continue to display the titlebar, if it's off screen, when we show the menu at the bottom of the screen. So, this is like the current fennec behavior, except without the site menu deployed.
Assignee | ||
Comment 5•14 years ago
|
||
Assignee | ||
Comment 6•14 years ago
|
||
A very early working prototype. Still to do: - Icons. - Two-row layout for portrait mode. - Handle overflow with a "More" button and a popup. - Different menus for different UI screens. - Animated transition?
Updated•14 years ago
|
tracking-fennec: 2.0b3+ → 2.0-
Comment 7•14 years ago
|
||
We would take a minimal impl for Fennec 4
Reporter | ||
Comment 8•14 years ago
|
||
For the minimal implementation, I'd personally take - no overflow handling (can pick this up in a followup point release) - no animated transition, though it pains me to say it - just this menu for the main screen, not others for other screens (again, fuller implementation next time) We _will_ need portrait and landscape versions, and the icons.
Assignee | ||
Comment 10•14 years ago
|
||
This patch implements a menu for the main screen only. Pressing the menu button in the tool panel or awesomescreen does nothing. The menu has correct layouts in landscape and portrait for up to 6 buttons. It does not handle overflow for >6 buttons. (There are four buttons by default, but add-ons could insert more.) This patch uses existing toolbar icons. I had to resize the "add-ons" icon to give it the same dimensions as the others. (The image itself is unchanged; I just centered it in a slightly larger transparent background.)
Attachment #487748 -
Attachment is obsolete: true
Attachment #487752 -
Attachment is obsolete: true
Attachment #489226 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 11•14 years ago
|
||
Assignee | ||
Comment 12•14 years ago
|
||
Updated•14 years ago
|
Flags: in-testsuite?
Flags: in-litmus?
Comment 13•14 years ago
|
||
Comment on attachment 489226 [details] [diff] [review] patch using "android" in the code and xul is less than desirable. This is the closest thing we have to a Firefox "menu" like on desktop, which is called "appmenu". Could we change the ID's to appmenu-* and the JS class to ApplicationMenu? Remember to add the string for "Site Options"
Attachment #489226 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 14•14 years ago
|
||
Here are some specs for this feature: https://wiki.mozilla.org/Mobile/Projects/PlatformIntegration#App_Menu
Assignee | ||
Comment 15•14 years ago
|
||
(In reply to comment #13) > using "android" in the code and xul is less than desirable. This is the closest > thing we have to a Firefox "menu" like on desktop, which is called "appmenu". > Could we change the ID's to appmenu-* and the JS class to ApplicationMenu? > > Remember to add the string for "Site Options" Pushed, with both of these fixes: http://hg.mozilla.org/mobile-browser/rev/4fe8d35b7f59 I will file followup bugs for better icons, handling overflow, animated transitions, and anything else that comes up. (Only the icons should be blocking release.)
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment 16•14 years ago
|
||
verified FIXED on build: Mozilla/5.0 (Android; Linux armv71; rv:2.0b8pre) Gecko/20101110 Namoroka/4.0b8pre Fennec/4.0b3pre
Status: RESOLVED → VERIFIED
Flags: in-litmus? → in-litmus?(ayanshah62)
Comment 17•14 years ago
|
||
https://litmus.mozilla.org/show_test.cgi?id=13798 https://litmus.mozilla.org/show_test.cgi?id=13799 https://litmus.mozilla.org/show_test.cgi?id=13800 https://litmus.mozilla.org/show_test.cgi?id=13802 https://litmus.mozilla.org/show_test.cgi?id=13803 https://litmus.mozilla.org/show_test.cgi?id=13804
Updated•14 years ago
|
Flags: in-litmus?(ayanshah62) → in-litmus+
You need to log in
before you can comment on or make changes to this bug.
Description
•