Skip to content
This repository has been archived by the owner on Jun 4, 2024. It is now read-only.

Boolean attributes in button: disabled #32

Open
Akronix opened this issue Dec 21, 2017 · 3 comments
Open

Boolean attributes in button: disabled #32

Akronix opened this issue Dec 21, 2017 · 3 comments

Comments

@Akronix
Copy link
Contributor

Akronix commented Dec 21, 2017

Hello.

I'm trying to use the disabled attribute in the button element.

Firstly, I've noticed that its type is String instead of Boolean.
Secondly, I've trying to set the prop disabled='disabled', but it does not work; and it is still enabled.

Other props of button with expected similar behaviour (boolean props) are: autoFocus, contentEditable, draggable, hidden, spellCheck, etc.

@Akronix
Copy link
Contributor Author

Akronix commented Jan 29, 2018

I finally got it done using the disabled attribute with a string as it is currently in the upstream repo.

Nevertheless, I was taking a look on how to change the PropTypes of html attributes, but I didn't find the way to do it. I guess that changing it directly in Button.react.js isn't the right way.

@radekwlsk
Copy link

radekwlsk commented Jul 18, 2018

Have you tried setting it to simply True? I am doing that with html.Button for a long time and never had an issue with html.Button("I am a disabled button", disabled=True).

@Akronix
Copy link
Contributor Author

Akronix commented Jul 23, 2018

Hmm I just checked and it does work! But the prop for disabled in the component is still Boolean 🤔

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

No branches or pull requests

3 participants