telerik / kendo-angular

Issue tracker - Kendo UI for Angular
http://www.telerik.com/kendo-angular-ui/
Other
467 stars 217 forks source link

[Button] Dynamically enabling/disabling the component breaks the keyboard navigation #4249

Closed georgi-sla closed 3 weeks ago

georgi-sla commented 6 months ago

Describe the bug Currently, when the disabled state of the Button component is dynamically enabled/disabled, the keyboard navigation gets broken:

To Reproduce

  1. Create two Button components (using the kendo-button selector) - one enabled and one disabled.
  2. Dynamically update the disabled state of the two Buttons.

Please, check out the following StackBlitz demo: https://stackblitz.com/edit/angular-u9svkg-a9sft9

Expected behavior Dynamically updating the disabled state of the Button component should also update the keyboard navigation:

Additional context From what I have noticed, it seems like the tabindex of the Button component is not dynamically updated.

slavenai commented 5 months ago

Reproducible with kendo-button and span[kendoButton]. Works as expected with button[kendoButton].

dtopalov commented 1 month ago

The kendo-button and span[kendoButton] approaches for creating a Button component are deprecated. Please use button[kendoButton] instead.