cssinjs/jss

Extend function property values are converted to strings

Open

#1134 aperta il 17 giu 2019

Vedi su GitHub
 (0 commenti) (2 reazioni) (0 assegnatari)JavaScript (434 fork)batch import
bughelp wanted

Metriche repository

Star
 (7000 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Expected behavior: A clear and concise description of what you expected to happen.

const styles = {
  linkCommon: {
    color: () => "orange"
  },
  link: {
    extend: "linkCommon"
  }
};

The link should be orange.

Describe the bug: A clear and concise description of what happend instead of the expected behavior.

The function is being converted to a string before being inserted into the DOM.

image

Codesandbox link:

Playground: https://codesandbox.io/embed/core-jss-playground-ef0j4

Versions (please complete the following information):

  • jss: 10 alpha 18
  • jss-preset-default: 10 alpha 18
  • Browser Chrome 75
  • OS: OX X

Feel free to add any additional versions which you may think are relevant to the bug.

Not to be confused with https://github.com/cssinjs/jss/issues/479

Guida contributor

Direzione di ricerca
Indaga su come la proprietà `extend` fonde gli stili quando i valori sono funzioni. Esamina i file sorgente nel repository JSS, in particolare la logica di fusione degli stili (ad es., `src/`). Il problema mostra che quando una funzione viene utilizzata come valore in uno stile esteso, viene convertita in una stringa. Confronta con la gestione delle funzioni negli stili non estesi. Verifica se esiste un caso di test per scenari simili e considera di aggiungerne uno. Il link al playground suggerisce che il codice utilizza jss preset default; assicurati che la correzione funzioni con il preset predefinito.
Tech stack
javascript
Dominio
frontend
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
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75