cssinjs/jss

react-jss: Observables issue

Open

#1193 aperta il 17 set 2019

Vedi su GitHub
 (0 commenti) (0 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: Observables should not change the {link: true} to {link: false}

Describe the bug: If Observable's internal method is enumerable, then react-jss considers it a dynamic style and adds an unnecessary empty class. On the other hand, if the method is not enumerable, then the empty class is not added, but it does not link the styles and observable styles do not work.

Codesandbox link: https://codesandbox.io/s/react-jss-v1000alpha-25-observable-issue-g8998

Versions (please complete the following information):

  • jss: 10.0.0-alpha.25
  • Browser [e.g. chrome, safari]: Chrome
  • OS [e.g. Windows, macOS]: Windows

Guida contributor