Summary: | [iOS][FCR] Add new look for checkboxes | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Aditya Keerthi <akeerthi> | ||||||
Component: | Forms | Assignee: | Aditya Keerthi <akeerthi> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | benjamin, cdumez, changseok, cmarcelo, esprehn+autocc, ews-watchlist, glenn, gyuyoung.kim, kondapallykalyan, macpherson, menard, pdr, sam, thorton, webkit-bug-importer, wenson_hsieh | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | Other | ||||||||
Hardware: | iPhone / iPad | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 218864 | ||||||||
Attachments: |
|
Description
Aditya Keerthi
2020-11-11 09:30:39 PST
Created attachment 413831 [details]
Patch
Comment on attachment 413831 [details]
Patch
The change itself seems fine, but I think it would be nice to add a runtime-enabled setting for this, so that it can be tested in some capacity, in the future.
Comment on attachment 413831 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=413831&action=review > Source/WebCore/css/html.css:775 > +input[type="radio"] { > + margin: 3px 2px; There are other separated out input[type="radio"] and input[type="checkbox"] lower down in the file. Now that these aren't for both anymore, can we just merge these with the ones lower down? Created attachment 414254 [details]
Patch
(In reply to Sam Weinig from comment #4) > Comment on attachment 413831 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=413831&action=review > > > Source/WebCore/css/html.css:775 > > +input[type="radio"] { > > + margin: 3px 2px; > > There are other separated out input[type="radio"] and input[type="checkbox"] > lower down in the file. Now that these aren't for both anymore, can we just > merge these with the ones lower down? These changes are now made in a separate stylesheet, so that this feature can be turned on/off at runtime. Committed r269913: <https://trac.webkit.org/changeset/269913> All reviewed patches have been landed. Closing bug and clearing flags on attachment 414254 [details]. |