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)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha1
People
(Reporter: allan, Unassigned)
Details
(Keywords: testcase)
Attachments
(2 files)
|
1.46 KB,
application/xhtml+xml
|
Details | |
|
1.35 KB,
patch
|
peterv
:
review+
peterv
:
superreview+
|
Details | Diff | Splinter Review |
the disabled state for a control is restored for a default input, but not for a
password input? Is that correct behaviour?
|
Reporter | |
Comment 1•19 years ago
|
||
|
|
Reporter | |
Comment 2•19 years ago
|
||
it also goes for other input controls. Found it in this testcase:
http://beaufour.dk/tmp/enabled-true.xhtml
|
||
Comment 3•19 years ago
|
||
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
|
|
||
Comment 4•19 years ago
|
||
Attachment #194481 -
Flags: superreview?(peterv)
Attachment #194481 -
Flags: review?(peterv)
|
||
Updated•19 years ago
|
Attachment #194481 -
Flags: superreview?(peterv)
Attachment #194481 -
Flags: superreview+
Attachment #194481 -
Flags: review?(peterv)
Attachment #194481 -
Flags: review+
|
|
Reporter | |
Comment 5•19 years ago
|
||
This fixes input.
Shouldn't the disabled state also be restored for textarea, button, select,
option, and optgroup?
|
|
||
Comment 6•19 years ago
|
||
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
|
||
Comment 7•19 years ago
|
||
(In reply to comment #6)
> We have existing bugs on the other form controls, actually.
Bug 277724 is one of them.
|
||
Comment 8•15 years ago
|
||
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).
|
||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•