entrouvert
/
publik-base-theme
15
1
Fork 1
Code Issues Pull Requests 10 Releases Wiki Activity

theme CD06 2022 #70929 #34

Merged
tjund merged 1 commits from wip/70929-cd06-2022 into wip/71572-vertical-steps-mixins 2022-11-28 14:50:41 +01:00
Conversation 7 Commits 1 Files Changed 24 +2838
tjund commented 2022-11-23 10:37:22 +01:00
Owner
Copy Link

avec 2 tickets core lié :

  • ajout {% block %} pour surcharger le template des steps wcs #71574
  • pouvoir appliquer les steps en mode vertical à tous les viewport #71572

Particularité notables du thème :

  • une fenêtre "togglabe" pour le menu user-info
  • une fenêtre "togglable" pour afficher les étapes d'une demande.

Charte Editoriale CD06

Combo, zones de saisies

  • Titre de la page : Insérer une cellule texte et une balise h1.
  • Bannière (bloc gauche) et Bannière (bloc droit) : (disponible uniquement pour le gabarit "Page d'accueil"), permet de saisir les informations de la bannière de la page d'accueil. L'image de la page s'affichera en arrière plan de la bannière.
  • Bas du contenu : S'affiche avec un arrière plan gris pour les gabarits qui ne possède pas de barre latérale.

Cellules

Les titres des cellules affichent un trapèze bleu. Ce trapèze est remplacé par l'image de la cellule.

Classes CSS additionnels

Options Cellules

  • gray-cell: ajoute un fond gris
  • buttons-list: affiche une liste de liens sous formes de boutons. Les boutons se positionnent les uns en dessous des autres.
  • inline-buttons-list: affiche une liste de liens sous formes de boutons. Les boutons se positionnent côtes à côtes s'ils ont la place
  • bordered-cell: ajouter une bordure à la cellule, les titres s'affichent sans trapèze (ex: bloc "des questions" sur la page d'accueil)

Option items liste de lien

  • voir-plus: option pour item de liste de lien. Transforme cet item en bouton avec flèche droite
avec 2 tickets core lié : * ajout {% block %} pour surcharger le template des steps wcs #71574 * pouvoir appliquer les steps en mode vertical à tous les viewport #71572 Particularité notables du thème : * une fenêtre "togglabe" pour le menu user-info * une fenêtre "togglable" pour afficher les étapes d'une demande. --- ## Charte Editoriale CD06 ### Combo, zones de saisies * *Titre de la page* : Insérer une cellule texte et une balise h1. * *Bannière (bloc gauche)* et *Bannière (bloc droit)* : (disponible uniquement pour le gabarit "Page d'accueil"), permet de saisir les informations de la bannière de la page d'accueil. L'image de la page s'affichera en arrière plan de la bannière. * *Bas du contenu* : S'affiche avec un arrière plan gris pour les gabarits qui ne possède pas de barre latérale. ### Cellules Les titres des cellules affichent un trapèze bleu. Ce trapèze est remplacé par l'image de la cellule. ### Classes CSS additionnels #### Options Cellules * `gray-cell`: ajoute un fond gris * `buttons-list`: affiche une *liste de liens* sous formes de boutons. Les boutons se positionnent les uns en dessous des autres. * `inline-buttons-list`: affiche une *liste de liens* sous formes de boutons. Les boutons se positionnent côtes à côtes s'ils ont la place * `bordered-cell`: ajouter une bordure à la cellule, les titres s'affichent sans trapèze (ex: bloc "des questions" sur la page d'accueil) #### Option items liste de lien * `voir-plus`: option pour item de liste de lien. Transforme cet item en bouton avec flèche droite
cd06-wcs-mobile-steps.png
56 KiB
cd06-combo-desktop.png
968 KiB
cd06-wcs-desktop.png
214 KiB
cd06-wcs-mobile.png
134 KiB
cd06-combo-mobile.png
187 KiB
cd06-wcs-mobile-steps.png cd06-combo-desktop.png cd06-wcs-desktop.png cd06-wcs-mobile.png cd06-combo-mobile.png
tjund requested review from smihai 2022-11-23 10:47:17 +01:00
fpeters requested changes 2022-11-23 11:07:59 +01:00
fpeters left a comment
Owner
Copy Link

