Setting readonly attr to true doesn't work for input=text

VERIFIED FIXED

Status

()

P3
normal
VERIFIED FIXED
20 years ago
18 years ago

People

(Reporter: rods, Assigned: buster)

Tracking

Trunk
x86
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

20 years ago
Created attachment 1241 [details]
Test setting readonly to true
(Reporter)

Comment 1

20 years ago
The test case attached here relies on Bug #11620 being fixed, I have the fix in
my tree.
Depends on: 11620
(Assignee)

Updated

20 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

19 years ago
Assignee: buster → rods
Status: ASSIGNED → NEW
(Assignee)

Comment 2

19 years ago
I think this is fixed from the text control's point of view.  Rod, can you
please retest with a version of the file picker that works?
(Reporter)

Comment 3

19 years ago
I am executing these lines:

  if (nsHTMLAtoms::disabled == aAttribute) {
    nsAutoString val(nsFormFrame::GetDisabled(this) ? "true":"false");
    mTextContent->SetAttribute(kNameSpaceID_None, nsHTMLAtoms::readonly, val,
PR_TRUE);
  }

and it still doesn't go back to r/w, I can't click in it.
(Assignee)

Updated

19 years ago
Assignee: rods → hyatt
(Assignee)

Comment 4

19 years ago
The text control frame code is correct.  I've fixed the file upload control
frame, so it's correct now too.
According to peterl, this is Hyatt's bug now.  He's changing the semantics of
"AttributeChanged" notifications so the frame always gets called when an HTML
attribute is changed.  Once this work is done, this bug will go away on it's
own.

Updated

19 years ago
Assignee: hyatt → buster

Comment 5

19 years ago
My checkin just landed.  Try it now.
(Assignee)

Updated

19 years ago
Status: NEW → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

19 years ago
fixed.

Comment 7

19 years ago
QA Contact massive update.

Comment 8

18 years ago
Verifying
  Windows 98 build 2000-09-19-05-M18
  Linux RedHat6.2 build 2000-09-19-21-M18
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.