Skip to content

7796 fix - kol-input-range Nummernfeld mit Mausklick nicht fokussierbar#7828

Merged
deleonio merged 2 commits intopublic-ui:7796-fix-kol-input-range-nummernfeld-nicht-mit-mausklick-fokussierbarfrom
aTTiCuZ-CodingPage:7796-fix-kol-input-range-nummernfeld-nicht-mit-mausklick-fokussierbar
Jul 21, 2025
Merged

7796 fix - kol-input-range Nummernfeld mit Mausklick nicht fokussierbar#7828
deleonio merged 2 commits intopublic-ui:7796-fix-kol-input-range-nummernfeld-nicht-mit-mausklick-fokussierbarfrom
aTTiCuZ-CodingPage:7796-fix-kol-input-range-nummernfeld-nicht-mit-mausklick-fokussierbar

Conversation

@aTTiCuZ-CodingPage
Copy link
Copy Markdown
Contributor

Refs: #7796

@deleonio deleonio requested a review from sdvg June 24, 2025 10:45
Copy link
Copy Markdown
Contributor

@deleonio deleonio left a comment

Choose a reason for hiding this comment

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

Hi @sdvg, kannst Du bitte diesen PR verifizieren - ob das passt.

Das mit ID könnte wirklich helfen - sehr interessant. Meine Erwartung wäre, dass wenn keine ID gesetzt wird, dass die immer undefined ist.

Warum der onClick raus ist, ist mir nicht klar.

@deleonio deleonio changed the base branch from develop to 7796-fix-kol-input-range-nummernfeld-nicht-mit-mausklick-fokussierbar June 24, 2025 10:48
@sdvg
Copy link
Copy Markdown
Contributor

sdvg commented Jun 24, 2025

Das mit ID könnte wirklich helfen - sehr interessant. Meine Erwartung wäre, dass wenn keine ID gesetzt wird, dass die immer undefined ist.

Wenn die ID nicht explizit auf undefined gesetzt wird, wird die Standard-ID gesetzt, die mit dem <label for="id"> verknüpft ist.

Tatsächlich würde ich die ID eher behalten: Wenn sie nicht gesetzt ist, wird bei einem Klick auf das Label das input range fokussiert, nicht wie normalerweise das input number.

@aTTiCuZ-CodingPage Übersehe ich hier etwas? Was war der Grund, die ID auf undefined zu setzen?

Warum der onClick raus ist, ist mir nicht klar.

Mir ist nicht klar, warum das onClick jemals gesetzt war :)
Ich denke das Entfernen das onClick war der eigentliche Bugfix hier. Das Fokussieren findet bereits automatisch durch die delegatesFocus-Property bzw. das verlinkte Label statt.

Comment thread packages/components/src/components/input-range/shadow.tsx Outdated
Copy link
Copy Markdown
Contributor

@sdvg sdvg left a comment

Choose a reason for hiding this comment

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

@deleonio deleonio merged commit 938d353 into public-ui:7796-fix-kol-input-range-nummernfeld-nicht-mit-mausklick-fokussierbar Jul 21, 2025
9 checks passed
@publicuibot publicuibot bot locked and limited conversation to collaborators Jul 21, 2025
@public-ui public-ui unlocked this conversation Aug 21, 2025
@public-ui public-ui locked and limited conversation to collaborators Aug 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

kol-input-range: technische Fokus für Nummerneingabefeld nicht fokussierbar

3 participants