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(forms): Add function option for disabledReason #71144

Merged
merged 2 commits into from
May 28, 2024

Conversation

roggenkemper
Copy link
Member

this pr adds the option to have disabledReason be a function. this will help align it with disabled which already allows for functions, which means that the tooltip we display can differ, depending on why it is disabled. Right now, you can check a variety of conditions to see if it should be disabled or not, but can only show one reason.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 17, 2024
@roggenkemper roggenkemper merged commit 0c65db2 into master May 28, 2024
42 checks passed
@roggenkemper roggenkemper deleted the roggenkemper/disabledreasonfunction branch May 28, 2024 16:47
volokluev pushed a commit that referenced this pull request May 28, 2024
this pr adds the option to have `disabledReason` be a function. this
will help align it with `disabled` which already allows for functions,
which means that the tooltip we display can differ, depending on why it
is disabled. Right now, you can check a variety of conditions to see if
it should be disabled or not, but can only show one reason.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants