Closed Bug 11868 Opened 21 years ago Closed 21 years ago

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

Categories

(Core :: Layout: Form Controls, defect, P3)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: rods, Assigned: buster)

References

Details

Attachments

(1 file)

No description provided.
The test case attached here relies on Bug #11620 being fixed, I have the fix in
my tree.
Depends on: 11620
Status: NEW → ASSIGNED
Assignee: buster → rods
Status: ASSIGNED → NEW
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?
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: rods → hyatt
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.
Assignee: hyatt → buster
My checkin just landed.  Try it now.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
fixed.
QA Contact massive update.
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.