Closed Bug 306616 Opened 19 years ago Closed 19 years ago

disabled is not restored for all types of input controls on refresh

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9alpha1

People

(Reporter: allan, Unassigned)

Details

(Keywords: testcase)

Attachments

(2 files)

the disabled state for a control is restored for a default input, but not for a
password input? Is that correct behaviour?
Attached file Testcase
it also goes for other input controls. Found it in this testcase:
http://beaufour.dk/tmp/enabled-true.xhtml
The issue is that we never restore state for passwords, which is wrong. We don't
want to restore the value, but I think we do want to restore the disabled state...
Assignee: dveditz → general
Status: UNCONFIRMED → NEW
Component: Form Manager → DOM
Ever confirmed: true
QA Contact: ian
Attached patch FixSplinter Review
Attachment #194481 - Flags: superreview?(peterv)
Attachment #194481 - Flags: review?(peterv)
Attachment #194481 - Flags: superreview?(peterv)
Attachment #194481 - Flags: superreview+
Attachment #194481 - Flags: review?(peterv)
Attachment #194481 - Flags: review+
This fixes input.

Shouldn't the disabled state also be restored for textarea, button, select,
option, and optgroup?
Fixed on trunk.

We have existing bugs on the other form controls, actually.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9alpha
(In reply to comment #6)
> We have existing bugs on the other form controls, actually.
Bug 277724 is one of them.
A summary explanation of why Firefox maintains disabled state would have been nice - at least for programmers to know about this semantics and to work around it.

See also bug #293733.

New bugs regarding "persistence of disabled state" bug #468614 (filed in 2008) and bug #502812 (filed in 2009).
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: