Skip to content

8358 fix: inputs focus verhalten#8540

Closed
BF150 wants to merge 27 commits intorelease/3from
8358-inputs-focus-verhalten
Closed

8358 fix: inputs focus verhalten#8540
BF150 wants to merge 27 commits intorelease/3from
8358-inputs-focus-verhalten

Conversation

@BF150
Copy link
Copy Markdown
Contributor

@BF150 BF150 commented Sep 24, 2025

  • removed delegates focus in inputs
  • added _tabIndex to inputs
  • fixed _tabIndex for select
  • added tests for all components with _tabIndex

changed components:

  • combobox
  • input checkbox
  • input color
  • input date
  • input email
  • input file
  • input number
  • input password
  • input radio
  • input range
  • input text
  • select
  • single select
  • textarea

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 24, 2025

@BF150 BF150 marked this pull request as ready for review September 24, 2025 14:24
@BF150 BF150 added the v3 label Sep 24, 2025
@BF150 BF150 self-assigned this Sep 24, 2025
@BF150 BF150 linked an issue Sep 24, 2025 that may be closed by this pull request
@deleonio deleonio force-pushed the 8358-inputs-focus-verhalten branch from a509d84 to b193998 Compare September 25, 2025 03:11
…ntation

- Deprecated the _tabIndex property in various components (input-password, input-radio, input-range, input-text, link-button, link, popover-button, select, single-select, split-button, textarea) to encourage using the default tabIndex on host elements.
- Updated related documentation to reflect the changes and indicate that the property will be removed in the next major version.
- Adjusted shadow DOM configurations for link-button, link, and split-button components to simplify shadow root settings.
- Updated snapshots in tests to align with the removal of shadowrootdelegatesfocus attribute.
- Modified sample scenarios to remove tabIndex from disabled components and ensure proper usage.

Refs: #8358
@deleonio deleonio force-pushed the 8358-inputs-focus-verhalten branch from b193998 to 2e4245f Compare September 25, 2025 03:16
/**
* Defines which tab-index the primary element of the component has. (https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/tabindex)
*
* @deprecated We prefer to use it on the host element with the default tabIndex. Please make sure not to use tabIndex for disabled elements. The property will be removed in the next major version.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Der normale Tabindex wird nach meinem Verständnis nicht funktionieren wie erwartet, da er die umgebende Komponente verändert nicht die innere (html input, button und so weiter).
Deswegen hab ich den _tabIndex in allen inputs hinzugefügt, damit der nach innen weiter gereicht wird.
Hast du einen Plan wie der normale tabindex nach innen kommen soll, ohne delegatesFocus?

@deleonio deleonio changed the base branch from develop to release/3 October 11, 2025 07:06
@BF150 BF150 changed the base branch from release/3 to develop October 29, 2025 07:49
@BF150 BF150 changed the base branch from develop to release/3 October 29, 2025 07:50
@BF150 BF150 marked this pull request as draft October 29, 2025 12:36
@BF150 BF150 closed this Oct 30, 2025
@publicuibot publicuibot bot locked and limited conversation to collaborators Oct 30, 2025
@deleonio deleonio deleted the 8358-inputs-focus-verhalten branch December 5, 2025 01:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐞 Bug: Inkonsistentes Focusverhalten bei Eingabefeldern

2 participants