267196 - Start snippets: Ctrl-T should be Ctrl+T

Status: RESOLVED FIXED

(www.mozilla.org :: General, defect)

Description

[Daniel Wang]

html/start-snippets/en/snippet-5.html reads
  Know your shortcuts. Ctrl-Tab or Ctrl-PgDn to move between tabs, Ctrl-L to
  access location bar, and Ctrl-M to compose a new email message. Learn more.

html/start-snippets/en/snippet-6.html
  Know your shortcuts. Ctrl-Tab or Ctrl-PgDn to move between tabs, Ctrl-L to
  access location bar, and Ctrl-M to compose a new email message. Learn more.

This is simply not how we write shortcuts. Please change - to +

Comment 1

[Anne (:annevk)]

I'm willing to fix this. Can someone point me to the person who should give
permission for the change?

Comment 2

[Anne (:annevk)]

CC'ing Pike, who might know something about it.

Comment 3

[Axel Hecht]

This affects all localisations, dbaron did some general scope fixes.
I'd suggest to fix this for everybody and announce the change on .l10n.

Comment 4

[Stefan [:stefanh]]

Anne, while you do this -- could you fix the mac accel keys as well? Since I 
suspect that it would be harder to make Google fetch the pages two times than 
one...

Comment 5

[Anne (:annevk)]

Stefan, not until the discussion in bug 268926 is over and people agreed upon a
solution.

Comment 6

[Axel Hecht]

The mac accel stuff is a separate bug, and is far more complicated.

Don't mix the two, as with the mac stuff, we likely will need a google permission.

Comment 7

[Stefan [:stefanh]]

(In reply to comment #6)
> The mac accel stuff is a separate bug, and is far more complicated.
> 
> Don't mix the two, as with the mac stuff, we likely will need a google permission.

Fair enough.

Comment 8

[Anne (:annevk)]

I only checked and editted 5 and 6. I did not fix km and ko which characters I
could not load in my editor.

Comment 9

[Axel Hecht]

fixed de/snippet-3.html

Comment 10

[Anne (:annevk)]

Marking this fixed. There is a follow-up snippet bug which has a patch to solve
the remaining issues: bug 289538.