Overridable class strategy
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 52/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- css, javascript, tailwindcss
- Área
- frontend
Línea de trabajo
Start in src/index.js around line 353, where the class strategy is registered, and review the related strategy declarations. Change the class strategy's layer as requested, then verify the generated styles remain available while allowing later form styling to override them.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What version of @tailwindcss/forms are you using?
v0.5.7
Describe your issue
I read the discussion https://github.com/tailwindlabs/tailwindcss-forms/issues/22. The thread has been closed by adding class strategy.
Sorry to bring this up again
However, the class strategy is not easily overridable because it was placed later in the CSS file (at the components layer).
No matter the strategy used, tailwindcss-forms should always be a base, ready to be overridden by other stylings.
Why the need for overrides?
I can't talk for others, as you'll see some of them on the discussion links above, but for me, tailwindcss-forms adds colors that almost always need overriding because they stand out like sore thumbs.
What should we do?
Can we move CSS declarations of class strategy from components layer to base layer?
Basically, modify the code here:
https://github.com/tailwindlabs/tailwindcss-forms/blob/c9d9da3e010b194a1f0e9c36fbd98c83e4762840/src/index.js#L353
from addComponents to addBase.
Why do I, or other similar people, use tailwindcss-forms at all?
tailwindcss has its ecosystem, and some 3rd-party components expect tailwindcss-forms to be there as the base. We still want to work with tailwindcss-forms and the people creating those components, even if we are not a fan of the approaches.
- Lenguaje dominante
- HTML
- Estrellas
- 4.6k
- Forks
- 230
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de tailwindlabs/tailwindcss-forms
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
tailwindlabs/tailwindcss-forms#184 · 1 comentario ·
-
Dark mode background overrides text colors on checkboxes when using `class` dark mode variantAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
tailwindlabs/tailwindcss-forms#74 · 9 comentarios · 11 reacciones ·
Todos los issues de tailwindlabs/tailwindcss-forms
Issues similares
-
bug ux
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
CodeForFire/lagebuch#714 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
area:desktop priority:p2 type:chore
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
area: right sidebar
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
zulip/zulip#40449 · 3 comentarios ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día