cssinjs/jss

Using arrow function on jss-rtl dependent property does not works well

Aperta

#1234 aperta il 21 nov 2019

Vedi su GitHubGuida contributor
 (10 commenti) (4 reazioni) (0 assegnatari)JavaScript (434 fork)batch import
bughelp wanted

Metriche repository

Star
 (7000 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Expected behavior:

The rtl plugin should switch the styles.

Describe the bug:

The rtl plugin doesn't switch all the styles.

const useStyles = createUseStyles({
  myButton1: {
    paddingLeft: "16px" ✅
  },
  myButton2: {
    paddingRight: ({ isMobile }) => (isMobile ? 0 : "26px") ❌
  },
  myLabel: props => ({
    paddingRight: "8px" ✅
  })
});

Codesandbox link:

https://codesandbox.io/s/react-jss-playground-37lqs

Versions (please complete the following information):

  • jss: 10.0.0
  • Browser [e.g. chrome, safari]: all
  • OS [e.g. Windows, macOS]: all

First reported in https://github.com/mui-org/material-ui/issues/18477.

Guida contributor

Direzione di ricerca
Esamina come il plugin RTL gestisce i valori delle funzioni negli stili JSS. Concentrati sul codice che elabora le funzioni freccia e verifica che applichi correttamente le trasformazioni RTL a tutti i valori.
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
ReactJSS
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50