cssinjs/jss

[react-jss]: Dynamic expanded styles are not updated correctly

Open

#1 051 ouverte le 6 mars 2019

Voir sur GitHub
 (0 commentaires) (2 réactions) (0 assignés)JavaScript (7 000 stars) (434 forks)batch import
bughelp wanted

Description

Expected behavior: Dynamic styles should correctly override the previously applied styles.

Describe the bug: Dynamic styles did not correctly override the previously applied styles.

Codesandbox link: https://codesandbox.io/s/742q2630yj ( Click "All Borders" button to toggle between two sets of styles )

Versions (please complete the following information):

  • jss: 10.0.0-alpha.12
  • Browser [e.g. chrome, safari]: chrome
  • OS [e.g. Windows, macOS]: Windows Feel free to add any additional versions which you may think are relevant to the bug.

Guide contributeur

Stack technique
javascriptreact
Domaine
frontend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
ReactJSSCSS in JS
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
Investigate dynamic style update logic in react jss. The bug likely resides in the `useStyles` hook or equivalent where dynamic styles are merged. Check the codesandbox for reproduction steps. Examine how JSS handles style expansion and update lifecycle.