Closed Bug 56982 Opened 24 years ago Closed 24 years ago

Form fields are reset when switching theme

Categories

(Core Graveyard :: Skinability, defect, P3)

PowerPC
Mac System 8.5

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: mpt, Assigned: bugs)

Details

(Keywords: dataloss)

Build: 2000101620, Mac OS 9.0 To reproduce: * Visit <http://bugzilla.mozilla.org/show_bug.cgi?id=44437>. * Check the `Remove selected CCs' checkbox. * Select one of the radio buttons (e.g. `Reopen bug'). * Enter a long and detailed comment in the `Additional Comments:' field. * Switch your theme. What should happen: * The theme gets switched. What actually happens: * The theme gets switched. * All form elements on the page are reset to their original states -- the checkbox is unchecked, the radio buttons are reset, and your comment is deleted.
Note that this is not the same as bug 43350, as it applies to all form elements, not just text fields.
Keywords: dataloss
Say, Matthew -- this works for me with the 20001030am branch builds on mac/linux/win32 --- radio/checkbox/text/textarea/select all come back to the values/state to which they had been changed before the skin switch. Works for you, too?
The page is also reloaded from the web server when I switch themes. Is that a different bug?
The reload is intentional, since the current document is (I believe) torn down in the process of the skin switch. Depending on cache settings (modulo cache bugs) then this would likely cause an HTTP GET, but under the same conditions as going Back to a previous URL, including a form page. [Of course, that is just what I believe, and I could be completely on crack]. By the way, mpt -- when I looked at this on 11/01, this "Form fields" bug was working correctly -- the form fields state is preserved during a skin switch; the fields are not reset. And you? when the skin is switched.
Hmm. I nominate myself worst QA ever. Yeah, this works for me also. It has for awhile. I thought hyatt's state and preservation fix (which included the current webpage hack) fixed this awhile back.
Pretriage of skinnability bugs, marking fixed, this looks like it still works.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
QA Contact: blakeross → pmac
Verified on Mac (commercial build: 2001-06-04-08-0.9.1)
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.