|Summary:||REGRESSION: Allow setting the checked attribute in js and in markup for unnamed radio buttons (dominoes)|
|Product:||WebKit||Reporter:||Beth Dakin <bdakin>|
|Component:||Layout and Rendering||Assignee:||Adele Peterson <adele>|
|Severity:||Normal||Keywords:||HasReduction, InRadar, Regression|
|Version:||523.x (Safari 3)|
|OS:||OS X 10.4|
Description Beth Dakin 2007-03-15 13:19:23 PDT
1. Go to http://www01.order.dominos.com/olo/faces/order/step2_choose_pizza.jsp 2. Enter an address. (This part is tricky, some addresses don't provide the pizza store that repros the bug. One that does is 218 Broadway, Revere, MA 02151) 2. Confirm the address and nearest store by clicking "Next" 3. Click "Choose size and crust" under "Build your own pizza" 4. At any given time, one of the radio buttons in the "Hand-tossed," "Deep-dish," "Thin," or "Brooklyn" column should be selected, but as you select them, the button never turns blue. The selection registers in that the view changes, but the button never looks pressed, and it isn't actually pressed. This bug prevents ordering a pizza since this generates an error when you try to add the pizza, saying that you have not selected a size or crust.
Comment 2 Beth Dakin 2007-03-15 13:21:36 PDT
Does this automatically cc me? I can never remember...
Comment 3 David Kilzer (:ddkilzer) 2007-03-15 19:13:14 PDT
(In reply to comment #2) > Does this automatically cc me? I can never remember... Yes, it should, but what email you get depends on your settings. (For example, if you don't care when people just add a Cc: address, you won't get email.)
Comment 4 Beth Dakin 2007-03-15 21:56:54 PDT
Created attachment 13654 [details] reduction: self-closing input tag Here is a reduction. Looks like we used to allow self-closing input tags but don't anymore. Firefox allows them too.
Comment 5 Beth Dakin 2007-03-15 21:57:31 PDT
Comment 6 Beth Dakin 2007-03-15 22:06:24 PDT
I take that back. repros without self-closing as well.
Comment 7 Beth Dakin 2007-03-15 22:09:50 PDT
It's the name attribute. Adding a name attribute makes it work.
Comment 8 Beth Dakin 2007-03-16 11:08:44 PDT
Comment 10 Adele Peterson 2007-03-19 17:40:12 PDT
Comment 11 Beth Dakin 2007-03-19 17:43:23 PDT
Comment on attachment 13711 [details] patch yay! r=me!
Comment 12 Adele Peterson 2007-03-19 17:46:15 PDT
Committed revision 20333.