casesandberg/react-color

Photoshop HSV up/down arrow controls are not working correctly

Open

#385 ouverte le 23 juil. 2017

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (11 679 stars) (946 forks)batch import
help wanted

Description

When using arrow up/down keys on the (S)aturation field, the Value fields jumps to 0. After this, losing focus of the Saturation field will have it jump back to 0 too, because the Value is 0 and the actual color value updates the color picker. You can see the docs page to confirm this.

Guide contributeur

Stack technique
javascriptreacthtmlcss
Domaine
frontend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Reactfamiliarity with color models
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Examine the Photoshop color picker component in `src/components/photoshop/Photoshop.js`. The bug is related to HSV input fields. Look at how the `Saturation` and `Value` fields interact with keyboard events. The issue reports that using arrow up/down on Saturation causes Value to jump to 0 and losing focus reverts it. Consider adding proper state updates and validation to prevent the Value from being incorrectly set to 0 when Saturation changes. There are no linked PRs or comments, so the fix is open. Ensure that the color conversion functions in `src/helpers/color.js` are correctly used.