Developer Tools are not accessible from the app button in Windows

RESOLVED FIXED in Firefox 4.0b3

Status

()

Firefox
Menus
RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: pcwalton, Assigned: pcwalton)

Tracking

unspecified
Firefox 4.0b3
All
Windows 7
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(blocking2.0 betaN+)

Details

Attachments

(3 attachments)

(Assignee)

Description

7 years ago
The only way to access the developer tools in Windows is to either (a) turn on the menu by right-clicking or (b) use the keyboard shortcuts. This hurts discoverability, which is particularly important for the beta audience. For this reason I'm proposing that this block betaN.

I'd like to propose adding a "Developer >" submenu to the "Firefox" button on Windows. Chrome has a "Tools" menu that contains, among other things, View Source, the JavaScript Console, and the Developer Tools. We could add the Web Console, the Inspector, and the View Source option to our "Developer" submenu.
seconded. I think we talked about this during one of our meetings this week and I think it'd be a solid addition to the button menu and aid in discoverability on windows.
(Assignee)

Updated

7 years ago
Keywords: uiwanted
(Assignee)

Comment 2

7 years ago
Created attachment 461256 [details] [diff] [review]
Proposed patch.

Patch to implement this attached. It adds a "Developer >" submenu (which contains View Source, Inspect, and Web Console) below the "History >" menu and above the Customize menu. Tested on try server.
Attachment #461256 - Flags: ui-review?(limi)
Attachment #461256 - Flags: feedback?(rcampbell)
Comment on attachment 461256 [details] [diff] [review]
Proposed patch.

looks good.
Attachment #461256 - Flags: feedback?(rcampbell) → feedback+
(Assignee)

Updated

7 years ago
Attachment #461256 - Flags: review?(dolske)
(Assignee)

Comment 4

7 years ago
Created attachment 461371 [details]
Developer menu screenshot.

Screenshot attached.
Comment on attachment 461256 [details] [diff] [review]
Proposed patch.

Yup, having this in a Developer menu was indeed the plan. There are some tweaks we have to do to the menu in general soon, but I think this can land in the meantime.
Attachment #461256 - Flags: ui-review?(limi) → ui-review+
Comment on attachment 461256 [details] [diff] [review]
Proposed patch.

>+        <menu id="appmenu_developer"
>+              label="&developerMenu.label;">
>+          <menupopup>

Add |id="appmenu_developer_popup"| here. Mostly just so extensions can easily get ahold of it.


>+            <menuitem id="appmenu_pageSource"
>+                      label="&pageSourceCmd.label;"
>+                      command="View:PageSource"/>

Hmm, I was going to r- because the 3 menuitems added here do not have accesskey set, but I see that none of the appmenu entries have accesskeys.

Dao, why's that? Because it's assumed that the appmenu will be mouse-driven, and that keyboard users will be using the traditional menu structure?


>+<!ENTITY developerMenu.label "Developer">

Hmm, I almost wonder if this should this be "Develop" to fit into the traditional verb-oriented menu labels, but that sounds really awkward.
Attachment #461256 - Flags: review?(dolske) → review+
> Dao, why's that? Because it's assumed that the appmenu will be mouse-driven,
> and that keyboard users will be using the traditional menu structure?

Yes.

Why is there a separator between Page Source and Inspect?

Updated

7 years ago
Blocks: 556174
Component: Developer Tools → Menus
QA Contact: developer.tools → menus
(Assignee)

Comment 8

7 years ago
(In reply to comment #7)
> > Dao, why's that? Because it's assumed that the appmenu will be mouse-driven,
> > and that keyboard users will be using the traditional menu structure?
> 
> Yes.
> 
> Why is there a separator between Page Source and Inspect?

I added that because Page Source is under "View" in the standard menus, while "Inspect" and "Web Console" are under "Tools".
(Assignee)

Updated

7 years ago
Attachment #461256 - Flags: approval2.0?
Comment on attachment 461256 [details] [diff] [review]
Proposed patch.

a=me
Attachment #461256 - Flags: approval2.0? → approval2.0+
(Assignee)

Comment 10

7 years ago
Created attachment 461596 [details] [diff] [review]
Proposed patch (modified per dolske's comments).

Modified per dolske's comments.
(Assignee)

Updated

7 years ago
Keywords: uiwanted → checkin-needed
http://hg.mozilla.org/mozilla-central/rev/65e62e55194e
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b3

Comment 12

7 years ago
The menu item should say "Developer Tools". No need to file a bug for it, so I'm commenting here.
Blocks: 583418
Er, I managed to check in the wrong version of the patch.

The tree's closed right now, though.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Landed the differences:
http://hg.mozilla.org/mozilla-central/rev/6d7dc709017d
(In reply to comment #14)
> Landed the differences:
> http://hg.mozilla.org/mozilla-central/rev/6d7dc709017d

Should this be re-resolved as fixed then?
Status: REOPENED → RESOLVED
blocking2.0: ? → betaN+
Last Resolved: 7 years ago7 years ago
Resolution: --- → FIXED

Comment 16

7 years ago
Another thing - the access keys aren't displayed.(ctrl+shift+k, ctrl+shift+i)
Ctrl+Shift+U would fit for Page Source.
You need to log in before you can comment on or make changes to this bug.