search

MozillaDanmark / mozilla-dk

Danske versioner af produkter og hjemmesider fra Mozilla
https://pontoon.mozilla.org/da/
2 stars 0 forks source link

Forkert access key i DevTools #308

Open jakobkappel opened 6 years ago

jakobkappel commented 6 years ago

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 commented 6 years ago

Det var godtnok en underlig fejl.

Jeg kigger på det.

jakobkappel commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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

jakobkappel commented 6 years ago

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

Så kan vi vel lukke dette issue?

jakobkappel commented 6 years ago

Hov, jo, de er oversat: https://bitbucket.org/mozilladanmark/mozilla-dk/src/123154a4340d70a5f6a0a3f0c92a21fa45661c66/devtools/client/debugger.dtd?at=default&fileviewer=file-view-default#debugger.dtd-153

jakobkappel commented 6 years ago

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

jakobkappel commented 6 years ago

Venter på https://bugzilla.mozilla.org/show_bug.cgi?id=1403925

jakobkappel commented 6 years ago

forkert status

jakobkappel commented 3 years ago

@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 commented 3 years ago

"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 commented 3 years ago

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 commented 3 years ago

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