cssinjs/jss

[nested] Nested selectors are not calculated properly with `$` in it

Open

#1321 aperta il 26 mar 2020

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: Nested selectors should include escaped $ in its parent's selector

Describe the bug: selectors that are nested drop the $ and keep the escape backslash, while their parent selector includes \$.

  1. Open up https://cssinjs.org/repl
  2. On line 23, change the selector to '$Button' or '\$Button'
  3. Observe the nested selectors generated

The issue happened in my react app where components are mangled by webpack, and some of them starts with $ (like $z). In that situation, the nested selectors do not work.

Codesandbox link: Just use the https://cssinjs.org/repl

Versions (please complete the following information):

  • jss: 10.0.0
  • 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.

Annotation 2020-03-25 174036

Guida contributor

Direzione di ricerca
Investiga la logica di parsing dei selettori in JSS, in particolare come gestisce il `$` escapato nei selettori nidificati. Cerca casi di test e la parte di codice che combina selettori padre e figlio.
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
JavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75