styleguidist/react-styleguidist

Para components should use `theme.fontSize.base` or `theme.fontSize.text`

Open

#1661 aperta il 13 ago 2020

Vedi su GitHub
 (11 commenti) (0 reazioni) (0 assegnatari)TypeScript (1416 fork)github user discovery
buggood first issuehelp wanted

Metriche repository

Star
 (11.091 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Current behavior

When stylegudist is configured to use a font size for base and/or text to be anything other than 16px, the paragraph content generated from markdown does not use the specified font size.

Question: Is a markdown paragraph considered to be either base or text? I am assuming it should be text at the very least.

To reproduce

  1. Config:

    module.exports = {
  theme: {
    fontSize: {
      base: 24,
      text: 36,
    },
  },
};

  2. Some markdown file:

    # I am an H1

I am just a paragraph! My font size should be really big!

https://github.com/styleguidist/example/pull/8

Expected behavior

The generated paragraph content should use the theme provided.

Or there should be some explanation of how the "theme" is defined: How is a paragraph not "text" or "base"?

Guida contributor

Direzione di ricerca
Esamina il componente Paragraph nel codice sorgente di styleguidist, traccia dove dovrebbe essere applicato theme.fontSize.base o theme.fontSize.text e verifica che i paragrafi markdown utilizzino i valori del tema corretti.
Tech stack
typescriptjavascriptreact
Dominio
frontenddeveloper experiencetooling
Tipo issue
Bug
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
GitJavaScriptReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70