FormidableLabs / FormidableLabs/react-live
pre-defined style fontFamily breaks Prism theme
- Vorherrschende Sprache
- TypeScript
- Sterne
- 4.6k
- Forks
- 260
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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.
Beitragsleitfaden
Rechercherichtung
Beginne in src/components/Editor/index.js ungefähr bei Zeile 51 und vergleiche die vordefinierte fontFamily mit dem referenzierten Prism theme CSS. Prüfe, wie className und Inline-Stile derzeit behandelt werden; die Aufgabe ist abgeschlossen, wenn die vordefinierte fontFamily das Prism theme nicht mehr überschreibt und das angeforderte Verhalten zur Stil-Anpassung umgesetzt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- react, typescript
- Bereich
- frontend
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100