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

Clarify appearance: none; behavior #36893

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

jeff-hykin
Copy link
Contributor

@jeff-hykin jeff-hykin commented Nov 21, 2024

Description

Clarify the standardized behavior of appearance: none;

Motivation

Some people misinterpret the original wording as behavior similar to display: none; or visibility: hidden; when in reality it is more like a css-reset.

Here is some supporting evidence of old defintion causing confusion

Screenshot_2024-11-20-22-25-29-40

Additional details

Quote from the spec

The element is rendered following the usual rules of CSS. Replaced elements other than widgets are not affected by this and remain replaced elements. Widgets must not have their native appearance, and instead must have their primitive appearance. See § 7.2.2 Effects of appearance on Decorative Aspects of Elements and § 7.2.3 Effects of appearance on Semantic Aspects of Elements for details.

CSS spec:

> The element is rendered following the usual rules of CSS. Replaced elements other than widgets are not affected by this and remain replaced elements. Widgets must not have their native appearance, and instead must have their primitive appearance. See § 7.2.2 Effects of appearance on Decorative Aspects of Elements and § 7.2.3 Effects of appearance on Semantic Aspects of Elements for details.
@jeff-hykin jeff-hykin requested a review from a team as a code owner November 21, 2024 03:48
@jeff-hykin jeff-hykin requested review from estelle and removed request for a team November 21, 2024 03:48
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs size/xs [PR only] 0-5 LoC changed labels Nov 21, 2024
Copy link
Contributor

github-actions bot commented Nov 21, 2024

Preview URLs

(comment last updated: 2024-11-22 14:53:46)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Member

@estelle estelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you. I made some suggestions.

@@ -44,7 +44,7 @@ Some examples are provided, but the list is not exhaustive.

- `none`

- : Hides certain features of widgets, such as arrow displayed in select element, indicating that list can be expanded.
- : If the element is a widget (native form control) it will be forced to use a standardized primitive appearance instead of a platform native appearance (ex: operating system specific). Non-widget elements, including replaced elements like img and video, are unaffected.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- : If the element is a widget (native form control) it will be forced to use a standardized primitive appearance instead of a platform native appearance (ex: operating system specific). Non-widget elements, including replaced elements like img and video, are unaffected.
- : If the element is a widget (native form control), it will be forced to use a standardized primitive appearance instead of a platform-native or operating system specific appearance, supporting the usual rules of CSS. This value has no effect on non-widget elements, including replaced elements like {{htmlelement("img")}} and {{htmlelement("video")}}.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:CSS Cascading Style Sheets docs size/xs [PR only] 0-5 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants