Bug 9245

Summary: Quirksmode: Incorrect handling of disabled BUTTON elements
Product: WebKit Reporter: Adele Peterson <adele>
Component: FormsAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: ian
Priority: P2    
Version: 420+   
Hardware: PC   
OS: OS X 10.4   
Bug Depends on:    
Bug Blocks: 9610    
Attachments:
Description Flags
Patch adele: review+

Adele Peterson
Reported 2006-06-02 11:58:11 PDT
http://www.quirksmode.org/bugreports/archives/safari/index.html "Adding disabled attributes to <BUTTON> elements in Safari does not have the desired effect. Disabled buttons with text content are not greyed out and still respond visually to clicks (albeit without generating any onclick events). In <BUTTON> elements with image content, the addition of a disabled atribute has no effect whatsoever. Test page http://www.philronan.pwp.blueyonder.co.uk/safari-bugs/bad-buttons.html Workaround is included Reported by: Philip Ronan."
Attachments
Patch (5.38 KB, patch)
2006-07-09 15:09 PDT, Anders Carlsson 		adele: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Joost de Valk (AlthA)
Comment 1 2006-06-24 13:24:21 PDT
This should be P1 imho, as it allows users to do things the webdesigner didn't want them to be able to.
Dave Hyatt
Comment 2 2006-06-24 14:37:40 PDT
P1 = crashes and regressions.
Joost de Valk (AlthA)
Comment 3 2006-06-24 14:52:33 PDT
Hyatt: I strongly disagree and would like to discuss this with you.
Anders Carlsson
Comment 4 2006-07-09 15:09:51 PDT
Created attachment 9312 [details] Patch
Darin Adler
Comment 5 2006-07-09 23:13:08 PDT
Website compatibility problems should almost never be P1 bugs; exceptions would be problems that affect very important and prominent websites or things that cause security problems. And I agree this bug does not deserve a P1. But I guess we won't resolve this by discussing it in a bug report. And this bug has a fix already, so it doesn't matter too much!
Anders Carlsson
Comment 6 2006-07-10 01:56:45 PDT
Fixed in r15296
Note You need to log in before you can comment on or make changes to this bug.