-
Notifications
You must be signed in to change notification settings - Fork 2
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
#360250: add pin element #104
base: main
Are you sure you want to change the base?
Conversation
Sophiaw-hub
commented
Apr 15, 2024
•
edited
Loading
edited
- Das Pin-Element befindet sich bereits in einem Formular und muss nicht extra in eins gezogen werden. Denn der Use Case benötigt in der Regel für die Pin-Eingabe nur das zugehörige Element und ggf. noch ein Textfeld.
- Bei der Eingabe werden nur Zahlen zugelassen.
- Es wird validiert, ob das Element befüllt ist.
- Über Style Configs kann eingestellt werden, ob das Formular automatisch abgeschickt werden soll.
- Über Style Configs kann eingestellt werden, ob die Pin-Elemente mit einer Nummer gelabelt werden sollen.
b2eba80
to
7c350a2
Compare
Bumps [intl-tel-input](https://github.com/jackocnr/intl-tel-input) from 19.5.7 to 21.2.7. - [Release notes](https://github.com/jackocnr/intl-tel-input/releases) - [Changelog](https://github.com/jackocnr/intl-tel-input/blob/master/CHANGELOG.md) - [Commits](jackocnr/intl-tel-input@v19.5.7...v21.2.7) --- updated-dependencies: - dependency-name: intl-tel-input dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sieht insgesamt schon sehr gut aus, aber ich hab immer was zu meckern
inputPin.setAttribute('inputmode', 'numeric'); | ||
inputPin.setAttribute('pattern', "[0-9]"); | ||
|
||
inputPin.addEventListener('focusin', (e) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Event Listener kannst du auch mit setAttribute('@focusin', 'myAlpineFunction')
registrieren
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich bekomme das auf diesen Weg leider nicht hin. Generell, sobald ich versuche, die Methode auszulagern. Wird das Event nicht mehr erkannt.
}, | ||
|
||
_cleanUp() { | ||
// TODO: auch im DOM lösbar? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sehr gute Idee!
Wenn du nur Nummern willst: <input type="number">
Nur eine Zahl: <input min="0" max="9">
Min eine Zahl: <input required>
ich denke, das sollte das hier abdecken
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Das Einsetzten von größeren Zahlen als 9 konnte ich nicht über min/max verhindern. Aber das Einsetzen von Buchstaben. Dadurch ist _cleanUp() auf jeden Fall kürzer geworden.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Das muss ich mir nochmal in Ruhe anschauen 😅
1dc8df5
to
97ee5e9
Compare
@Sophiaw-hub magst du noch das Changelog updaten? :) |