Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Forkert access key i DevTools #308

Open
jakobkappel opened this issue Oct 1, 2017 · 13 comments
Open

Forkert access key i DevTools #308

jakobkappel opened this issue Oct 1, 2017 · 13 comments
Labels
bug Something isn't working Firefox

Comments

@jakobkappel
Copy link
Collaborator

Original report by Jørgen Rasmussen (Bitbucket: Joergen, GitHub: Joergen).


https://hg.mozilla.org/l10n-central/da/file/default/devtools/client/debugger.dtd#l160
Access key til "Tilføj betinget breakpoint" vises som (C). Jeg ved ikke hvorfor. Den danske er b.
B bruges også af "Tilføj breakpoint". Kan ændres til T.

@jakobkappel
Copy link
Collaborator Author

Det var godtnok en underlig fejl.

Jeg kigger på det.

@jakobkappel
Copy link
Collaborator Author

Den engelske fil ser sådan ud:

<!-- LOCALIZATION NOTE (debuggerUI.seMenuBreak): This is the text that
  -  appears in the source editor context menu for adding a breakpoint. -->
<!ENTITY debuggerUI.seMenuBreak     "Add Breakpoint">
<!ENTITY debuggerUI.seMenuBreak.key "B">

<!-- LOCALIZATION NOTE (debuggerUI.seMenuCondBreak): This is the text that
  -  appears in the source editor context menu for adding a conditional
  -  breakpoint. -->
<!ENTITY debuggerUI.seMenuCondBreak     "Add Conditional Breakpoint">
<!ENTITY debuggerUI.seMenuCondBreak.key "B">

<!-- LOCALIZATION NOTE (debuggerUI.seMenuBreak): This is the text that
  -  appears in the source editor context menu for editing a breakpoint. -->
<!ENTITY debuggerUI.seEditMenuCondBreak     "Edit Conditional Breakpoint">
<!ENTITY debuggerUI.seEditMenuCondBreak.key "B">


Skal vi prøve at gøre det samme (altså ændre til "b" de tre steder)?

@jakobkappel
Copy link
Collaborator Author

Original comment by Jørgen Rasmussen (Bitbucket: Joergen, GitHub: Joergen).


Måske skulle vi sætte den på hold. Jeg kan ikke regne ud, hvad der foregår.
breakpoint1.png
breakpoint2.png
brepoint3.png
"Rediger breakpoint" findes kun et sted og har ingen access key.
https://hg.mozilla.org/l10n-central/da/file/default/devtools/client/debugger.properties#l341

@jakobkappel
Copy link
Collaborator Author

Nej - men det er åbenbart ikke engang det, der er skyld i problemet: https://bugzilla.mozilla.org/show_bug.cgi?id=1403925

@jakobkappel
Copy link
Collaborator Author

Original comment by Jørgen Rasmussen (Bitbucket: Joergen, GitHub: Joergen).


Så kan vi vel lukke dette issue?

@jakobkappel
Copy link
Collaborator Author

Ja, jeg lukker - så må vi vente på at de fikser problemet.

@jakobkappel
Copy link
Collaborator Author

@jakobkappel
Copy link
Collaborator Author

forkert status

@jakobkappel jakobkappel added major Firefox bug Something isn't working labels Jun 29, 2020
@jakobkappel jakobkappel removed the major label Jul 13, 2020
@jakobkappel
Copy link
Collaborator Author

@Joergen, hvordan ser det hér ud nu på Windows?

Så vidt jeg kan se burde alt være godt efter at "Tilføj betinget breakpoint" er erstattet med "Tilføj betingelse" (der har "t" som accesskey: https://hg.mozilla.org/l10n-central/da/file/default/devtools/client/debugger.properties#l532)

@Joergen
Copy link
Collaborator

Joergen commented Jul 13, 2020

"Tilføj betingelse" er i en anden genvejsmenu. Jeg tog nogle skærmbilleder fra samme sted som de oprindelige skærmbilleder.
Som du kan se, er der stadig problemer med med access keyen til "Tilføj breakpoints" og "Fjern breakpoints".

breakpoint1
breakpoint2
breakpoint3
breakpoint4

Du har to menupunkter med "Deaktiver...", som bruger D, som access key.

Så er der dette skærmbillede fra en anden genvejsmenu, hvor menupunkterne blandes med andre menupunkter (suk).

breakpoint5
Her kan editor.removeLogPoint.accesskey=V ændres til L

@jakobkappel
Copy link
Collaborator Author

Tak for skæmbillederne. Men sikke en rodedynge.

Jeg er for træt i hovedet lige nu til at turde kaste mig ud i det. Så jeg vender tilbage med en lidt mere frisk hjerne i morgen :)

@jakobkappel
Copy link
Collaborator Author

Det fik jeg så ikke tid til. Jeg håber, at det kan lykkes i dag.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Firefox
Projects
None yet
Development

No branches or pull requests

2 participants