Plutôt avoir un lien symbolique de templates/variants/alpes-maritimes-2022/combo/json/ vers templates/variants/alpes-maritimes-2018/combo/json/

(ça indiquerait aussi à la relecture que ces fichiers ne sont pas touchés).

Plutôt avoir un lien symbolique de templates/variants/alpes-maritimes-2022/combo/json/ vers templates/variants/alpes-maritimes-2018/combo/json/ (ça indiquerait aussi à la relecture que ces fichiers ne sont pas touchés).
tjund commented 2022-11-23 11:15:51 +01:00
Author
Owner
Copy Link

L'export combo de la home page

L'export combo de la home page
export_page_cd06-2022_20221123.json
20 KiB
tjund force-pushed wip/70929-cd06-2022 from f57ed247eb to 9e9ef11c2c 2022-11-23 16:11:38 +01:00 Compare
tjund commented 2022-11-23 16:14:11 +01:00
Author
Owner
Copy Link

Plutôt avoir un lien symbolique de templates/variants/alpes-maritimes-2022/combo/json/ vers templates/variants/alpes-maritimes-2018/combo/json/

Done

> Plutôt avoir un lien symbolique de templates/variants/alpes-maritimes-2022/combo/json/ vers templates/variants/alpes-maritimes-2018/combo/json/ Done
smihai commented 2022-11-23 22:32:30 +01:00
Owner
Copy Link

Le fichier "alpes-maritimes-2022/wcs/tracking_code_input.html" n'est pas en trop? surtout qu'il hérite de "combo/wcs/tracking_code_input.html" ?

Le code JS de "formdata_steps.html" peut aller dans le fichier "extra.js" qui sera pris en compte lors de la création de themes.json (make themes.json).

La modification de "toulouse-2022/_vars.scss" me semble sans rapport avec le thème.

Dans config.json la variable theme_color comporte d'habitude la valeur du $primary-color et ainsi on se fait une idée à quelle couleur s'attendre dans le thème. Dans ton cas elle devrait avoir la valeur de $cyan-dark.

Le fichier "alpes-maritimes-2022/wcs/tracking_code_input.html" n'est pas en trop? surtout qu'il hérite de "combo/wcs/tracking_code_input.html" ? Le code JS de "formdata_steps.html" peut aller dans le fichier "extra.js" qui sera pris en compte lors de la création de themes.json (*make themes.json*). La modification de "toulouse-2022/_vars.scss" me semble sans rapport avec le thème. Dans *config.json* la variable *theme_color* comporte d'habitude la valeur du *$primary-color* et ainsi on se fait une idée à quelle couleur s'attendre dans le thème. Dans ton cas elle devrait avoir la valeur de *$cyan-dark*.
tjund commented 2022-11-25 09:57:48 +01:00
Author
Owner
Copy Link

Le fichier "alpes-maritimes-2022/wcs/tracking_code_input.html" n'est pas en trop? surtout qu'il hérite de "combo/wcs/tracking_code_input.html" ?

Merci, supprimé

Le code JS de "formdata_steps.html" peut aller dans le fichier "extra.js" qui sera pris en compte lors de la création de themes.json (make themes.json).

Pourquoi j'ajouterais un code necessaire uniquement pour les pages démarches à toutes les pages du portail ?

La modification de "toulouse-2022/_vars.scss" me semble sans rapport avec le thème.

Merci

Dans config.json la variable theme_color comporte d'habitude la valeur du $primary-color et ainsi on se fait une idée à quelle couleur s'attendre dans le thème. Dans ton cas elle devrait avoir la valeur de $cyan-dark.

Ok, mais {{theme_color}} est utilisé dans les templates de mail. Et dans la maquette reçu, c'est $cyan qui est utilisé pour les mails.
Mais si tu préfères, je change pour $primary-color: $cyan et remplace toutes le occurances de $primary-color par cyan-dark.

