Bug 13087

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 RenderingAssignee: Adele Peterson <adele>
Severity: Normal Keywords: HasReduction, InRadar, Regression
Priority: P1    
Version: 523.x (Safari 3)   
Hardware: Mac   
OS: OS X 10.4   
URL: http://www.dominos.com
Description Flags
reduction: self-closing input tag
patch bdakin: review+

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 1 Beth Dakin 2007-03-15 13:19:46 PDT
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
Thanks, Dave!
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 on attachment 13654 [details]
reduction: self-closing input tag

Okay, it turns out this reduction is not valid. Our behavior here matches IE, and we made the change intentionally. There is some other JavaScript at the site that makes the radio button still click in IE. I am working on another reduction.
Comment 9 Adele Peterson 2007-03-19 17:30:49 PDT
Created attachment 13711 [details]
Comment 10 Adele Peterson 2007-03-19 17:40:12 PDT
We were matching a WinIE quirk that does not allow a user to check and uncheck an unnamed radio button.  But they still allow the checked attribute to be set in html, and changed in javascript.  So this change matches that behavior.
Comment 11 Beth Dakin 2007-03-19 17:43:23 PDT
Comment on attachment 13711 [details]

yay! r=me!
Comment 12 Adele Peterson 2007-03-19 17:46:15 PDT
Committed revision 20333.