Saumur: intégration graphique, retouches (#71873) #48

Merged
aberriot merged 4 commits from wip/71821-saumur-retouches into main 2022-11-30 17:16:43 +01:00
Owner

A relire commit par commit, pour plus de compréhension et en se référant au ticket 71821 pour les détails, mais en récapitulatif :

  • on règle un problème d'alignement des boutons du formulaire, qui doivent être alignés à droite
  • on rassemble visuellement les cellules des colonnes en une seule grosse cellule, comme demandé par le client (cf captures ci-dessous)
  • on règle un souci d'overlap sur mobile, en retirant le positionnement absolu des toplinks et en utilisant des inline-blocks
  • on rajoute un template spécifique de cellule liste de lien pour avoir un sélecteur de commune qui puisse rediriger vers la bonne page cf vidéo (ça renvoie des 404 parce que j'ai mis des liens invalides). Pour l'utiliser, il faut spécifier services-communes comme slug dans les paramètres avancés de la cellule liste de lien.

Cf aussi un export de ma page d'accueil pour qui voudrait tester en local et gagner du temps ;)

Regroupement visuels des cellules

Avant :

image

Après :

image

A relire commit par commit, pour plus de compréhension et en se référant au ticket 71821 pour les détails, mais en récapitulatif : - on règle un problème d'alignement des boutons du formulaire, qui doivent être alignés à droite - on rassemble visuellement les cellules des colonnes en une seule grosse cellule, comme demandé par le client (cf captures ci-dessous) - on règle un souci d'overlap sur mobile, en retirant le positionnement absolu des toplinks et en utilisant des inline-blocks - on rajoute un template spécifique de cellule liste de lien pour avoir un sélecteur de commune qui puisse rediriger vers la bonne page cf vidéo (ça renvoie des 404 parce que j'ai mis des liens invalides). Pour l'utiliser, il faut spécifier `services-communes` comme slug dans les paramètres avancés de la cellule liste de lien. Cf aussi un export de ma page d'accueil pour qui voudrait tester en local et gagner du temps ;) # Regroupement visuels des cellules Avant : ![image](/attachments/a403f2a4-9edf-4b66-8563-dadcb05ce403) Après : ![image](/attachments/14e5f0f5-cdcb-47d6-85d4-1e2673e812c5)
aberriot added 3 commits 2022-11-30 12:10:25 +01:00
aberriot requested review from csechet 2022-11-30 12:10:30 +01:00
Owner

Juste une remarque au passage, sans vouloir préempter la relecture à Corentin : je serais pour laisser dans les messages de commit seulement des numéros de ticket des projets publics de briques de Publik, pas ceux des tickets clients (c’est plus pratique pour nous, et pour d’éventuel·les contributeur·rices tier·ces, qui n’ont pas accès aux tickets clients).

Juste une remarque au passage, sans vouloir préempter la relecture à Corentin : je serais pour laisser dans les messages de commit seulement des numéros de ticket des projets publics de briques de Publik, pas ceux des tickets clients (c’est plus pratique pour nous, et pour d’éventuel·les contributeur·rices tier·ces, qui n’ont pas accès aux tickets clients).
Owner

Juste une remarque au passage, sans vouloir préempter la relecture à Corentin : je serais pour laisser dans les messages de commit seulement des numéros de ticket des projets publics de briques de Publik, pas ceux des tickets clients (c’est plus pratique pour nous, et pour d’éventuel·les contributeur·rices tier·ces, qui n’ont pas accès aux tickets clients).

(Disclaimer hors-sujet)
Et je vois à l’instant que ça se fait dans d’autres PR, pour ce projet en particulier.

J’avoue que j’ai zappé le moment où on a dit que gitea ça voulait dire plus besoin de ticket technique public de brique dans redmine, seulement du ticket privé client. Je serais pour laisser des numéros de tickets publics dans les messages de commit.

> Juste une remarque au passage, sans vouloir préempter la relecture à Corentin : je serais pour laisser dans les messages de commit seulement des numéros de ticket des projets publics de briques de Publik, pas ceux des tickets clients (c’est plus pratique pour nous, et pour d’éventuel·les contributeur·rices tier·ces, qui n’ont pas accès aux tickets clients). (Disclaimer hors-sujet) Et je vois à l’instant que ça se fait dans d’autres PR, pour ce projet en particulier. J’avoue que j’ai zappé le moment où on a dit que gitea ça voulait dire plus besoin de ticket technique public de brique dans redmine, seulement du ticket privé client. Je serais pour laisser des numéros de tickets publics dans les messages de commit.
Author
Owner

Juste une remarque au passage, sans vouloir préempter la relecture à Corentin : je serais pour laisser dans les messages de commit seulement des numéros de ticket des projets publics de briques de Publik, pas ceux des tickets clients (c’est plus pratique pour nous, et pour d’éventuel·les contributeur·rices tier·ces, qui n’ont pas accès aux tickets clients).

En pratique c'est l'enfer, parce que ça supposerait de créer 4 tickets techniques publics, un adossé à chaque ticket support, donc gérer le double de tickets. Sachant que ça concerne de toutes façon une intégration graphique qui n'intéresse personne à part nous et le client, ça me parait relativement futile comme exercice.

À mon avis il faudrait plutôt prendre le problème dans l'autre sens et sortir les intégrations des clients de pbt.

> Juste une remarque au passage, sans vouloir préempter la relecture à Corentin : je serais pour laisser dans les messages de commit seulement des numéros de ticket des projets publics de briques de Publik, pas ceux des tickets clients (c’est plus pratique pour nous, et pour d’éventuel·les contributeur·rices tier·ces, qui n’ont pas accès aux tickets clients). En pratique c'est l'enfer, parce que ça supposerait de créer 4 tickets techniques publics, un adossé à chaque ticket support, donc gérer le double de tickets. Sachant que ça concerne de toutes façon une intégration graphique qui n'intéresse personne à part nous et le client, ça me parait relativement futile comme exercice. À mon avis il faudrait plutôt prendre le problème dans l'autre sens et sortir les intégrations des clients de pbt.
Owner

En pratique c'est l'enfer, parce que ça supposerait de créer 4 tickets techniques publics, un adossé à chaque ticket support, donc gérer le double de tickets. Sachant que ça concerne de toutes façon une intégration graphique qui n'intéresse personne à part nous et le client, ça me parait relativement futile comme exercice.

Ok, les rares fois où j’ai dû faire ça, j’ai simplement créé un seul ticket pbt dans redmine en liant les différentes demandes du client. Cela fait un espace public où devs et CPFs (voire les clients) peuvent venir discuter du rendu global, ce qui reste je crois pertinent même avec gitea.

Et pas sûr que cet espace n’intéresse personne d’autre, il y a des clients qui gèrent leur propres dépôts de thème (iMio notamment), qui surveillent l’évolution de pbt et qui peuvent être intéressé·es par les discussions qui y ont eu lieu (genre comprendre à quelle fin tel template custom a été ajouté dans tel thème etc).

Mais j’entends le surcoût engendré, peut-être que c’est moi qui débloque. Mon précédent message venait surtout de ma surprise à ce changement de pratique un peu implicite, plutôt que d’une doléance style “c’était mieux avant.” :)

À mon avis il faudrait plutôt prendre le problème dans l'autre sens et sortir les intégrations des clients de pbt.

Elles iraient où ?

> En pratique c'est l'enfer, parce que ça supposerait de créer 4 tickets techniques publics, un adossé à chaque ticket support, donc gérer le double de tickets. Sachant que ça concerne de toutes façon une intégration graphique qui n'intéresse personne à part nous et le client, ça me parait relativement futile comme exercice. Ok, les rares fois où j’ai dû faire ça, j’ai simplement créé un seul ticket pbt dans redmine en liant les différentes demandes du client. Cela fait un espace public où devs et CPFs (voire les clients) peuvent venir discuter du rendu global, ce qui reste je crois pertinent même avec gitea. Et pas sûr que cet espace n’intéresse personne d’autre, il y a des clients qui gèrent leur propres dépôts de thème (iMio notamment), qui surveillent l’évolution de pbt et qui peuvent être intéressé·es par les discussions qui y ont eu lieu (genre comprendre à quelle fin tel template custom a été ajouté dans tel thème etc). Mais j’entends le surcoût engendré, peut-être que c’est moi qui débloque. Mon précédent message venait surtout de ma surprise à ce changement de pratique un peu implicite, plutôt que d’une doléance style “c’était mieux avant.” :) > À mon avis il faudrait plutôt prendre le problème dans l'autre sens et sortir les intégrations des clients de pbt. Elles iraient où ?
Owner

Elles iraient où ?

N'ayons pas cette conversation ici, svp.

> Elles iraient où ? N'ayons pas cette conversation ici, svp.
csechet requested changes 2022-11-30 15:50:44 +01:00
@ -206,0 +220,4 @@
// communes selector
div#services-communes {
background: $cell-background;
Owner

div#services-communes est déjà une cellule, pas besoin de définir le background ici.

div#services-communes est déjà une cellule, pas besoin de définir le background ici.
Author
Owner

Bien vu, je viens d'enlever

Bien vu, je viens d'enlever
aberriot marked this conversation as resolved
@ -0,0 +2,4 @@
<h2 class="commune--title">{{ title }}</h2>
<div class="cell--body">
<form>
<select id="commune--select-{{ cell.id }}" class="commune--select">
Owner

commune--title et commune--select ne sont pas utilisées dans le CSS (sur la version izi-ici de cette cellule, commune--select ne l'est pas non plus d'ailleurs)

commune--title et commune--select ne sont pas utilisées dans le CSS (sur la version izi-ici de cette cellule, commune--select ne l'est pas non plus d'ailleurs)
Author
Owner

Exact, merci c'est retiré 👍

Exact, merci c'est retiré 👍
aberriot marked this conversation as resolved
Owner

Et ça tombe à pic que je relise ce select comme liste de liens : je vais avoir besoin de quelque chose de similaire pour le CD28 :)

Et ça tombe à pic que je relise ce select comme liste de liens : je vais avoir besoin de quelque chose de similaire pour le CD28 :)
aberriot force-pushed wip/71821-saumur-retouches from 5bb88f3e0f to 31f12bb620 2022-11-30 16:23:22 +01:00 Compare
csechet approved these changes 2022-11-30 17:16:17 +01:00
aberriot merged commit f8b4ed979b into main 2022-11-30 17:16:43 +01:00
aberriot deleted branch wip/71821-saumur-retouches 2022-11-30 17:16:43 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
4 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#48
No description provided.