8000 Radio button ignores disabled state · Issue #407 · gwtbootstrap3/gwtbootstrap3 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Nov 15, 2019. It is now read-only.

Radio button ignores disabled state #407

Open
ahumellihuk opened this issue Sep 28, 2015 · 8 comments
Open

Radio button ignores disabled state #407

ahumellihuk opened this issue Sep 28, 2015 · 8 comments
Labels
bootstrap bug
Milestone
1.0

Comments

@ahumellihuk
Copy link

Value change events are triggered even when the radio button is disabled.
@sjardine sjardine added this to the 1.0 milestone Oct 21, 2015
@slugmandrew
Copy link
Contributor

I can work out how to stop the ValueChangeEvent firing but I can't stop the active style being set for some reason 😕

@sjardine
Copy link
Member

OK. Please describe exactly the problem. Could you provide an example for me to work with? The easiest thing to do is fork the seed project and create the demo there.

Thanks!

@sjardine
Copy link
Member

Can you please provide additional detailed information?

@slugmandrew
Copy link
Contributor

It wasn't me that opened the issue, but I did have a go at fixing it. As I said I managed to fix it so that when pressed the radio button did not fire the ValueChangeEvent, but it still looked like it had fired the event, which would be more confusing for the user.

I'll try to fork the seed project sometime in the next couple of weeks and set up an example. Unfortunately I'm very busy atm.

@sjardine
Copy link
Member

I can create the example if you could give help explain the setup. I haven't been able to duplicate.

@vlahoda
Copy link
vlahoda commented May 31, 2016

This error is reproducible even in the demo at http://gwtbootstrap3.github.io/gwtbootstrap3-demo/#buttonGroups . In the Radio buttons example at the very bottom of that page it is possible to click on the disabled Option4, which in turn gets assigned the style "active".

@nemoo2580
Copy link
Contributor

It's also a bootstrap bug:
twbs/bootstrap#16703

@crevete
Copy link
Contributor
crevete commented Aug 2, 2016

You are right, this is a bootstrap bug, but still not fixed in 3.3.7. Hope it will be fixed soon...

@crevete crevete added bootstrap bug and removed bug labels Aug 21, 2016
This was referenced Nov 7, 2016
Merged
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bootstrap bug
Projects
None yet
Milestone
1.0
Development

No branches or pull requests
6 participants
@slugmandrew @sjardine @vlahoda @ahumellihuk @crevete @nemoo2580
0