285529 - Option-forward delete does not work in page text input fields

Status: RESOLVED FIXED

(Camino Graveyard :: HTML Form Controls, defect)

Description

[Daniel Wilson]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8b2) Gecko/20050304 Camino/0.8+
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8b2) Gecko/20050304 Camino/0.8+

Pressing Option-forward delete should delete the word to the right of the cursor.

Reproducible: Always

Steps to Reproduce:
1. Input text into a page form.
2. Position the text input cursor to the immediate left of a word.
3. Press Option-forward delete.

Actual Results:  
Error beep.

Expected Results:  
The word to the immediate right of the cursor should be deleted.

Comment 1

[Jasper]

Yup I can confirm.

Comment 2

[Chris Lawson (gone)]

Do we have any idea why this might be happening? This would be really nice polish for 1.0 if someone can come up with a patch fairly quickly. I'd be willing to give it a shot if someone can point me in the right direction.

Comment 3

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

It's probably platformHTMLBindings (or whatever that file is called), so it can't make 1.0.  Wevah, Simon, is that right?

See also bugs like bug 299565, bug 188602, bug 175620.

Comment 4

[Wevah (Nate Weaver)]

Yeah, I'm pretty sure that's the case. As usual, we don't get it for free since it's not a real OS X text field.

Comment 5

[Chris Lawson (gone)]

Would that be this file?

http://lxr.mozilla.org/mozilla/source/content/xbl/builtin/mac/platformHTMLBindings.xml

cl

Comment 6

[Uri Bernstein]

This is an issue with Firefox as well. Should I report a separate bug, or is this one really a core bug?

Comment 7

[Chris Lawson (gone)]

That file, as I understand it, gets pulled into both Camino and Firefox, so when it's fixed, it'll be fixed in both instances, I think.

cl

Comment 8

[Uri Bernstein]

Attachment: patch

Comment 9

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 228952 [details] [diff] [review]
patch

Looks reasonable.  sr=bzbarsky

Comment 10

[Samuel Sidler (old account; do not CC)]

We should get this on the trunk before requesting branch approval.

Comment 11

[Uri Bernstein]

Checked in:
Checking in content/xbl/builtin/mac/platformHTMLBindings.xml;
/cvsroot/mozilla/content/xbl/builtin/mac/platformHTMLBindings.xml,v  <--  platformHTMLBindings.xml
new revision: 1.18; previous revision: 1.17
done

Comment 12

[Samuel Sidler (old account; do not CC)]

Comment on attachment 228952 [details] [diff] [review]
patch

Requesting approval for Uri.

This is a Mac-only change to platformHTMLBindings.xml which will add opt-delete (forward delete to be specific) as a key command. It has r/sr from aaronlev/bz, respectively. (Note that aaronlev is the module owner for this code.)

Comment 13

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 228952 [details] [diff] [review]
patch

a=drivers, please land on the 181branch

Comment 14

[Uri Bernstein]

Checked into 1.8 branch:

Checking in content/xbl/builtin/mac/platformHTMLBindings.xml;
/cvsroot/mozilla/content/xbl/builtin/mac/platformHTMLBindings.xml,v  <--  platformHTMLBindings.xml
new revision: 1.16.16.2; previous revision: 1.16.16.1
done