Closed Bug 728128 Opened 12 years ago Closed 12 years ago

Make behavior of the "hardware" buttons configurable

Categories

(Firefox OS Graveyard :: General, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
DeveloperPhone

People

(Reporter: cjones, Assigned: vingtetun)

References

Details

Attachments

(1 file)

Currently the "back" button is fired as the escape key and the home button as the home key.  We probably don't need that to be configurable.

But the menu button is dispatched to content specially (with a nonstandard DOM event), and the search button enables a special graphics debug mode.  In production phones we don't want the debug mode enable-able.  And we'll want to remap the menu button to "view-source" for Gaia.

So all this should be configurable.  Prefs seem ok.  A magical button-layout.json alongside homescreen.html, read by the chrome package on startup, is another option.
Attached patch PatchSplinter Review
The patch add 2 preferences:
 b2g.keys.menu.enabled (true by default)
 b2g.keys.search.enabled (false by default)
Assignee: nobody → 21
Status: NEW → ASSIGNED
Attachment #600208 - Flags: review?(jones.chris.g)
Attachment #600208 - Flags: review?(jones.chris.g) → review+
Can you land this soon?  Our friends are eagerly awaiting it.  I can if you're not.
(In reply to Chris Jones [:cjones] [:warhammer] from comment #2)
> Can you land this soon?  Our friends are eagerly awaiting it.  I can if
> you're not.

I will land it in a few minutes if possible.
(In reply to Vivien Nicolas (:vingtetun) from comment #3)
> (In reply to Chris Jones [:cjones] [:warhammer] from comment #2)
> > Can you land this soon?  Our friends are eagerly awaiting it.  I can if
> > you're not.
> 
> I will land it in a few minutes if possible.

Hum. By landing do you mean git-mc? If so I probably do to need a branch first, that will take some times.
Straight to upstream m-c is fine.
https://hg.mozilla.org/mozilla-central/rev/16283b69a10a
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: