REGRESSION: Allow setting the checked attribute in js and in markup for unnamed radio buttons (dominoes)

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Beth Dakin]

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.

[Beth Dakin]

<rdar://problem/5064316>

[Beth Dakin]

Does this automatically cc me? I can never remember...

[David Kilzer (:ddkilzer)]

(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.)

[Beth Dakin]

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.

[Beth Dakin]

Thanks, Dave!

[Beth Dakin]

I take that back. repros without self-closing as well.

[Beth Dakin]

It's the name attribute. Adding a name attribute makes it work.

[Beth Dakin]

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.

[Adele Peterson]

Created attachment 13711 [details]
patch

[Adele Peterson]

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.

[Beth Dakin]

Comment on attachment 13711 [details]
patch

yay! r=me!

[Adele Peterson]

Committed revision 20333.