palantir/blueprint

FR: Support small modifier for NumericInput

Open

#3 132 ouverte le 7 nov. 2018

Voir sur GitHub
 (2 commentaires) (1 réaction) (0 assignés)TypeScript (2 167 forks)batch import
P3Package: coreType: feature requesthelp wanted

Métriques du dépôt

Stars
 (20 263 stars)
Métriques de merge PR
 (Merge moyen 43j 10h) (27 PRs mergées en 30 j)

Description

Environment

  • Package version(s): 3.8
  • Browser and OS versions: Chrome 70

Feature request

I like to make a form using small modifier on components. I can use small modifier on InputGroup and Button but MenuItem and NumericInput does not support small modifier. I tried className={Classes.SMALL} but it did not work. You can look at screenshot below as an example

Could you implement support for these components? Probably there are other components that does not support small modifier. I prefer support for all components which can be used inside a form.

Examples

<Button small={true} rightIcon="double-caret-vertical" .../> <InputGroup small={true} .../> <NumericInput leftIcon="arrow-top-left" .../> // does not support small modifier <NumericInput .../>

Guide contributeur

Direction de recherche
Cette demande propose d'ajouter un modificateur 'small' à NumericInput, similaire au support existant dans Button et InputGroup. Étudiez l'implémentation de la prop 'small' dans ces composants (par exemple, Button.tsx et InputGroup.tsx) pour comprendre le modèle. Ensuite, examinez NumericInput.tsx et ses styles associés dans numeric input.scss pour ajouter le modificateur Small. La demande mentionne que className={Classes.SMALL} n'a pas fonctionné, donc la correction implique probablement d'ajouter une prop ou une classe CSS qui applique le style de petite taille. Il n'y a pas de PR liés ni de commentaires des mainteneurs, donc l'implémentation est simple et peut suivre le modèle existant.
Stack technique
typescript
Domaine
frontend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScriptReact
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75