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

feat(Field.Option): support disabled property #3029

Closed
langz opened this issue Dec 8, 2023 · 3 comments
Closed

feat(Field.Option): support disabled property #3029

langz opened this issue Dec 8, 2023 · 3 comments
Labels
enhancement New feature or request
Milestone

Comments

@langz
Copy link
Contributor

langz commented Dec 8, 2023

🚀 Feature Proposal

In the light of the following two CSBs:

It's observed that we don't support the disabled property for the Field.Option component.
In the following CSB/examples, would it make sense to be able to disable one or more of the options, without disabling all?
Disabling all options could/should be done through the Field.Selection's disabled property.

Here's a screenshot of the CSB, where I would guess that it could make sense to disable one or multiple options:
image

@joakbjerk
Copy link
Contributor

joakbjerk commented Dec 28, 2023

partly solved by this pr. Field.Option inside a dropdown are still not possible to disable

@joakbjerk joakbjerk removed their assignment Jan 2, 2024
@snorrekim
Copy link
Contributor

As far as I can tell, Dropdown does not support disabling individual options, neither does DrawerList. So it wouldn't be expected in Forms either. So should we close this issue? @tujoworker

Or do we want to implement disabled for DrawerList and Dropdown?

@tujoworker
Copy link
Member

Yes, seems to be fine now 💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

4 participants