Light and Light Vibrant: strings, regexps, escapes and invalid tokens are below 4.5:1 on the editor background

Aperta Adatta ai principianti
  • Qualcosa non va?
#19 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
65/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
typescript, vscode
Ambito
accessibility, design

Direzione di ricerca

Inizia da src/palette.ts, dove sono definiti i ruoli syntax.string, syntax.regexp, syntax.escape e syntax.invalid, e individua i due scope #f44747 hard-coded (invalid.illegal.non-null-typehinted.php e token.error-token) che aggirano questi ruoli. Verifica i valori proposti, di un passo più scuri, rispetto alla tabella di contrasto nella issue, usando la luminanza relativa WCAG rispetto a #ffffff. Il lavoro è completo quando ogni ruolo elencato raggiunge almeno 4,5:1 sullo sfondo chiaro e gli scope hard-coded usano syntax.invalid. I temi scuri devono restare invariati.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

accessibility

Measured with WCAG 2.2 relative luminance against editor.background (#ffffff):

Role Colour Ratio
syntax.string (green 600) #13924c 4.00
syntax.regexp (teal 600) #169d96 3.33
syntax.escape (cyan 600) #1a95cc 3.38
syntax.invalid (red 500) #e62845 4.42
hard-coded #f44747 (invalid.illegal.non-null-typehinted.php, token.error-token) #f44747 3.60

Light Vibrant (Display P3) is lower still for the same roles (3.0 to 4.0, indicative).

The next step of each ramp already passes: green 700 (5.7), teal 700 (4.9), cyan 700 (5.0), red 600 (5.9). Proposal: move those four roles one step darker in src/palette.ts and point the two hard-coded #f44747 scopes at syntax.invalid (they are also never converted to P3 in the vibrant themes). Strings would become visibly darker green, which is the visible trade-off. Dark themes are not affected.

🤖 Generated with Claude Code

Lingua principale
TypeScript
Stelle
7
Fork
2
Merge medio
1h 16m
PR unite (30g)
18

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di twbs/bootstrap-vscode-theme

Tutte le issue di twbs/bootstrap-vscode-theme

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.