Development #64542
Theme Miribel
Début:
25 avril 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Intégration thème Miribel à partir des maquettes et du style guide réalisé pra j'Articule.
Maquette et style guide dans ticket projet #64350
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Jund il y a presque 2 ans
- Sujet changé de theme Miribel à Theme Miribel
- Description mis à jour (diff)
Mis à jour par Thomas Jund il y a presque 2 ans
- Fichier miribel-wcs-mobile.png miribel-wcs-mobile.png ajouté
- Fichier miribel-wcs-desktop.png miribel-wcs-desktop.png ajouté
- Fichier miribel-combo-mobile.png miribel-combo-mobile.png ajouté
- Fichier miribel-combo-desktop.png miribel-combo-desktop.png ajouté
- Fichier 0001-miribel-new-theme-64542.patch 0001-miribel-new-theme-64542.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Corentin Séchet il y a presque 2 ans
- Fichier links-in-cell-text.png links-in-cell-text.png ajouté
C'est de toute beauté.
J'ai juste relevé un bug sur l'affichage des liens dans les descriptions de formulaires, j'ai pushé une correction sur la branche.
Et, pour ma culture, quel est l'intérêt de faire :
.tracking-code-input-cell { .gru-content &.cell { ... } }
plutôt que :
.gru-content .tracking-code-input-cell.cell { ... }
?
Mis à jour par Thomas Jund il y a presque 2 ans
Merci pour la relecture.
Le résultat post compilation est le même.quel est l'intérêt de faire.
Pour moi l'intérêt est
- au niveau de la clarté du code : 1 seul selecteur est plus facile à repérer.
- et aussi pouvoir utiliser la fonction sass @extend. Par exemple, si je veux qu'une cellule hérite du même design que la cellule tracking-code, je peux faire `@extend .tracking-code-input-cell`, ce qui n'est pas possible avec un selecteur complexe.
Mis à jour par Corentin Séchet il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
C'est clair, merci du tuyau.
Mis à jour par Thomas Jund il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 95d65dd64d39472bd233afa3d69621ede0b6639a Author: Thomas JUND <tjund@entrouvert.com> Date: Tue Apr 12 14:41:54 2022 +0200 miribel: new theme (#64542)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
miribel: new theme (#64542)