Microsoft/vscode

Use modern (CSS color level 4) syntax for colors in the color picker

Open

#139.840 aperta il 28 dic 2021

Vedi su GitHub
 (7 commenti) (28 reazioni) (1 assegnatario)TypeScript (10.221 fork)batch import
editor-color-pickerfeature-requesthelp wanted

Metriche repository

Star
 (74.848 star)
Metriche merge PR
 (Merge medio 11h 43m) (1000 PR mergiate in 30 g)

Descrizione

When converting colors using the color picker, the old comma-based syntax is used:

image (it should also show hsl here, at the top of the picker, not rgba)

Please update the picker to use the modern (CSS color level 4) syntax for colors.

With regard to the exact syntax, I agree with these suggestions from @mathiasbynens:

  • consider being opinionated w.r.t. how to represent alpha values. IMHO using percentage notation is most readable, as it clarifies the range: e.g. 70% vs 0.7
  • consider being opinionated about using the deg suffix where applicable, e.g. in the first parameter for hsl() and the last non-alpha parameter for lch(). Again, because this clarifies that the range lies between 0-360.

i.e. Use percentages for alpha, and always include the deg suffix.

This also matches the syntax displayed in Chrome developer tools.

Guida contributor

Direzione di ricerca
Trova il codice sorgente del selettore colore, individua le funzioni di conversione del colore e aggiornale per produrre la sintassi CSS Color Level 4 (ad esempio, `hsl(200deg 50% 30% / 80%)` invece di `rgba(0,0,0,0.8)`). Usa la percentuale per l'alpha e includi il suffisso `deg` per l'angolo di tonalità. Fai riferimento al formato degli strumenti di sviluppo di Chrome per coerenza.
Tech stack
typescript
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75