Use SASS modules, remove all uses of @import rule #17
Conversation
| css: { | ||
| preprocessorOptions: { | ||
| scss: { | ||
| additionalData: `@import "./src/styles/_variables.scss"; @import "./src/styles/mixins/_media-queries.scss";`, |
There was a problem hiding this comment.
Pourquoi enleve ca?
C'est poche de devoir import et prefixer les helpers.
@use "../../../../styles/variables" as v;
v.get-spacing(md);
There was a problem hiding this comment.
On pourait le remplacer par ca et evite les changements partout.
@use "./src/styles/variables" as *; @use "./src/styles/mixins/media-queries" as *;
|
|
||
| @media (max-width: (get-media(xxs) - 1)) { | ||
| @media (max-width: (v.get-media(xxs) - 1)) { |
There was a problem hiding this comment.
Je sais que ça n'a rien à avoir avec la tâche, mais sais-tu si il y a une raison à l'utilisation du max-width? Il me semble qu'on avait tous parlé de faire du mobile first et donc de toujours utiliser du min-width. Ça peut créer des problème d'utiliser du max et du min-width ensemble, il y a souvent 1px où les styles explosent quand on utilise les deux
Updates MUI to v6 with Pigment CSS
GitHub Issue or Internal Use Azure Devops Work Item ID:
Proposed Changes
What is the current behavior?
Our SASS code uses the @import rule.
What is the new behavior?
The SASS team discourages the continue use of the @import rule (it will be phased out eventually). You can read more about it here:
https://sass-lang.com/documentation/at-rules/import/