Summary: | [iOS] iPhone should not allow selecting <optgroup> in <select multiple> | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joseph Pecoraro <joepeck> | ||||
Component: | WebKit2 | Assignee: | Joseph Pecoraro <joepeck> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | dbates, enrica, joepeck, simon.fraser | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Joseph Pecoraro
2014-10-22 17:33:13 PDT
Created attachment 240316 [details]
[PATCH] Proposed Fix
Patch may not apply as it may be waiting on another change near this.
Comment on attachment 240316 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=240316&action=review > Source/WebKit2/UIProcess/ios/forms/WKFormSelectPicker.mm:284 > + // FIXME: <rdar://problem/18745253> > + // Group picker content view should not be checkable. If we get this delegate > + // reset the styles for the group content view so it still appears unselected. I don’t think this comment makes it clear enough that this is a workaround for the above bug and can be removed if it’s fixed. > Source/WebKit2/UIProcess/ios/forms/WKFormSelectPicker.mm:288 > + [[view titleLabel] setTextColor:[UIColor colorWithWhite:0.0 alpha:0.5]]; Hard-coding this seems really unfortunate and could cause us trouble later. What mitigates that, I suppose, is that we will remove this code when the above mentioned bug is fixed. (In reply to comment #3) > Comment on attachment 240316 [details] > [PATCH] Proposed Fix > > View in context: > https://bugs.webkit.org/attachment.cgi?id=240316&action=review > > > Source/WebKit2/UIProcess/ios/forms/WKFormSelectPicker.mm:288 > > + [[view titleLabel] setTextColor:[UIColor colorWithWhite:0.0 alpha:0.5]]; > > Hard-coding this seems really unfortunate and could cause us trouble later. > What mitigates that, I suppose, is that we will remove this code when the > above mentioned bug is fixed. Seeing as we already had a constant for disabled options, I'll add one and share it for groups. This is only duplicating our own styles, not another framework's styles. |