Bug 7675 - When new text fields change from visibility:hidden to visibility:visible, value doesn't display
Summary: When new text fields change from visibility:hidden to visibility:visible, val...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Forms (show other bugs)
Version: 420+
Hardware: Macintosh OS X 10.4
: P2 Normal
Assignee: Adele Peterson
URL:
Keywords:
Depends on:
Blocks: 6986 7676
  Show dependency treegraph
 
Reported: 2006-03-08 22:15 PST by Adele Peterson
Modified: 2006-03-09 15:27 PST (History)
0 users

See Also:


Attachments
test case (457 bytes, text/html)
2006-03-08 22:17 PST, Adele Peterson
no flags Details
patch (4.89 KB, patch)
2006-03-09 14:10 PST, Adele Peterson
justin.garcia: review-
Details | Formatted Diff | Diff
new patch (4.91 KB, patch)
2006-03-09 15:14 PST, Adele Peterson
adele: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adele Peterson 2006-03-08 22:15:30 PST
See test case.
Comment 1 Adele Peterson 2006-03-08 22:17:07 PST
Created attachment 6949 [details]
test case

this test starts with a new text field with visibility:hidden and a value of "Test Passed".  In the onload handler, the test field becomes visible, but the value doesn't show up.
Comment 2 Adele Peterson 2006-03-09 14:10:50 PST
Created attachment 6965 [details]
patch
Comment 3 Justin Garcia 2006-03-09 14:53:37 PST
Comment on attachment 6965 [details]
patch

The shadowdiv may have a depth > 1, use traverseNextNode instead of nextSibling
Comment 4 Adele Peterson 2006-03-09 15:14:47 PST
Created attachment 6966 [details]
new patch

Good call, Justin.  This patch uses traverseNextNode.
Comment 5 Adele Peterson 2006-03-09 15:21:57 PST
Comment on attachment 6966 [details]
new patch

Hyatt reviewed this.