highlightjs/highlight.js

(themes) "faded" comments should be optional, not default.

Open

#3483 aperta il 15 feb 2022

Vedi su GitHub
 (10 commenti) (1 reazione) (0 assegnatari)JavaScript (3497 fork)batch import
enhancementgood first issuehelp welcometheme

Metriche repository

Star
 (22.960 star)
Metriche merge PR
 (Merge medio 6m) (2 PR mergiate in 30 g)

Descrizione

I think perhaps for accessibility that a pass over ALL the comment styles in all (1st party, not Base16) themes and pulling them up to a 3 or 4.5 contrast and then recommend that those who want the common "i can barely see the comments" look/feel should apply opacity of their desired choosing to hljs-comment. That way we equally support both accessibility and user choice.

I'm going to split this into a separate issue and add it to the v12 list.

Update: Definitely seems like we should first update the theme checker tool to use APCA.

Originally posted by @joshgoebel in https://github.com/highlightjs/highlight.js/issues/3482#issuecomment-1040565915

Guida contributor

Direzione di ricerca
Indaga i file di tema attuali nel repository, in particolare gli stili dei commenti. Comprendi lo standard di contrasto APCA e come si applica all'evidenziazione della sintassi. Esamina lo strumento di controllo del tema menzionato nell'issue per vedere come può essere aggiornato per utilizzare APCA. Controlla l'issue #3482 per ulteriore contesto. L'obiettivo è far sì che gli stili dei commenti abbiano un contrasto sufficiente (3-4.5) e raccomandare l'uso dell'opacità per un aspetto 'sbiadito'.
Tech stack
javascript
Dominio
frontendaccessibility
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Abbastanza chiara
Prerequisiti
CSSAccessibility
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65