> Le fichier "alpes-maritimes-2022/wcs/tracking_code_input.html" n'est pas en trop? surtout qu'il hérite de "combo/wcs/tracking_code_input.html" ? Merci, supprimé > Le code JS de "formdata_steps.html" peut aller dans le fichier "extra.js" qui sera pris en compte lors de la création de themes.json (make themes.json). Pourquoi j'ajouterais un code necessaire uniquement pour les pages démarches à toutes les pages du portail ? > La modification de "toulouse-2022/_vars.scss" me semble sans rapport avec le thème. Merci > Dans config.json la variable theme_color comporte d'habitude la valeur du $primary-color et ainsi on se fait une idée à quelle couleur s'attendre dans le thème. Dans ton cas elle devrait avoir la valeur de $cyan-dark. Ok, mais `{{theme_color}}` est utilisé dans les templates de mail. Et dans la maquette reçu, c'est $cyan qui est utilisé pour les mails. Mais si tu préfères, je change pour `$primary-color: $cyan` et remplace toutes le occurances de `$primary-color` par `cyan-dark`.
tjund force-pushed wip/70929-cd06-2022 from 9e9ef11c2c to 3c560a07c6 2022-11-25 09:58:37 +01:00 Compare
fpeters commented 2022-11-25 10:04:55 +01:00
Owner
Copy Link

Pourquoi j'ajouterais un code necessaire uniquement pour les pages démarches à toutes les pages du portail ?

Parce qu'ainsi le code est téléchargé une fois plutôt qu'à chaque fois qu'une page de démarche est affichée.

(pour le coup ici ça n'est pas significatif et peu importe, mais c'est une raison)

> Pourquoi j'ajouterais un code necessaire uniquement pour les pages démarches à toutes les pages du portail ? Parce qu'ainsi le code est téléchargé une fois plutôt qu'à chaque fois qu'une page de démarche est affichée. (pour le coup ici ça n'est pas significatif et peu importe, mais c'est une raison)
tjund commented 2022-11-25 10:26:36 +01:00
Author
Owner
Copy Link

Parce qu'ainsi le code est téléchargé une fois plutôt qu'à chaque fois qu'une page de démarche est affichée.

Bonne raison, avec la contrepartie de scanner le dom sur

> Parce qu'ainsi le code est téléchargé une fois plutôt qu'à chaque fois qu'une page de démarche est affichée. Bonne raison, avec la contrepartie de scanner le dom sur
tjund closed this pull request 2022-11-25 10:26:36 +01:00
tjund reopened this pull request 2022-11-25 10:27:47 +01:00
smihai commented 2022-11-25 11:52:05 +01:00
Owner
Copy Link

Ok, mais {{theme_color}} est utilisé dans les templates de mail. Et dans la maquette reçu, c'est $cyan qui est utilisé pour les mails.
Mais si tu préfères, je change pour $primary-color: $cyan et remplace toutes le occurances de $primary-color par cyan-dark.

Je trouve que cela sera plus lisible, mais fais comme tu le sens.

> > Ok, mais `{{theme_color}}` est utilisé dans les templates de mail. Et dans la maquette reçu, c'est $cyan qui est utilisé pour les mails. > Mais si tu préfères, je change pour `$primary-color: $cyan` et remplace toutes le occurances de `$primary-color` par `cyan-dark`. Je trouve que cela sera plus lisible, mais fais comme tu le sens.
fpeters force-pushed wip/70929-cd06-2022 from 3c560a07c6 to c9e4141f33 2022-11-26 12:41:54 +01:00 Compare
fpeters changed target branch from main to wip/71572-vertical-steps-mixins 2022-11-26 12:42:34 +01:00
smihai approved these changes 2022-11-28 09:59:32 +01:00
smihai left a comment
Owner
Copy Link

Go

Go
tjund merged commit c9e4141f33 into wip/71572-vertical-steps-mixins 2022-11-28 14:50:41 +01:00
tjund deleted branch wip/70929-cd06-2022 2022-11-28 14:50:41 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
smihai
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#34
No description provided.
Delete Branch "wip/70929-cd06-2022"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?