1150786 - Add the localized string and accesskey for "Debug" add-on button

Status: RESOLVED FIXED

(Toolkit :: Add-ons Manager, defect)

Description

[YF (Yang)]

Attachment: patch v1

The "Disable" button have "D", so the to "E". 


Use the Alt+Shift+E to test it in add-on details pane on Windows.

Comment 1

[Dave Townsend [:mossop]]

Comment on attachment 8587809 [details] [diff] [review]
patch v1

Review of attachment 8587809 [details] [diff] [review]:
-----------------------------------------------------------------

I think I'd rather go with B or G here. E conflicts with the Enable button and while we never display the two buttons at the same time at the moment we might want to do that in the future to allow for debugging an add-on's startup code.

Comment 2

[YF (Yang)]

Attachment: Patch v2 - "B"

You are right. 

This "B" may to trigger the Bookmarks menu in the past, I hope not many people use it in this interface.


p.s. maybe the "U" can also be considered, if we think it is not a commonly shortcut, the like we have &Install.

Comment 3

[Dave Townsend [:mossop]]

Comment on attachment 8588831 [details] [diff] [review]
Patch v2 - "B"

Review of attachment 8588831 [details] [diff] [review]:
-----------------------------------------------------------------

Let's go with this.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/fx-team/rev/6d49087d2a35

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/6d49087d2a35