cssinjs/jss

[jss-plugin-props-sort] CSS Property borderColor is not applied to both borderLeft and borderRight

Open

#1 161 ouverte le 14 juil. 2019

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)JavaScript (434 forks)batch import
bughelp wanted

Métriques du dépôt

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

Description

Expected behavior: CSS Property borderColor shoud apply to both CSS Properties borderLeft and borderRight.

Describe the bug: CSS Property borderColor is not being applied to both borderLeft and borderRight. It is only applied to borderLeft.

Codesandbox link: Sandbox

Versions (please complete the following information):

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

Guide contributeur

Direction de recherche
Le problème est que la propriété raccourcie 'borderColor' n'est pas étendue pour inclure 'borderRight' dans la sortie triée. Étudiez le code source de jss plugin props sort (probablement dans packages/jss plugin props sort/src/index.js) pour voir comment les propriétés CSS sont triées et comment les extensions des raccourcis sont gérées. Vérifiez s'il manque un mappage pour 'borderColor' afin d'inclure 'borderRight'. De plus, examinez le sandbox lié pour la reproduction. La correction peut impliquer d'ajouter 'borderRight' à la liste des propriétés auxquelles 'borderColor' doit correspondre, ou d'ajuster la logique de tri pour gérer correctement les raccourcis CSS.
Stack technique
javascript
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
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70