Change Proposal: Increment/Decrement Spinner Buttons for `NumberInput`
Les mainteneurs répondent en général sous 3 jours
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- typescript
- Domaine
- frontend
Piste de recherche
Commencez par examiner le composant NumberInput existant et son comportement actuel pour numberMode, minValue et maxValue, puis inspectez la pull request ouverte #271 afin de comprendre le travail déjà en cours. Le travail est considéré comme terminé lorsque les boutons de spinner optionnels prennent en charge les paramètres proposés de pas et de visibilité, bornent les valeurs aux limites et laissent les utilisations existantes inchangées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
What is being proposed?
Add optional up/down spinner buttons to NumberInput so users can nudge a numeric value by a configurable step, instead of only being able to type a value directly.
Why is this needed?
NumberInput currently only accepts typed input. For small, discrete adjustments (e.g. incrementing an integer count), typing is slower and more error-prone than a click-to-step control, which is a common affordance for numeric fields. There was previously no way to adjust the value without full keyboard entry.
What will change?
-
Behaviour:
NumberInputcan now render a vertical up/downButtonGroupinside the field as an end adornment. Clicking increases/decreases the current value by step, clamped tominValue/maxValue. Buttons disable automatically at the limits. -
New props:
step?: number: the increment/decrement amount, defaults to1.spinnerButtons?: "always" | "onHover" | "never": controls spinner visibility. Defaults to"onHover"for all number modes except"scientific", which defaults to"never"(since stepping isn't a natural operation on scientific-notation values).- No new component: this extends the existing
NumberInput.
Interface changes (if any)
Additive/optional only — existing usages of NumberInput are unaffected.
<NumberInput
label="Quantity"
numberMode="natural"
step={1}
spinnerButtons="always" // "always" | "onHover" | "never"
/>
Breaking change?
- Yes
- No
Next steps
A maintainer will review this issue.
If accepted, it will be marked as accepted and a PR may then be opened.
- Langage dominant
- TypeScript
- Étoiles
- 9
- Forks
- 4
- Merge moyen
- 3 j 12 h
- PR mergées (30 j)
- 14
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de DiamondLightSource/sci-react-ui
-
accepted
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
DiamondLightSource/sci-react-ui#325 · 1 commentaire ·
Les mainteneurs répondent en général sous 3 jours
-
accepted
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
DiamondLightSource/sci-react-ui#283 · 1 commentaire ·
Les mainteneurs répondent en général sous 3 jours
-
needs-triage
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
DiamondLightSource/sci-react-ui#365 ·
Les mainteneurs répondent en général sous 3 jours
-
accepted
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
DiamondLightSource/sci-react-ui#357 ·
Les mainteneurs répondent en général sous 3 jours
-
Spacing at the top of SidebarNav should be configurablePeut-être pris @swara030 l’a pris il y a 16 jours. Ouverteaccepted
Difficulté 3/5 1-2 jours Accessibilité débutants 65/100
DiamondLightSource/sci-react-ui#334 · 1 commentaire ·
Les mainteneurs répondent en général sous 3 jours
Toutes les issues de DiamondLightSource/sci-react-ui
Issues similaires
-
[Bug]: Pen taps on sidebar rows start a drag and lose their clickPeut-être pris @makhnatkin l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
b310-digital/teammapper#1515 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 79/100
puckeditor/puck#1884 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
outline/outline#14378 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
nicobailon/pi-web-access#642 ·
Les mainteneurs répondent en général sous 1 jour