cssinjs/jss

[react-jss] Observable rules/values

Open

#995 ouverte le 20 janv. 2019

Voir sur GitHub
 (10 commentaires) (0 réactions) (0 assignés)JavaScript (7 000 stars) (434 forks)batch import
bugcomplexity:moderatehelp wanted

Description

Expected behavior: Observables should work without explicitly passing link: true as options and should be part of the static stylesheet. And no dynamic stylesheet should be generated when only observables are used.

Describe the bug: Currently, observables are part of the static sheet but the user needs to pass link: true and a dynamic sheet will also be generated.

Codesandbox link: https://codesandbox.io/s/01ro5xn32p

Versions (please complete the following information):

  • jss: 10.0.0-alpha.8

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
ReactJSSobservablesCSS in JS
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
The issue reports that observables in react jss require the `link: true` option and generate a dynamic stylesheet unnecessarily. Investigate the JSS source code, particularly the handling of observables in static sheets. Look at the codesandbox link for reproduction. The fix likely involves modifying the sheet generation logic to only generate a static sheet when only observables are used and remove the need for `link: true` explicitly.