Development #57286
nouveau thème : La Rochelle
0%
Description
Preuve de concept, intégration native.
Ticket client #56681, en attente des éléments graphiques de leur part.
Fichiers
Demandes liées
Révisions associées
la-rochelle: add fallback font to sans-serif (#57286)
Historique
Mis à jour par Paul Marillonnet il y a plus de 2 ans
- Statut changé de Nouveau à En cours
Éléments graphiques fournis dans le ticket client.
Mis à jour par Paul Marillonnet il y a plus de 2 ans
- Fichier account-management-wide.png account-management-wide.png ajouté
- Fichier cell-example.png cell-example.png ajouté
- Fichier form-full.png form-full.png ajouté
- Fichier form-full-narrow.png form-full-narrow.png ajouté
- Fichier login-narrow.png login-narrow.png ajouté
- Fichier __style__-one-column.png __style__-one-column.png ajouté
- Fichier __style__-two-columns.png __style__-two-columns.png ajouté
- Fichier 0001-add-la-rochelle-theme-57286.patch 0001-add-la-rochelle-theme-57286.patch ajouté
- Tracker changé de Support à Bug
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
En se rapprochant des éléments graphiques fournis dans le ticket client ça donne ça, avec les effets de survol qu’on ne voit pas dans les captures.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 2 ans
critiques visuels :
- les labels de formulaire qui ressemblent à des boutons c'est amha une mauvaise idée
critique code :
- `unset` je me demande si c'est pas un peu prématuré à utiliser (taux de support). Moi j'ose pas encore. `initial` c'est mieux (taux de support) mais pas de IE.
- li::marker : non, là support vraiment insuffisant
J'ai tendance à utiliser tout ce qui n'est pas supporté par IE11 uniquement en amélioration progressive.
input[type="text"], textarea, div.qommon-map, div.file-button, input[type="email"], input[type="date"], input[type="password"], select { border-radius: 7px; border: 1px solid #cccccc; }
Je ne comprends pas pourquoi tu écris ça plutot que d'utiliser les vars scss $widget-border et $widget-border-radius
div.gru-content button { font-weight: bold; &[name="submit"] { background: $primary-color; &:hover { background: $primary-color-focus; } } &[name="cancel"] { background: #cd3939; &:hover { background: #a72b2b; } } }
utlise plutôt
%button {] et %cancel-button {}
div.gru-content div.cell h2:first-child { color: $secondary-color-focus; }
$title-color ?
Mis à jour par Paul Marillonnet il y a plus de 2 ans
- Statut changé de Solution proposée à En cours
Ok, merci pour la relecture. Tout pris en compte sauf
- li::marker : non, là support vraiment insuffisant
dont je n’ai pas encore trouvé de substitut qui conviendrait à un ensemble plus grand de navigateurs.
Je regarde.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 2 ans
J'ai poussé 2 petits commits :
1 pour virer du code inutile
1 autre pourt remplacer les `initial`
dont je n’ai pas encore trouvé de substitut qui conviendrait à un ensemble plus grand de navigateurs.
Je regarde.
Ça me va, vu l'utilisation que tu en fais, ça peut être considéré comme de l'amélioration progressive. En tout cas les users sous IE peuvent utiliser la plateforme.
Mis à jour par Paul Marillonnet il y a plus de 2 ans
- Fichier 0001-add-la-rochelle-theme-57286.patch 0001-add-la-rochelle-theme-57286.patch ajouté
- Statut changé de En cours à Solution proposée
Thomas Jund a écrit :
J'ai poussé 2 petits commits :
1 pour virer du code inutile
1 autre pourt remplacer les `initial`
Vu, merci.
Ça me va, vu l'utilisation que tu en fais, ça peut être considéré comme de l'amélioration progressive. En tout cas les users sous IE peuvent utiliser la plateforme.
Ok, nouveau patch qui inclut tes deux commits, et la modification suivante :
+div#columns div.wcs-forms-of-category-cell ul li::marker {
+ content: '';
+}
+
Pour que les marqueurs de liste non ordonnée n’apparaissent pas dans les cellules listant les formulaires d’une catégorie.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Paul Marillonnet il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 430979173198612016321b6b4afb27805161c84b Author: Paul Marillonnet <pmarillonnet@entrouvert.com> Date: Wed Oct 13 15:38:10 2021 +0200 add la rochelle theme (#57286)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Paul Marillonnet il y a plus de 2 ans
- Lié à Development #58338: la rochelle : corrections ajouté
add la rochelle theme (#57286)