casesandberg/react-color

Wrong Brightness Coefficients

Open

#184 ouverte le 2 juil. 2016

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

Description

https://github.com/casesandberg/react-color/blob/master/modules/tinycolor2/index.js#L76

For now there are coefficients for NTSC, not sRGB. And i think that w3c is little bit outdated.

Also see this demo: http://codepen.io/dom1n1k/pen/LZWjbj

See sRGB table: http://www.brucelindbloom.com/index.html?WorkingSpaceInfo.html Or point 9 here: http://www.poynton.com/notes/colour_and_gamma/ColorFAQ.html

Guide contributeur

Stack technique
javascript
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.
mostly clear
Prérequis
JavaScriptcolor theory basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
The issue points to wrong brightness coefficients in the tinycolor2 module at modules/tinycolor2/index.js line 76. The current coefficients are for NTSC, but should be sRGB coefficients. The user provides references to sRGB coefficients from Bruce Lindbloom and Poynton. To fix, replace the coefficient values in the luminance calculation with correct sRGB values. Ensure the change matches the expected output from the demo link. No linked PRs exist.