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.

Sign up for GitHub

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

Jump to bottom

css.properties.display - none not working on Safari for iOS #24247

Closed
RvmMim opened this issue Aug 29, 2024 · 3 comments · Fixed by #24392
Closed

css.properties.display - none not working on Safari for iOS #24247

RvmMim opened this issue Aug 29, 2024 · 3 comments · Fixed by #24392
Labels
data:css 🎨 Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS

Comments

@RvmMim
Copy link

RvmMim commented Aug 29, 2024 •
edited
Loading

What type of issue is this?

Missing compatibility data

What information was incorrect, unhelpful, or incomplete?

The CSS display:none is said fully supported on Safari for iOS while it doesn't work on <option> HTML element

What browsers does this problem apply to, if applicable?

Safari

What did you expect to see?

With <option style="display:none">, I was expecting to not see the option.
But the option is displayed on Safari for iOS.

Did you test this? If so, how?

I displayed an html page with a <select> with <option>s, one having the style="display:none" attribute.

Can you link to any release notes, bugs, pull requests, or MDN pages related to this?

https://developer.mozilla.org/fr/docs/Web/CSS/display#compatibilit%C3%A9_des_navigateurs
https://stackoverflow.com/questions/4398966/how-can-i-hide-select-options-with-javascript-cross-browser

Do you have anything more you want to share?

No response

MDN URL

https://developer.mozilla.org/fr/docs/Web/CSS/display

MDN metadata

MDN page report details
  • Query: css.properties.display
  • Report started: 2024-08-29T15:05:36.209Z
@github-actions GitHub Actions
Copy link

This issue was automatically closed because the title was left as the default, and a summary was not added.

If this is not a spam issue, please replace the <SUMMARIZE THE PROBLEM> part of the title with a short summary of the reported issue, and then post a follow-up comment. A maintainer will review your issue and reopen it if needed.

@github-actions github-actions bot added the invalid 🚫 Invalid issues or pull requests (wrong repo, spam, duplicates, etc.). This won't get merged. Sorry! label Aug 29, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 29, 2024
@RvmMim RvmMim changed the title css.properties.display - <SUMMARIZE THE PROBLEM> css.properties.display - none not working on Safari for iOS Aug 29, 2024
@RvmMim
Copy link
Author

RvmMim commented Aug 29, 2024

This issue was automatically closed because the title was left as the default, and a summary was not added.

If this is not a spam issue, please replace the <SUMMARIZE THE PROBLEM> part of the title with a short summary of the reported issue, and then post a follow-up comment. A maintainer will review your issue and reopen it if needed.

title updated

@queengooborg queengooborg reopened this Aug 29, 2024
@queengooborg queengooborg added data:css 🎨 Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS and removed invalid 🚫 Invalid issues or pull requests (wrong repo, spam, duplicates, etc.). This won't get merged. Sorry! labels Aug 29, 2024
queengooborg added a commit to queengooborg/browser-compat-data that referenced this issue Sep 9, 2024
@queengooborg
Add missing css.properties.display.none.option_is_hidden feature
5b928f6 
This PR adds the missing `none.option_is_hidden` member of the `display` CSS property. The data comes from manual testing, running test code through BrowserStack, SauceLabs, custom VMs and/or locally.

Test Code:

```html
<style id="test-style">
	#hidden {
		display: none;
	}
</style>
<select>
	<option>One</option>
	<option id="hidden">Two</option>
	<option>Three</option>
</select>
```

Additional Notes: This fixes mdn#24247.
@queengooborg queengooborg mentioned this issue Sep 9, 2024
Merged
@caugner
Copy link
Contributor

caugner commented Oct 11, 2024

Reproduced locally with Safari 18.0 and this example.

queengooborg added a commit that referenced this issue Oct 14, 2024
@queengooborg @caugner
Add missing css.properties.display.none.option_is_hidden feature (#24392
6a52973 
)

* Add missing css.properties.display.none.option_is_hidden feature

This PR adds the missing `none.option_is_hidden` member of the `display` CSS property. The data comes from manual testing, running test code through BrowserStack, SauceLabs, custom VMs and/or locally.

Test Code:

```html
<style id="test-style">
	#hidden {
		display: none;
	}
</style>
<select>
	<option>One</option>
	<option id="hidden">Two</option>
	<option>Three</option>
</select>
```

Additional Notes: This fixes #24247.

* Update css/properties/display.json

Co-authored-by: Claas Augner <495429+caugner@users.noreply.github.com>

---------

Co-authored-by: Claas Augner <495429+caugner@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
data:css 🎨 Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add missing css.properties.display.none.option_is_hidden feature queengooborg/browser-compat-data
5 participants
@caugner @queengooborg @RvmMim and others