Intent to Ship: checkVisibility: contentVisibilityAuto, opacityProperty, and visibilityProperty

129 views
Skip to first unread message

Vladimir Levin

unread,
Nov 6, 2023, 3:21:34 PM11/6/23
to blink-dev

Contact emails

vmp...@chromium.org

Explainer

None

Specification

https://drafts.csswg.org/cssom-view/#dom-element-checkvisibility

Summary

This feature adds three more values to the checkVisibility (https://drafts.csswg.org/cssom-view/#dom-element-checkvisibility) dictionary members: * opacityProperty as an alias for checkOpacity * visibilityProperty as an alias for checkVisibilityCSS * contentVisibilityAuto as a property that allows checking whether an element is in a subtree skipped by content-visibility: auto.


See https://github.com/w3c/csswg-drafts/pull/9549



Blink component

Blink>DOM

TAG review

None

TAG review status

Not applicable, it's a small addition to an existing feature

Risks



Interoperability and Compatibility

None



Gecko: No signal (https://github.com/mozilla/standards-positions/issues/920)

WebKit: Positive (https://github.com/w3c/csswg-drafts/issues/9487#issuecomment-1782109845) The naming proposal came from Safari

Web developers: No signals

Other signals:

Ergonomics

There are no ergonomic risks, since this adds extra properties to an existing feature.



Activation

There are no activation risks, since this adds extra (useful) properties to an existing feature



Security

There are no extra security issues here that were not already considered during the checkVisibility review.



WebView application risks

Does this intent deprecate or change behavior of existing APIs, such that it has potentially high risk for Android WebView-based applications?

None



Debuggability

This would be debugged as any other script APIs



Will this feature be supported on all six Blink platforms (Windows, Mac, Linux, Chrome OS, Android, and Android WebView)?

Yes

Is this feature fully tested by web-platform-tests?

Yes

Will write tests as a part of the implementation



Flag name on chrome://flags



Finch feature name

CheckVisibilityExtraProperties

Requires code in //chrome?

False

Estimated milestones

Shipping on desktop121
DevTrial on desktop121
Shipping on Android121
DevTrial on Android121
Shipping on WebView121


Anticipated spec changes

Open questions about a feature may be a source of future web compat or interop issues. Please list open issues (e.g. links to known github issues in the project for the feature specification) whose resolution may introduce web compat/interop risk (e.g., changing to naming or structure of the API in a non-backward-compatible way).

https://github.com/w3c/csswg-drafts/pull/9549

Link to entry on the Chrome Platform Status

https://chromestatus.com/feature/5070043440480256

Links to previous Intent discussions

Intent to prototype: https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CADsXd2NjBPfNAftFFjSKxyX0PXtWpuK%2BQ1QrUaAQrjV9EfouOQ%40mail.gmail.com

This intent message was generated by Chrome Platform Status.

Yoav Weiss

unread,
Nov 8, 2023, 11:58:38 AM11/8/23
to blink-dev, Vladimir Levin
LGTM1

Mike Taylor

unread,
Nov 8, 2023, 12:01:29 PM11/8/23
to Yoav Weiss, blink-dev, Vladimir Levin

LGTM2

--
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+...@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/21f32a6c-ea3c-4923-925a-3067831ca645n%40chromium.org.

Chris Harrelson

unread,
Nov 8, 2023, 12:03:37 PM11/8/23
to Mike Taylor, Yoav Weiss, blink-dev, Vladimir Levin
Reply all
Reply to author
Forward
0 new messages