casesandberg / casesandberg/react-color

Addition/Deletion of number for Saturation or Lightness in the ChromePicker replaces the value with "%%".

Open
#645 6 comments 8 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
12.3k
Forks
920
PR merge metrics
No merged PRs in 30d

Description

Steps to reproduce:

1) Using the ChromePicker component, assuming saturation/lightness value to be 100%, delete the two 0's from the number.
Expected Saturation/Lightness value: 1% , Result Saturation/Lightness value: 1%%
2) Now click anywhere on the screen, the value changes to 100%. Expected value was 1%.

![ChromePicker](https://user-images.githubusercontent.com/51934156/65911372-65388500-e381-11e9-876c-2abe72d1bf3d.PNG)

3) Similarly, when entered a number in the Saturation/ /Lightness the number is appended by two percentages "1%%" or "12%%" or "122%%". This is also reproducible for Hue component.

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the issue in the ChromePicker component, editing the saturation, lightness, and hue number inputs as described. Trace how those inputs format and update their percentage values, then verify that editing does not create duplicate percent signs and preserves the entered value after focus leaves the field.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, react
Domain
frontend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.