gajus/babel-plugin-react-css-modules

Not compatible with css-loader@3.0.0

Open

#258 ouverte le 24 juin 2019

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)JavaScript (176 forks)batch import
help wantedquestion

Métriques du dépôt

Stars
 (2 044 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

I have localIdentName set to '[path]___[name]__[local]___[hash:base64:5]' in the css-loader, but as of 3.0.0 this generates a name (in css-loader) where slashes and dots aren't replaced with dashes but rather escaped.

That's a bit uglier to read now (anyway, they aren't really meant for humans) -- but the main problem is that this plugin now generates invalid className values because it's using the old logic.

Example:

className: rb-client-js-modules-landing-___Landing-module__landing-page-card___kz6iY css: .rb\/client\/js\/modules\/landing\/___Landing\.module__landing-page-card___kz6iY

Guide contributeur

Direction de recherche
Examinez comment css loader@3.0.0 echappe les noms de classes (en ajoutant une barre oblique inverse avant les caracteres speciaux) et mettez a jour la logique de generation de className du plugin pour correspondre a la nouvelle logique d'echappement.
Stack technique
javascriptreactnodejs
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptReactCSS Modules
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70