cssinjs/jss

createUseStyles + @media + Adapting based on props: not updating correctly

Open

#1327 aperta il 7 apr 2020

Vedi su GitHub
 (7 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: When a property is changed, styles that are inside a @media selector are not being re-generated.

Describe the bug: When a property is changed, all styles that are properties-dependent should be re-generated and re-applied to the component.

Steps:

  1. "Swap color" button loads with background red and border/text-shadow blue
  2. Click on the "Swap color" button
  3. Background is changed to "blue" ✅
  4. Text-shadow is changed to "red" ✅
  5. Border is kept "blue" ❌

Codesandbox link: https://codesandbox.io/s/createusestyles-media-vf5zs

Versions (please complete the following information):

  • react-jss: 10.1.1
  • Browser [e.g. chrome, safari]: chrome (but all)
  • OS [e.g. Windows, macOS]: macOs (but all) Feel free to add any additional versions which you may think are relevant to the bug.

Guida contributor

Direzione di ricerca
Investiga come JSS gestisce gli stili dinamici all'interno delle query @media. Concentrati sul ciclo di aggiornamento quando le proprietà cambiano e perché la regola del bordo non viene rivalutata mentre le altre sì. Esamina il codice in packages/react jss/src/createUseStyles.js e le classi StyleSheet e Rule del core JSS per la gestione delle media query.
Tech stack
javascriptreact
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.
Mezza giornata
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
ReactCSS in JS
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60