Skip to content

Conversation

@autonome
Copy link
Contributor

@autonome autonome commented Oct 2, 2024

Weird one.

The meaning of the visible property changed, but hasn't been implemented everywhere.

The old meaning is still implemented by Apple, so the implementation is not consistent.

Given that incompatibility, leaving this as baseline false provides the best guidance for developers making decisions today.

Cainuse has all green however. If we merge this feature as baseline false, I'll go file an issue there about it.

@github-actions github-actions bot added the feature definition Creating or defining new features or groups of features. label Oct 2, 2024
Copy link
Collaborator

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

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

Huh, this is so weird. Good find. Some suggestions about keys (I think we can sweep in some more keys) and the description. Thank you!

@ddbeck
Copy link
Collaborator

ddbeck commented Oct 2, 2024

Cainuse has all green however. If we merge this feature as baseline false, I'll go file an issue there about it.

The caniuse entries are just generated from BCD, so only https://caniuse.com/mdn-api_barprop_visible_returns_popup shows the Safari difference. This is normal; no action required.

autonome and others added 4 commits October 3, 2024 11:43
Co-authored-by: Daniel D. Beck <daniel@ddbeck.com>
Co-authored-by: Daniel D. Beck <daniel@ddbeck.com>
@autonome autonome requested a review from ddbeck October 3, 2024 04:50
Copy link
Collaborator

@ddbeck ddbeck 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!

@ddbeck ddbeck merged commit f09faf9 into web-platform-dx:main Oct 3, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature definition Creating or defining new features or groups of features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants