Closed Bug 1309561 Opened 8 years ago Closed 8 years ago

Rename *-key messages to *-command

Categories

(L20n :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: stas, Assigned: zbraniecki)

References

Details

(Whiteboard: [gecko-l20n])

Attachments

(2 files, 1 obsolete file)

For example:

    new-navigator-key =
        [xul/key]        N

should become:

    new-navigator-command =
        [xul/key]        N
Blocks: 1291693
Assignee: nobody → gandalf
Attached patch patch v1 (obsolete) — Splinter Review
Renaming -key to -command plus one OS()
Attachment #8804056 - Flags: review?(stas)
Attachment #8804056 - Attachment is obsolete: true
Attachment #8804056 - Flags: review?(stas)
Comment on attachment 8804057 [details]
Bug 1309561 - Rename *-key messages to *-command;

https://reviewboard.mozilla.org/r/88204/#review87340

This should not land as it is right now, but the fixes are simple. Thanks.

::: browser/base/content/browser-menubar.inc
(Diff revision 1)
> -#else
> -#ifdef XP_MACOSX
> -                          data-l10n-id="quit-application-menuitem-mac"
> -#else
>                            data-l10n-id="quit-application-menuitem"
> -#endif

What is the reason for introducing this OS() in this patch?

::: browser/base/content/browser-menubar.inc:94
(Diff revision 1)
>                            data-l10n-id="quit-application-menuitem"
> -#endif
>  #ifdef XP_UNIX
>                            key="key_quitApplication"
>  #endif
>  #endif

There's one #endif too many here which breaks the build.  Try building locally or pushing to try.

::: browser/base/content/browser-sets.inc:238
(Diff revision 1)
> -    <key id="key_close" data-l10n-id="close-key" command="cmd_close" modifiers="accel"/>
> -    <key id="key_closeWindow" data-l10n-id="close-key" command="cmd_closeWindow" modifiers="accel,shift"/>
> +    <key id="key_close" data-l10n-id="close-command" command="cmd_close" modifiers="accel"/>
> +    <key id="key_closeWindow" data-l10n-id="close-command" command="cmd_closeWindow" modifiers="accel,shift"/>
>      <key id="key_toggleMute" key="&toggleMuteCmd.key;" command="cmd_toggleMute" modifiers="control"/>
> -    <key id="key_undo" data-l10n-id="undo-key" modifiers="accel"/>
> +    <key id="key_undo" data-l10n-id="undo-command" modifiers="accel"/>
>  #ifdef XP_UNIX
> -    <key id="key_redo" data-l10n-id="undo-key" modifiers="accel,shift"/>
> +    <!-- On unix, redo is ctrl+shift+z => undo key + shift -->

I would phrase that as "On Unix, Redo's command key is the Undo's command key + Shift."

::: browser/locales/en-US/browser/menubar.ftl:134
(Diff revision 1)
>      [xul/key]        A
>  
>  find-on-menuitem =
>      [xul/label]      Find in This Page…
>      [xul/accesskey]  F
> -find-on-key =
> +find-on-command1 =

This should be `find-on-command =`.

::: browser/locales/en-US/browser/menubar.ftl:139
(Diff revision 1)
> -find-on-key =
> +find-on-command1 =
>      [xul/key]        f
>  find-again-menuitem =
>      [xul/label]      Find Again
>      [xul/accesskey]  g
> -find-again-key1 =
> +find-again-command =

And this should be `find-again-command1 =`.
Attachment #8804057 - Flags: review?(stas) → review-
manual patch in case mozreview one is too confusing.
Comment on attachment 8804339 [details] [diff] [review]
key2command.patch

I'm not able to r+ in mozreview, so r=me on this squashed patch.  Thanks!
Attachment #8804339 - Flags: review+
Landed: http://hg.mozilla.org/projects/larch/rev/cd69a5db20d5
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
(In reply to Zibi Braniecki [:gandalf][:zibi] from comment #8)
> Landed: http://hg.mozilla.org/projects/larch/rev/cd69a5db20d5

This is not the patch that I r+'ed.  It contains the OS() builtin and removes the #ifdef in browser/base/content/browser-menubar.inc.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
https://hg.mozilla.org/projects/larch/rev/0c5eca45fc7f3eee4295e70f77110869a0196a87
Bug 1309561 - Follow-up to the previous patch. Land the review feedback fixes. r=stas
Aaand, follow-up landed: http://hg.mozilla.org/projects/larch/rev/0c5eca45fc7f
Status: REOPENED → RESOLVED
Closed: 8 years ago8 years ago
Resolution: --- → FIXED
Comment on attachment 8804057 [details]
Bug 1309561 - Rename *-key messages to *-command;

https://reviewboard.mozilla.org/r/88204/#review90092
Attachment #8804057 - Flags: review?(stas)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: