Projet

Général

Profil

Development #51618

Nouveau thème: Sables d'Olonne

Ajouté par Thomas Jund il y a environ 3 ans. Mis à jour il y a environ 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
03 mars 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

ticket de référence : #43051


Fichiers

Révisions associées

Révision 4b566df8 (diff)
Ajouté par Thomas Jund il y a environ 3 ans

new theme: sable-d-olonne (#51618)

Historique

#1

Mis à jour par Thomas Jund il y a environ 3 ans

  • Tracker changé de Support à Development
#3

Mis à jour par Marie Kuntz -> retour le 13 mai il y a environ 3 ans

Impec, à part le menu qui se promène au milieu en mobile (mais je suppose que c'est dû à la capture d'écran hasardeuse de FF)

#4

Mis à jour par Nicolas Roche il y a environ 3 ans

C'est beau !

Quelques notes juste pour moi, ou celui qui déploiera le thème.
  • pas de menu, on utilise que celui du pwa.
  • pour les cellules liens, lies vers un formulaire et le code de suivi dans combo, il faut ajouter la classe CSS cell-icon dans les options de la cellule (notamment pour avoir la loupe centré dans la cellule code de suivi)
Pour le côté esthétique je suis mal placé pour juger, et ne vois que ces quelques améliorations éventuelles (mais perso je demande rien ici) :
  • top-link : les liens relatif à la connexion dans l’ovale violet sont soulignés blancs au survol, j'aurais plutôt vu un changement de couleur du texte (pour mieux coller au "bouton" connexion).
  • sur la maquette, le code de suivi affiché dans wcs prend toute la hauteur du formulaire (c'est peut-être mieux comme ça : de ne pas encombrer la page).

(Côté technique j'avoue que je suis aussi un peu dépassé, j'ai surtout testé en fait et j'ai rien vu de choquant.)
Pour le côté fonctionnel, il y a quand même 0002 qui me pose question.

Au cas où voici 3 patchs :

  • 0001 Patch où je corrige juste des éspacements :
    • un espace qui traînent dans _custom.scss
    • espaces insécables dans config.json
        "header:logo": { "label": "Têtière : logo" },
        "emails:logo": {"label": "Emails : logo"},
        "footer:logo": {"label": "Pied de page : logo"}
      
  • 0002 : Je n'ai pas réussi à utiliser les spécificités du thème sans déplacer les templates :
      $ cd templates/variants/sables-d-olonne/
      $ mv combo/cells/cell-icon/link-cell.html combo/
      $ mv combo/form.html combo/wcs/
      $ mv combo/cells/cell-icon/tracking_code_input.html combo/wcs/
    
  • 0003 : Pour utiliser les "cell-icon" CSS j'ai d'abord cru qu'il me faudrait changer le slug de la cellule avec par exemple 'cell-icon-violet'. Du coup peut-être changer le commentaire dans le CSS.
    //    Slug 'cell-icon', 'cell-icon-violet', 'cell-icon-cyan'
    

J'aurais mis : CSS additional classes : 'cell-icon', 'cell-icon violet', 'cell-icon cyan'

#5

Mis à jour par Thomas Jund il y a environ 3 ans

pas de menu, on utilise que celui du pwa.

Ouep, pour le moment il n'est pas prévu d'utiliser la nav principale.

top-link : les liens relatif à la connexion dans l’ovale violet sont soulignés blancs au survol, j'aurais plutôt vu un changement de couleur du texte (pour mieux coller au "bouton" connexion).

Je suis d'accord, mais aucun survol de défini dans les maquettes. Alors je laisse comme ça pour les laisser décider.

sur la maquette, le code de suivi affiché dans wcs prend toute la hauteur du formulaire (c'est peut-être mieux comme ça : de ne pas encombrer la page).

Oui mais on est sur du float, donc pas possible techniquement. Pour cela il faudrait hacker en flexbox et hack pas terrible, parce que pas de parent spécifique. Il y a un ticket ouvert par Fred (depuis longtemps) pour passer le layout de WCS vers le layout "1 col + sidebar" de combo, ce serait une solution plus propre (mais pas encore pris le temps, parce que ticket qui va demander de passer en revue tous les thèmes existant).

pour les cellules liens, liens vers un formulaire et le code de suivi dans combo, il faut ajouter la classe CSS cell-icon dans les options de la cellule (notamment pour avoir la loupe centré dans la cellule code de suivi).

Non, j'ai fais le choix de conserver les templates existant pour être utilisable au cas où et mieux pour les updates.
J'ai donc créer des templates spécifiques via un slug `cell-icon`. Slug disponible pour cellules `lien`, `lien vers démarche`, `code de suivi`.
Ensuite possible d'y ajouter les class `violet` ou `cyan`.

Exemple: créer une cellule lien, ajouter le slug `cell-icon`, donc possibilité d'ajouter une ïcone via ressources combo puis optionnellement d'ajouter une class `violet`.

Le class violet est aussi en option pour les cellules texte.

Je te joins ma home page de test pour import de ton côté

#6

Mis à jour par Nicolas Roche il y a environ 3 ans

J'ai donc créé des templates spécifiques via un slug `cell-icon`

arf, #7141 si quelqu'un d'autre que moi se demande d'où ça sort.

Exemple: créer une cellule lien, ajouter le slug `cell-icon`, donc possibilité d'ajouter une ïcone via ressources combo puis optionnellement d'ajouter une class `violet`.

Peu-être ce commentaire alors ?

//    Slug 'cell-icon'
//    Extra classes for CSS styling 'violet', 'cyan'

Pour moi c'est bon, mais je réalise que je ne suis que backup en fait.
(Je rends la main au CPT)

#8

Mis à jour par Nicolas Roche il y a environ 3 ans

  • Statut changé de Solution proposée à Solution validée

(vraiment histoire de dire, il reste les typos sur les blancs : plus que celle de _custom.scss puisque tu as remplacé l'espace insécable dans config.json)

#9

Mis à jour par Thomas Jund il y a environ 3 ans

ah si, vu l.172, un espace planqué entre 2 tabs. Je patch. Merci nico.

#10

Mis à jour par Thomas Jund il y a environ 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 4b566df8dcdc98321befe399cced974defcbaa52 (HEAD -> main, origin/main)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Fri Feb 12 15:39:43 2021 +0100

    new theme: sables-d-olonne (#51618)
#11

Mis à jour par Frédéric Péters il y a environ 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF