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

The disabled buttons are clickables #800

Closed
viktorhajer opened this issue Oct 4, 2023 · 1 comment · Fixed by #804
Closed

The disabled buttons are clickables #800

viktorhajer opened this issue Oct 4, 2023 · 1 comment · Fixed by #804
Assignees

Comments

@viktorhajer
Copy link

What happened?

The disabled buttons should not trigger the click event.

What type of frontend frameware are you seeing the problem on?

Angular

Which version of iX do you use?

v2.0.1

Code to produce this issue.

<!-- event is fired (it should not) -->
<ix-button disabled (click)="doAction()">iX button</ix-button>

<!-- event is not fired (working well): -->
<button disabled (click)="doAction()">Normal button</button>
@viktorhajer viktorhajer added the triage We discuss this topic in our internal weekly label Oct 4, 2023
@danielleroux danielleroux added semvers: patch and removed triage We discuss this topic in our internal weekly labels Oct 4, 2023
@danielleroux danielleroux self-assigned this Oct 4, 2023
@viktorhajer
Copy link
Author

The main issue was fixed, but because of the pointer-events: none; style the button is not just not-clickable but you hide its title as well - which woulb be necessary for us (and also it is unexpected behaviour). Check it please.

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

Successfully merging a pull request may close this issue.

2 participants