[Webkit-unassigned] [Bug 53296] Implicit form submission shouldn’t work when the default button is disabled

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=53296





--- Comment #10 from Mathias Bynens <mathias at qiwi.be>  2011-01-31 10:12:39 PST ---
(From update of attachment 80632)
View in context: https://bugs.webkit.org/attachment.cgi?id=80632&action=review

Thanks for your feedback! I’ll tweak the code as per your suggestions and submit another patch later tonight.

>>>> LayoutTests/fast/forms/implicit-submission.html:-19
>>>> -    [ "Text input and text area and a disabled submit, text input focused", "!text,textarea,-submit", "y" ], // match IE, not FF.
>>> 
>>> Aha. This is interesting. So this change may break sites, expecting IE behavior?
>> 
>> The spec states that if the only submit button in a form is disabled, it shouldn’t submit implicitly. If there are no submit buttons at all, implicit submission should still work. This patch would make it so.
>> 
>> So yes, it would break sites that rely on IE-style behavior for forms that only have disabled submit buttons. (Not that it matters, but that sure sounds like an edge case to me.) For this particular case, it would instead match Firefox and Opera’s behavior, which happens to conform to the HTML spec and also makes much more sense.
>> 
>> Would this change be acceptable or not?
> 
> I think the change is acceptable, but we should watch for the reactions from the Web and be ready to revert this if it proves drastic.
> 
> Also, would love to see a test added with multiple submit buttons (some disabled), just to test your newly added code.

I believe there’s already a test for this: “Multiple text inputs and multiple submits, first submit disabled”. Let me know if I’m missing something.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.