36942 - Read-only text fields are partially editable

Status: VERIFIED FIXED

(SeaMonkey :: UI Design, defect, P1)

Description

[Stephen P. Morse]

1. Open the cookie viewer (tasks->personal managers->cookie manager->view stored 
cookies)

2. Select a cookie.  It's properties are displayed in text fields at the bottom 
of the dialog.

3. Type into any one of the properties.  Nothing happens.  This is correct 
behavior since the fields have a read-only attribute set to true.

4. Double click on any one of the properties.  It highlites.  I don't know if 
this is correct behavior or not for read-only fields.

5. Start typing when the property is hilited.  The text you type is appended at 
the head of the text that was already there.  This is incorrect behavior for 
read-only fields.

Note: you cannot do any deletions or arbitrary insertions into such fields 
(which is correct behavior) but you can do insertions at the head (incorrect 
behavior) as described above.

Comment 1

[Stephen P. Morse]

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

Comment 2

[Stephen P. Morse]

See also bug 25154.  These are not dups but they may be related.

Comment 3

[sairuh (rarely reading bugmail)]

could this be related to/dup of bug 31376?

Comment 4

[saari (gone)]

An event and/or ender issue. Not hyatt's.

Comment 5

[buster]

this is probably something broken in the editor's handling of "readonly"-ness.
basically, the editor is told it's readonly by having a flag set.  It then 
checks this flag before doing any writable action, like insert and delete.  
Somehow, either the text control is no longer setting the flag (see 
nsGfxTextControlFrame::AttributeChanged() and 
nsGfxTextControlFrame::InitializeTextControl())
or something broke in the editor itself and the flag is being ignored.

Or, the readonly flag is being requested in a way that the text control frame 
doesn't handle.  Does readonly work in viewer sample 8?  If so, the editor 
itself is probably fine, but the frame is to blame.

Comment 6

[John Morrison]

You can type into the readonly controls on sample #8 (input type=text,
input type=password,textarea).

Comment 7

[Stephen P. Morse]

This is no longer demonstrable with the cookie viewer because the properties 
field has been changed to no longer use html elements.  Bug still exists but 
you'll need some other way to demonstrate it, such as sample #8 as jrgm 
mentioned above.

Comment 8

[rubydoo123]

using the build from 5/2 I can reproduce this problem. Characters typed in get 
added to the front of the existing text string. here is an easy piece of code to 
drop in to test:

<input type=text name=test size=20 maxlength=18 readonly value="try to delete" 
alt="textbox readonly"> readonly textbox with 13 characters, maxlength is set to 
18, size is set to 20<br>

Comment 9

[rubydoo123]

this is a regression, setting to m17

Comment 10

[kinmoz]

Accepting bug.

Comment 11

[kinmoz]

This problem should be fixed now. We've switched to Ender-Lite since this bug
was filed, and I checked in some Readonly/Disable fixes for bug #43208.

Comment 12

[mjudge]

yepo looks good i cant type in these fields.