Change Tilt button label to "3D View" and the accesskey to "w"

RESOLVED FIXED in Firefox 12

Status

()

Firefox
Developer Tools: Inspector
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: rc, Assigned: vporof)

Tracking

({dev-doc-complete})

unspecified
Firefox 12
dev-doc-complete
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [tilt])

Attachments

(2 attachments, 5 obsolete attachments)

(Reporter)

Description

5 years ago
Follow up from bug 689920.

The accelerator key for the 3D menu button in the Page Inspector is 'M'. We should change this to the number '3' key for Windows and Linux to match the string.
(Assignee)

Comment 1

5 years ago
Created attachment 582836 [details] [diff] [review]
v1

Also changed access key to D on mac.
Assignee: nobody → vporof
Status: NEW → ASSIGNED
Attachment #582836 - Flags: review?(rcampbell)
(Assignee)

Updated

5 years ago
Whiteboard: [tilt]
There should be at least a localization note in the DTD file telling localizers that accesskey2 is for Mac...
(Assignee)

Comment 3

5 years ago
Created attachment 582888 [details] [diff] [review]
v2

Added localization note.
Attachment #582836 - Attachment is obsolete: true
Attachment #582836 - Flags: review?(rcampbell)
Attachment #582888 - Flags: review?(rcampbell)
Comment on attachment 582888 [details] [diff] [review]
v2

Alt+3 selects the third tab on Linux.
Attachment #582888 - Flags: review?(rcampbell) → review-
Any reason against using D across platforms?
(Assignee)

Comment 6

