Open
Bug 1381862
Opened 7 years ago
Updated 2 years ago
Review and eventually simplify the various .properties files used for key shortcuts and menuitems
Categories
(DevTools :: General, enhancement, P3)
DevTools
General
Tracking
(firefox57 fix-optional)
NEW
Tracking | Status | |
---|---|---|
firefox57 | --- | fix-optional |
People
(Reporter: ochameau, Unassigned)
References
(Blocks 1 open bug)
Details
After bug 1359855, registering a new tool is going to require contributing to two .properties files in devtools/client/locales/en-US/: * key-shortcuts.properties Contains command-key, i.e. the key code for each tool, used for registering all key shortcuts early during Firefox startup. * startup.properties Contains label and accesskey, used to display one menu item for each tool in the "Web Developer" system menu, as well as to display the tool title in the toobox tabs. Also note that menus.properties is similar to startup.properties. It allows registering menu items in the "Web developer" system menu, but for everything that is not a panel in the toolbox. Like WebIDE, scratchpad, about:debugging, ... The complexity may come from devtools/client/framework/browser-menus which has to have distinct codepaths for menus related to panels (inspector, console, ...) and menus related to global tools (about:debugging, scratchpad, ...). What about merging startup.properties and menus.properties so that all menu definitions, including label and accesskey are in one place, no matter if the tool is in toolbox or not? We could also try to merge also with key-shortcuts.properties, but it would mean loading strings that are not useful early during Firefox startup.
Updated•7 years ago
|
Priority: -- → P3
Updated•7 years ago
|
status-firefox57:
--- → fix-optional
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•