|Summary:||RSS search field, Dashboard widgets failing due to CSS exception|
|Product:||WebKit||Reporter:||Darin Adler <darin>|
|Component:||DOM||Assignee:||Darin Adler <darin>|
|OS:||OS X 10.4|
Description Darin Adler 2005-06-19 23:44:55 PDT
5/17/05 2:54 PM John Sullivan: To reproduce: 1. go to feed://slashdot.org/index.rss (or any other RSS feed page) in Safari 2. At this point, note the following console message:  feed://slashdot.org/index.rss:Error - CSS exception 0 3. Type a character into the search field. Note that the following message appears in the console: (event handler):CSS exception 0 4. Continue typing characters into the search field until there are no articles found. Note that in addition to the CSS exception message for each typed character, a previous/more selector thingy appears on the page that should be blank except for the big gray "No Articles" text. (Maybe this thingy might be some holdover from an earlier design that's normally always hidden?) 5. Backspace until there are found articles again. Note that the big gray "No Articles" text isn't erased, and neither is the mysterious previous/more selector thingy. See screenshot.
Comment 1 Darin Adler 2005-06-19 23:45:45 PDT
In Radar: <rdar://problem/4122131> REGRESSION (412+): Typing in RSS page's search field causes CSS exception in console and bad display
Comment 2 Darin Adler 2005-06-19 23:46:44 PDT
Comment 3 Darin Adler 2005-06-19 23:47:34 PDT
Created attachment 2482 [details] Patch that ignores exceptions when setting a CSS property via the style.xxx = yyy syntax
Comment 4 Darin Adler 2005-06-19 23:47:59 PDT
Comment on attachment 2482 [details] Patch that ignores exceptions when setting a CSS property via the style.xxx = yyy syntax No layout test yet, but I think maybe still ready for review.
Comment 5 Maciej Stachowiak 2005-06-20 00:51:06 PDT
Comment on attachment 2482 [details] Patch that ignores exceptions when setting a CSS property via the style.xxx = yyy syntax Code looks good, just needs a test case added (or an argument why one is impossible or unneeded).
Comment 6 Darin Adler 2005-06-20 17:44:59 PDT
Created attachment 2512 [details] First cut at a test case I should probably roll this into the patch.
Comment 7 Darin Adler 2005-06-20 19:23:14 PDT
Created attachment 2514 [details] Newer patch, includes test case and ChangeLog
Comment 8 Darin Adler 2005-06-21 10:02:46 PDT
Comment on attachment 2514 [details] Newer patch, includes test case and ChangeLog Meant to request review, not review my own patch!