FormidableLabs / FormidableLabs/react-live

pre-defined style fontFamily breaks Prism theme

Aperta
#319 0 commenti 1 reazione 0 assegnatari Vedi su GitHub
Lingua principale
TypeScript
Stelle
4.6k
Fork
260
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Line 51 `fontFamily: "inherit"`

https://github.com/FormidableLabs/react-live/blob/e21fd338e61641b1960d1f3650d6ba7482ceae03/src/components/Editor/index.js#L48-L51

Will override Prism theme

https://github.com/PrismJS/prism/blob/0d4b6cb64e72a172fc64423c04aed4ac8b59cd0b/themes/prism.css#L12

Further, why not let users define all the style (margin, outline, padding, fontFamily) by themself through `className`? This way, React Live will become a true HeadlessUI component.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia in src/components/Editor/index.js intorno alla riga 51 e confronta la fontFamily predefinita con il Prism theme CSS di riferimento. Verifica come vengono gestiti attualmente className e gli stili inline; il lavoro è completato quando la fontFamily predefinita non sovrascrive più il tema Prism e il comportamento richiesto per la personalizzazione degli stili è stato implementato.

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

Valutazione

Stack tecnologico
react, typescript
Ambito
frontend
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.