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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Input, InputNumber, TextArea disabled attribute fix #1048

Merged

Conversation

anddrzejb
Copy link
Member

@anddrzejb anddrzejb commented Jan 26, 2021

馃 This is a ...

  • New feature
  • Bug fix
  • Site / documentation update
  • Demo update
  • Component style update
  • Bundle size optimization
  • Performance optimization
  • Refactoring
  • Code style optimization
  • Test Case
  • Branch merge
  • Other (about what?)

馃敆 Related issue link

#1047

馃挕 Background and solution

Missing disabled attribute on input element.
There is a certain assumption here (visible in the changes in Input.cs): if disabled attribute is set to true in any fashion, it will always prevail.

BTW: @ElderJames I noticed that for example Cascader component does not have Disabled property. Is this by design?

鈽戯笍 Self Check before Merge

鈿狅笍 Please check all items below before review. 鈿狅笍

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • Changelog is provided or not needed

@ElderJames ElderJames linked an issue Jan 27, 2021 that may be closed by this pull request
@ElderJames ElderJames merged commit 93064c0 into ant-design-blazor:master Jan 27, 2021
@ElderJames
Copy link
Member

BTW: @ElderJames I noticed that for example Cascader component does not have Disabled property. Is this by design?

@anddrzejb I think it is we missing them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Input Disabled Attribute
2 participants