Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[css-ui] Hide input[type=search] icons when appearance: textfield is set #895

Merged
merged 0 commits into from Aug 23, 2022

Conversation

nt1m
Copy link
Member

@nt1m nt1m commented May 22, 2022

d08a6f7

[css-ui] Hide input[type=search] icons when appearance: textfield is set
https://bugs.webkit.org/show_bug.cgi?id=238751
<rdar://91253463>

Reviewed by Aditya Keerthi and Antti Koivisto.

This forces the display value to none when appearance: textfield is set to make it look like input[type=text], matching the CSS UI spec.

There are however still two issues in order to make input[type=search] match input[type=text] fully with appearance: textfield:
- iOS has special UA styles in html.css that need to stop applying with appearance: textfield
- The focus outline offset is inflated on macOS (although this is an issue with other contexts too), see webkit.org/b/241102

* LayoutTests/TestExpectations:
* LayoutTests/fast/forms/search/search-padding-cancel-results-buttons.html:
* LayoutTests/platform/ios/TestExpectations:
* LayoutTests/platform/ios/fast/css/input-search-padding-expected.txt:
* LayoutTests/platform/gtk/fast/css/input-search-padding-expected.png:
* LayoutTests/platform/gtk/fast/css/input-search-padding-expected.txt:
* Source/WebCore/html/shadow/TextControlInnerElements.cpp:
(WebCore::SearchFieldResultsButtonElement::resolveCustomStyle):
(WebCore::SearchFieldCancelButtonElement::resolveCustomStyle):

Canonical link: https://commits.webkit.org/253691@main

71290c1

Misc iOS, tvOS & watchOS macOS Linux Windows
❌ πŸ§ͺ style ❌ πŸ›  ios   πŸ›  mac ❌ πŸ›  wpe ❌ πŸ›  πŸ§ͺ win
❌ πŸ›  ios-sim   πŸ›  mac-debug ❌ πŸ›  gtk ❌ πŸ›  wincairo
❌ πŸ§ͺ webkitperl ❌ πŸ§ͺ ios-wk2   πŸ›  mac-AS-debug ❌ πŸ§ͺ gtk-wk2
❌ πŸ§ͺ api-ios   πŸ§ͺ api-mac ❌ πŸ§ͺ api-gtk
❌ πŸ›  πŸ§ͺ jsc ❌ πŸ›  tv   πŸ§ͺ mac-wk1 ❌ πŸ›  jsc-armv7
❌ πŸ›  tv-sim   πŸ§ͺ mac-wk2 ❌ πŸ§ͺ jsc-armv7-tests
βœ… πŸ›  πŸ§ͺ merge ❌ πŸ›  watch   πŸ§ͺ mac-AS-debug-wk2 ❌ πŸ›  jsc-mips
❌ πŸ›  watch-sim   πŸ§ͺ mac-wk2-stress ❌ πŸ§ͺ jsc-mips-tests

@nt1m nt1m self-assigned this May 22, 2022
@nt1m nt1m added Forms For bugs specific to form elements (checkboxes, buttons, text fields, etc.) WebKit Nightly Build labels May 22, 2022
@nt1m nt1m requested review from pxlcoder and anttijk May 22, 2022 18:35
@webkit-early-warning-system webkit-early-warning-system added the merging-blocked Applied to prevent a change from being merged label May 22, 2022
@nt1m nt1m removed merging-blocked Applied to prevent a change from being merged Forms For bugs specific to form elements (checkboxes, buttons, text fields, etc.) WebKit Nightly Build labels May 22, 2022
@nt1m nt1m added Forms For bugs specific to form elements (checkboxes, buttons, text fields, etc.) WebKit Nightly Build labels May 22, 2022
@webkit-early-warning-system webkit-early-warning-system added the merging-blocked Applied to prevent a change from being merged label May 22, 2022
@nt1m nt1m removed the merging-blocked Applied to prevent a change from being merged label May 23, 2022
@nt1m nt1m force-pushed the search-cancel-button branch 2 times, most recently from 7d8f575 to 14a32aa Compare May 23, 2022 06:30
@webkit-early-warning-system webkit-early-warning-system added the merging-blocked Applied to prevent a change from being merged label May 23, 2022
@nt1m nt1m removed merging-blocked Applied to prevent a change from being merged Forms For bugs specific to form elements (checkboxes, buttons, text fields, etc.) WebKit Nightly Build labels May 23, 2022
@nt1m nt1m added Forms For bugs specific to form elements (checkboxes, buttons, text fields, etc.) WebKit Nightly Build labels May 23, 2022
@webkit-early-warning-system webkit-early-warning-system added the merging-blocked Applied to prevent a change from being merged label May 23, 2022
Source/WebCore/css/html.css Outdated Show resolved Hide resolved
@nt1m nt1m removed merging-blocked Applied to prevent a change from being merged Forms For bugs specific to form elements (checkboxes, buttons, text fields, etc.) WebKit Nightly Build labels May 23, 2022
@nt1m nt1m added Forms For bugs specific to form elements (checkboxes, buttons, text fields, etc.) WebKit Nightly Build labels May 24, 2022
@webkit-early-warning-system webkit-early-warning-system added the merging-blocked Applied to prevent a change from being merged label May 24, 2022
@nt1m nt1m removed the merging-blocked Applied to prevent a change from being merged label Aug 19, 2022
@nt1m nt1m changed the title [css-ui] Make input[type=search] with appearance: textfield look like input[type=text] [css-ui] Hide input[type=search] icons when appearance: textfield is set Aug 19, 2022
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Aug 19, 2022
@nt1m nt1m removed the merging-blocked Applied to prevent a change from being merged label Aug 19, 2022
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Aug 19, 2022
@nt1m nt1m removed the merging-blocked Applied to prevent a change from being merged label Aug 20, 2022
LayoutTests/platform/ios/TestExpectations Outdated Show resolved Hide resolved
Source/WebCore/html/shadow/TextControlInnerElements.cpp Outdated Show resolved Hide resolved
@nt1m nt1m added the merge-queue Applied to send a pull request to merge-queue label Aug 23, 2022
@webkit-commit-queue
Copy link
Collaborator

Committed 253691@main (d08a6f7): https://commits.webkit.org/253691@main

Reviewed commits have been landed. Closing PR #895 and removing active labels.

@webkit-early-warning-system webkit-early-warning-system merged commit d08a6f7 into WebKit:main Aug 23, 2022
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Aug 23, 2022
@nt1m nt1m deleted the search-cancel-button branch August 23, 2022 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Forms For bugs specific to form elements (checkboxes, buttons, text fields, etc.)
Projects
None yet
6 participants