312089 - document Delete key as Back/Shift+Delete as Forward on Mac

Status: RESOLVED FIXED

(Firefox Graveyard :: Help Documentation, defect)

Description

[Steffen Wilberg]

Bug 220590 made the Delete (a.k.a Backspace) key go back on Mac as well.
Currently trunk only.

Comment 1

[Steffen Wilberg]

Now it's checked in on the branch as well.

Comment 2

[Jeff Walden (gone starting June 8)]

Attachment: Patch

As the shortcut works on all platforms, I see no reason not to move it outside
the platform-specific parts entirely.  If you disagree, say so and we can
decide what we want to do instead.

Comment 3

[Jeff Walden (gone starting June 8)]

Comment on attachment 199615 [details] [diff] [review]
Patch

D'oh, it's Delete and not Backspace.  Respinning...

Comment 4

[Jeff Walden (gone starting June 8)]

Attachment: Use Del, not Backspace

Comment 5

[Steffen Wilberg]

Comment on attachment 199618 [details] [diff] [review]
Use Del, not Backspace

>         <span class="unix">
>           &altKey;+<kbd>Right Arrow</kbd><br/>
>-          &shiftKey;+<kbd>Backspace</kbd>
>+          &shiftKey;+<kbd>Backspace</kbd><br/>
>           &accelKey;+<kbd>]</kbd>
>         </span>
Ouch, thanks for catching this!

r=me.

Comment 6

[Steffen Wilberg]

Actually, I think that key, which is Backspace on other platforms, is called
"Delete". "Del" is the Delete key on other platforms.

Comment 7

[Mark Mentovai]

Steffen's right.  On pure Mac keyboards, the key in question is traditionally
labeled "delete", spelled out in full.

Comment 8

[Asa Dotzler [:asa]]

Comment on attachment 199618 [details] [diff] [review]
Use Del, not Backspace

Looks like we need a new patch here. Please request approval when you've got
that. Thanks.

Comment 9

[Steffen Wilberg]

Comment on attachment 199618 [details] [diff] [review]
Use Del, not Backspace

> Looks like we need a new patch here.
Nah, the person who checks this in merely needs to add "ete" after "Del" here:
>+          <kbd>Del</kbd>

>+          &shiftKey;+<kbd>Del</kbd>
That's it. Rerequesting approval.

Comment 10

[Scott MacGregor]

Comment on attachment 199618 [details] [diff] [review]
Use Del, not Backspace

just don't forget to add the  "ete".

Comment 11

[Jeff Walden (gone starting June 8)]

(In reply to comment #10)
> just don't forget to add the  "ete". 

I didn't.

Patch checked in on branch and trunk; post to n.p.m.l10n forthcoming.