5 years ago
(In reply to Dão Gottwald [:dao] from comment #5)
> Any reason against using D across platforms?

Alt+D focuses the awesomebar on Windows.
(Assignee)

Comment 7

5 years ago
(In reply to Dão Gottwald [:dao] from comment #4)
> Comment on attachment 582888 [details] [diff] [review]
> v2
> 
> Alt+3 selects the third tab on Linux.

Maybe we can use 3 on Windows, D on Mac OS X and Linux? I just tested in Windows, Alt+3 doesn't apparently do anything.
(Reporter)

Comment 8

5 years ago
that could work but it sure is a lot of keys for one button. :)
(Assignee)

Updated

5 years ago
Summary: Change Tilt accesskey to 3 on Linux and Windows → Change Tilt accesskey to "3" on Windows and "D" on OS X and Linux
(Assignee)

Comment 9

5 years ago
Created attachment 584169 [details] [diff] [review]
v3
Attachment #582888 - Attachment is obsolete: true
Attachment #584169 - Flags: review?(rcampbell)
Attachment #584169 - Flags: review?(dao)
Comment on attachment 584169 [details] [diff] [review]
v3

- use #ifdef XP_WIN
- set inspect3DButton.accesskey to D
- add inspect3DButton.windowsAccesskey with 3
Attachment #584169 - Flags: review?(dao) → review-
(In reply to Victor Porof from comment #7)
> (In reply to Dão Gottwald [:dao] from comment #4)
> > Comment on attachment 582888 [details] [diff] [review]
> > v2
> > 
> > Alt+3 selects the third tab on Linux.
> 
> Maybe we can use 3 on Windows, D on Mac OS X and Linux? I just tested in
> Windows, Alt+3 doesn't apparently do anything.

Err, doesn't Alt+D focus the location bar on Linux as well?
(Assignee)

Comment 12

5 years ago
Created attachment 584175 [details] [diff] [review]
v4

Addressed Dão's review comments.
Attachment #584175 - Flags: review?(rcampbell)
Attachment #584175 - Flags: review?(dao)
Comment on attachment 584175 [details] [diff] [review]
v4

-> comment 11 :(
Attachment #584175 - Flags: review?(dao) → review-
(Assignee)

Comment 14

5 years ago
(In reply to Dão Gottwald [:dao] from comment #13)
> Comment on attachment 584175 [details] [diff] [review]
> v4
> 
> -> comment 11 :(

Just saw it. Hmm, yes, it does.
3 on Windows and Linux, D on Mac?
(Assignee)

Comment 15

5 years ago
(In reply to Victor Porof from comment #14)
> (In reply to Dão Gottwald [:dao] from comment #13)
> > Comment on attachment 584175 [details] [diff] [review]
> > v4
> > 
> > -> comment 11 :(
> 
> Just saw it. Hmm, yes, it does.
> 3 on Windows and Linux, D on Mac?

Wait, no, comment 4 :(
Any other suggestions?
Label the button "3D View" and use V?
(Assignee)

Comment 17

5 years ago
(In reply to Dão Gottwald [:dao] from comment #16)
> Label the button "3D View" and use V?

Or "3D DOM" and continue using M. Which one sounds better?
3D DOM doesn't seem to make sense to me...
(Assignee)

Comment 19

5 years ago
Created attachment 584177 [details] [diff] [review]
v5
Attachment #584169 - Attachment is obsolete: true
Attachment #584175 - Attachment is obsolete: true
Attachment #584169 - Flags: review?(rcampbell)
Attachment #584175 - Flags: review?(rcampbell)
Attachment #584177 - Flags: review?(rcampbell)
Attachment #584177 - Flags: review?(dao)
(Assignee)

Comment 20

5 years ago
Alt+V opens the View menu on Linux (probably Windows too).
I think we should WONTFIX this one.
Comment on attachment 584177 [details] [diff] [review]
v5

We need to change the entity name here... Localizers have already picked up 3D on aurora, together with random bogus access keys.
Attachment #584177 - Flags: review?(dao) → review-
(In reply to Victor Porof from comment #20)
> Alt+V opens the View menu on Linux (probably Windows too).
> I think we should WONTFIX this one.

Alt+W?
(Assignee)

Comment 23

5 years ago
(In reply to Dão Gottwald [:dao] from comment #21)
> Comment on attachment 584177 [details] [diff] [review]
> v5
> 
> We need to change the entity name here... Localizers have already picked up
> 3D on aurora, together with random bogus access keys.

Is inspect3DButton.accesskey3d a good entity name?
No, accesskey3d doesn't make sense... You could just call it inspect3DViewButton.accesskey.
(Assignee)

Comment 25

5 years ago
Created attachment 584179 [details] [diff] [review]
v6

Found a better solution.
Attachment #584177 - Attachment is obsolete: true
Attachment #584177 - Flags: review?(rcampbell)
Attachment #584179 - Flags: review?(dao)
Comment on attachment 584179 [details] [diff] [review]
v6

>+<!-- LOCALIZATION NOTE (inspect3DButton.label): This button shows an alternate
>+  -  view for the Inspector, creating a 3D visualization of the webpage. -->
>+<!ENTITY inspect3DViewButton.label     "3D View">

update the entity reference in the localization note
Attachment #584179 - Flags: review?(dao) → review+
What can we do to prevent locales from picking access keys that conflict with the menu bar?
(Assignee)

Comment 28

5 years ago
Created attachment 584180 [details] [diff] [review]
[in-fx-team] v7

Seventh time the charm :)
Attachment #584180 - Flags: review?(rcampbell)
Attachment #584180 - Flags: review?(dao)
(Assignee)

Comment 29

5 years ago
(In reply to Dão Gottwald [:dao] from comment #27)
> What can we do to prevent locales from picking access keys that conflict
> with the menu bar?

For Tilt specifically, we could add a test to check if using the access key indeed opens the visualization.

Updated

5 years ago
Attachment #584180 - Flags: review?(dao) → review+
(In reply to Victor Porof from comment #29)
> (In reply to Dão Gottwald [:dao] from comment #27)
> > What can we do to prevent locales from picking access keys that conflict
> > with the menu bar?
> 
> For Tilt specifically, we could add a test to check if using the access key
> indeed opens the visualization.

Well, we'd to run that test on localized builds... The concern also isn't limited to this single button.
(Assignee)

Comment 31

5 years ago
(In reply to Dão Gottwald [:dao] from comment #30)
> (In reply to Victor Porof from comment #29)
> > (In reply to Dão Gottwald [:dao] from comment #27)
> > > What can we do to prevent locales from picking access keys that conflict
> > > with the menu bar?
> > 
> > For Tilt specifically, we could add a test to check if using the access key
> > indeed opens the visualization.
> 
> Well, we'd to run that test on localized builds... The concern also isn't
> limited to this single button.

Filed bug 713391 for this.
(Reporter)

Comment 32

5 years ago
Comment on attachment 584180 [details] [diff] [review]
[in-fx-team] v7

W, eh? Why not!
Attachment #584180 - Flags: review?(rcampbell) → review+
(Assignee)

Comment 33

5 years ago
So can we land this? :)
(Reporter)

Comment 34

5 years ago
yes. please put [land-in-fx-team] in the status whiteboard when ready to ship.
Whiteboard: [tilt] → [tilt][land-in-fx-team]
Comment on attachment 584180 [details] [diff] [review]
[in-fx-team] v7

Landed:
https://hg.mozilla.org/integration/fx-team/rev/268524cb6639

Quick note: accesskeys are case sensitive. You have "3D View" with "W" which should have been lowercase. (unfortunately, I just noticed after landing)

For future reference, please check: https://developer.mozilla.org/en/XUL_Accesskey_FAQ_and_Policies
Attachment #584180 - Attachment description: v7 → [in-fx-team] v7

Updated

5 years ago
Whiteboard: [tilt][land-in-fx-team] → [tilt][fixed-in-fx-team]
(Assignee)

Comment 36

5 years ago
Will this make the button text be "3D View (W)"? We can do a quick follow-up.
(In reply to Victor Porof from comment #36)
> Will this make the button text be "3D View (W)"? We can do a quick follow-up.

I hope it won't.
(Assignee)

Comment 38

5 years ago
Filed bug 718458.
https://hg.mozilla.org/mozilla-central/rev/268524cb6639
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [tilt][fixed-in-fx-team] → [tilt]
Target Milestone: --- → Firefox 12
This landed with a nonsensical commit message :/
Summary: Change Tilt accesskey to "3" on Windows and "D" on OS X and Linux → Change Tilt button label to "3D View" and the accesskey to "w"
(Reporter)

Comment 41

5 years ago
Copy Checkin Comment FTL.
Keywords: dev-doc-needed
This is already documented as "3D view"
Keywords: dev-doc-needed → dev-doc-complete
This is already documented as "3D view"
You need to log in before you can comment on or make changes to this bug.