Bug 241847

Summary: [css-conditional] @supports selector fails for ::placeholder & ::file-selector-button
Product: WebKit Reporter: Jan Nicklas <j.nicklas>
Component: CSSAssignee: Joonghun Park <jh718.park>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, jensimmons, jh718.park, koivisto, ntim, rniwa, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: All   
OS: All   
Attachments:
Description Flags
Reproduction Example none

Description Jan Nicklas 2022-06-22 02:06:49 PDT
Created attachment 460411 [details]
Reproduction Example

The `@supports selector ()` fails for shadow DOM pseudo element tests.

An example for supported shadow DOM pseudo element styling is `input::placeholder { color: orchid }`.

Therefore the expected behaviour would be that `@supports selector(input::placeholder) {` matches.
Furthermore the expected behaviour would also be that `@supports not selector(input::placeholder) {` does NOT match.

Instead the current behaviour is that `@supports selector(input::placeholder) {` does NOT match.
In addition the current behaviour is also that `@supports not selector(input::placeholder) {` does match.

This behaviour can be reproduced on different Safari versions:

- iOS 15.5 Safari
- MacOS Safari Version 15.5 (17613.2.7.1.8) 
- MacOS Safari Technology Preview Release 146 (Safari 15.4, WebKit 17614.1.14.10.6)

Current Firefox and Chrome versions behave correctly.

Please see the attached reproduction example.
Comment 1 Radar WebKit Bug Importer 2022-06-22 05:33:44 PDT
<rdar://problem/95683424>
Comment 2 Ryosuke Niwa 2022-06-22 22:16:50 PDT
Oh, this is nothing to do with Shadow DOM.
Comment 3 Tim Nguyen (:ntim) 2022-10-12 17:22:56 PDT
WPT PR is at: https://github.com/web-platform-tests/wpt/pull/36410
Comment 4 Tim Nguyen (:ntim) 2023-03-29 19:31:46 PDT
*** Bug 254682 has been marked as a duplicate of this bug. ***
Comment 5 Tim Nguyen (:ntim) 2023-04-24 12:02:09 PDT
Joonghun, any update on this?
Comment 6 Joonghun Park 2023-05-15 09:18:26 PDT
(In reply to Tim Nguyen (:ntim) from comment #5)
> Joonghun, any update on this?

I'm sorry for the delay. I made a patch and uploaded it to https://github.com/WebKit/WebKit/pull/13879. Please refer to it.
Comment 7 EWS 2023-05-15 16:49:03 PDT
Committed 264090@main (61a3212d56ea): <https://commits.webkit.org/264090@main>

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