57967 - delete previous word [Ctrl-Backspace], start of buffer/line [Ctrl-Home], end of buffer/line [Ctrl-End] functionality

Status: RESOLVED FIXED

(Core :: DOM: Editor, defect, P3)

Description

[David Orme]

On Windows and in the Motif widgets that Netscape uses, Ctrl-home, Ctrl-end, and
Ctrl-backspace have the following meanings inside edit boxes:

Ctrl-home: Move cursor (and scroll) to the beginning of the buffer/line
Ctrl-end: Move cursor (and scroll) to the end of the buffer/line
Ctrl-backspace: Delete the word to the left of the cursor

These keystrokes do not appear to be bound to anything in the new XUL text
widgets.  They should behave similar to the old Windows and Motif widgets.

Comment 1

[Peter Trudelle]

->editor.  Note that we don't use Motif widgets in Netscape6.

Comment 2

[rubydoo123]

reassign to anthonyd and moving to mozilla0.9

Comment 3

[Jesse Ruderman]

ctrl-home and ctrl-end are covered by bug 56071.

Comment 4

[anthonyd]

setting to p2
anthonyd

Comment 5

[anthonyd]

*** Bug 57832 has been marked as a duplicate of this bug. ***

Comment 6

[rubydoo123]

p3/correctness, leaving in moz0.9 but not nsbeta1

Comment 7

[anthonyd]

i tihnk this is a keybinding issue, if so, its a 5 minute fix.  If not...well

anthonyd

Comment 8

[Kathleen :Brade]

updating summary

This *might* be able to be done in JS.
Can someone explain how the functionality works?  What if you are in the middle 
of a word?  What if you are between spaces?  Is there a corresponding keybinding 
for deleting the word to the right of the caret?

Comment 9

[anthonyd]

bug is fixed, but i am blocked by 73867

here are the keybindings, these are the windows ones.  the only chagnes for the 
other platforms are the modifier keys.

<handler event="keypress" keycode="VK_BACK" modifiers="control" 
command="cmd_deleteWordBackward"/>
<handler event="keypress" keycode="VK_HOME" modifiers="control" 
command="cmd_moveTop"/>
<handler event="keypress" keycode="VK_END" modifiers="control" 
command="cmd_moveBottom"/>

Comment 10

[anthonyd]

i think this was accidentally set to 0.9.1

Comment 11

[anthonyd]

moving to 0.9.1 because thats when 73867 will get looked at.

anthonyd

Comment 12

[anthonyd]

moving to 0.9.2 becaue it doesnt look like rods will have time for 50255

Comment 13

[rubydoo123]

moving to 1.0

Comment 14

[anthonyd]

assigning this to brade. when we last talked about this, i mentioned this bug, 
but i was damned if i could find it.  i know kathy was fixing up keybindings, 
and this may already be fixed and the included patch wont be needed.

anthonyd

Comment 15

[rubydoo123]

moving into 9.3, needs testing

Comment 16

[sujay]

I just tried Ctrl-backspace and we do not delete the previous word.
I used 6/25 builds on windows.

Comment 17

[rubydoo123]

talked with Kathy, she explained that this is used very often and is a basic 
step used by a lot of folks

reviewed and approved

Comment 18

[Kathleen :Brade]

linux already has keybinding for delete previous word (control-w)
I will attach a patch for adding keybinding for Mac (alt-backspace) and Windows
(ctrl-backspace).

Note that this keybinding is being added to editors, inputs, and textareas. 
However, Linux does not use this keybinding in Composer (for obvious reasons).

Akkana--please review
Kin--please sr=

Comment 19

[Kathleen :Brade]

Attachment: keybinding fixes for Mac and Windows

Comment 20

[kinmoz]

sr=[e-mail] on the 06/27/01 14:10 patch.

Comment 21

[Kathleen :Brade]

Attachment: keybinding fixes for Linux

Comment 22

[Kathleen :Brade]

Linux will also have control-backspace to delete previous word.
Kin--please sr= this part as well.
Akkana--please r= both attachments

Comment 23

[Akkana Peck]

r=akkana for both attachments.

Comment 24

[Kathleen :Brade]

fix checked into both trunk and 0.9.2 branch

Comment 25

[sujay]

I don't see this fixed in 6/29 builds...what am I doing wrong:

1) launch netscape
2) launch composer
3) enter a line of text like this: "one two three"

the caret is at the end of "three"

4) hit Ctrl-Backspace

nothing happens. Its supposed to delete previous word right?

Comment 26

[sujay]

this is not fixed at all using 6/29 branch builds...Beth also confirmed this..

REOPENing.

Comment 27

[Akkana Peck]

It does work for me on a linux trunk build pulled this morning.

Comment 28

[sujay]

I also tried on 6/29 windows trunk...still doesn't work for me.

Comment 29

[Kathleen :Brade]

Resolve this bug as fixed; Windows problem is #50255.  When that bug is fixed, 
this bug should work as expected on Windows.

If this problem is not fixed on the trunk or branch for Linux and Macintosh, 
please reopen.

Comment 30

[sujay]

verified on Mac and Linux branch. works fine..

will re-verify on Windows after 50255 is fixed.

Comment 31

[sujay]

updating status whiteboard.

Comment 32

[Wesha]

*** Bug 158647 has been marked as a duplicate of this bug. ***

Comment 33

[Wesha]

Updating summary for the sake of easier search.