Bug #23977
Le nom de la demande et son statut se surimprime dans les demandes en cours
0%
Description
apparemment valable dans tous les thèmes quand on a une largeur d'écran réduite mais juste au dessus du seuil de pixels pour téléphones
Fichiers
Révisions associées
nancy-2017: adjust cell list item style (broken by #23977)
Historique
Mis à jour par Anonyme il y a presque 6 ans
- Fichier 0001-cells-avoid-list-of-forms-overlapping-contents-23977.patch 0001-cells-avoid-list-of-forms-overlapping-contents-23977.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
- Assigné à mis à Anonyme
une idée à tester plus à fond de ce que ça pourrait donner pour éviter les problèmes liés au chevauchements dans les différents ".list-of-forms"
Mis à jour par Anonyme il y a presque 6 ans
- Fichier 0001-cells-avoid-lists-of-forms-contents-to-overlap-23977.patch 0001-cells-avoid-lists-of-forms-contents-to-overlap-23977.patch ajouté
- Fichier Screenshot-2018-5-22 Hautes-Alpes le département - Accueil(3).png Screenshot-2018-5-22 Hautes-Alpes le département - Accueil(3).png ajouté
- Fichier Screenshot-2018-5-22 Hautes-Alpes le département - Accueil(2).png Screenshot-2018-5-22 Hautes-Alpes le département - Accueil(2).png ajouté
- Fichier Screenshot-2018-5-22 Hautes-Alpes le département - Accueil(1).png Screenshot-2018-5-22 Hautes-Alpes le département - Accueil(1).png ajouté
- Fichier Screenshot-2018-5-22 Hautes-Alpes le département - Accueil.png Screenshot-2018-5-22 Hautes-Alpes le département - Accueil.png ajouté
- Patch proposed changé de Non à Oui
C'est toujours un WIP à discuter
Je ne suis pas sûr de la pertinence globale de la proposition d'amélioration.
D'autre part, j'ai reformaté le code et apprté une correction. Les captures témoignent des différents états de l'affichage proposé par ce patch.
Mis à jour par Frédéric Péters il y a presque 6 ans
Par jabber je notais :
et on peut essayer d'éviter les déplacements de code pour faciliter la relecture.
c'est-à-dire ici garder les niveaux actuels, le span.form-number avant le span.form-status, etc.
comme ça on voit immédiatement qu'il n'y a pas de modification au font-size ou opacity.
et je sais qu'il est écrit "essayer" et que ça pourrait donner l'idée que c'est optionnel.
Mis à jour par Frédéric Péters il y a presque 6 ans
- Fichier avant2.png avant2.png ajouté
- Fichier apres2.png apres2.png ajouté
- Fichier apres1.png apres1.png ajouté
- Fichier avant1.png avant1.png ajouté
Testé en local, il y a au moins deux situations où l'affichage se dégrade; a priori dégager la partie space-around améliorerait ça. (je ne sais pas pourquoi elle a été ajoutée)
Resterait quand même encore derrière une marge à gauche de la date, quand elle passe entièrement à la ligne, et pour contrer ça c'est sans doute mieux d'avoir une marge droite ajoutée au libellé (ce qui veut dire le mettre dans son propre span).
Plutôt discret mais quand même, ça pert l'alignement "base line" entre le texte et le numéro/date (qui est désormais posé plus bas).
Mis à jour par Frédéric Péters il y a presque 6 ans
Testé en local, il y a au moins deux situations où l'affichage se dégrade; a priori dégager la partie space-around améliorerait ça. (je ne sais pas pourquoi elle a été ajoutée)
À propos du space-around, il fait aussi que le texte est "centré" sur les cellules type "démarches d'une catégorie".
Mis à jour par Anonyme il y a presque 6 ans
- Fichier 0001-cells-avoid-lists-of-forms-contents-to-overlap-23977.patch 0001-cells-avoid-lists-of-forms-contents-to-overlap-23977.patch ajouté
- Fichier 0001-list-of-forms-wrap-the-title-into-a-span-23977.patch 0001-list-of-forms-wrap-the-title-into-a-span-23977.patch ajouté
- Fichier Screenshot-2018-5-22 Hautes-Alpes le département - Accueil.png Screenshot-2018-5-22 Hautes-Alpes le département - Accueil.png ajouté
- Fichier Capture d’écran 2018-05-22 à 18.11.31-fullpage.png Capture d’écran 2018-05-22 à 18.11.31-fullpage.png ajouté
- Fichier Capture d’écran 2018-05-22 à 18.11.15-fullpage.png Capture d’écran 2018-05-22 à 18.11.15-fullpage.png ajouté
- Fichier Capture d’écran 2018-05-22 à 18.11.10-fullpage.png Capture d’écran 2018-05-22 à 18.11.10-fullpage.png ajouté
Voilà une solution aux dernière remarques, avec modification de 2 templates dans combo (2e patch).
Serghei m'a expliqué les précautions à prendre si ce ticket est validé : pour la synchro des release entre combo et publik-base-theme, il faudra attendre le prochain cycle.
Mis à jour par Serghei Mihai il y a presque 6 ans
- Fichier long_form_titles.png long_form_titles.png ajouté
Tu as rajouté des mixins pour faire des flexbox, il faudrait s'en servir.
Si tu passes en flexbox
, il faut virer le display: block
sur les li > a
Les titres longs de formulaires croisent le statut et c'est illisible (j'ai testé au hasard sur le thème Orléans).
Mis à jour par Frédéric Péters il y a presque 6 ans
Tu as rajouté des mixins pour faire des flexbox, il faudrait s'en servir.
J'ai accepté ça à l'extérieur mais je trouve vraiment plus lisible de s'en passer.
Mis à jour par Anonyme il y a presque 6 ans
- Fichier Capture d’écran 2018-05-23 à 10.15.22-fullpage.png Capture d’écran 2018-05-23 à 10.15.22-fullpage.png ajouté
Serghei Mihai a écrit :
Tu as rajouté des mixins pour faire des flexbox, il faudrait s'en servir.
Si tu passes en
flexbox
, il faut virer ledisplay: block
sur lesli > a
Je ne vois pas à quel display
tu fais référence dans le dernier patch https://dev.entrouvert.org/attachments/24813/0001-cells-avoid-lists-of-forms-contents-to-overlap-23977.patch
Les titres longs de formulaires croisent le statut et c'est illisible (j'ai testé au hasard sur le thème Orléans).
Idem, je ne vois pas de soucis, même avec le thème d'Orléans et une phrase de lorem ipsum en titre
Mis à jour par Serghei Mihai il y a presque 6 ans
Frédéric Péters a écrit :
J'ai accepté ça à l'extérieur mais je trouve vraiment plus lisible de s'en passer.
Mais alors il faut utiliser vendor-prefix
, non?
Mis à jour par Serghei Mihai il y a presque 6 ans
Elias Showk a écrit :
Idem, je ne vois pas de soucis, même avec le thème d'Orléans et une phrase de lorem ipsum en titre
Au temps pour moi. Ça fonctionne.
Mis à jour par Frédéric Péters il y a presque 6 ans
Mais alors il faut utiliser vendor-prefix, non?
Non ma position sur les flexbox est d'écrire les propriétés natives + la version -ms, pas encombrer davantage.
Mis à jour par Anonyme il y a presque 6 ans
Frédéric Péters a écrit :
Mais alors il faut utiliser vendor-prefix, non?
Non ma position sur les flexbox est d'écrire les propriétés natives + la version -ms, pas encombrer davantage.
C'est ce que j'ai fait dans mon dernier patch
Mis à jour par Frédéric Péters il y a presque 6 ans
(Oui je répondais à Serghei qui suggérait autre chose)
Mis à jour par Frédéric Péters il y a presque 6 ans
- Fichier Screenshot-2018-5-30 Publik local de Fred - démarches de l_usager.png Screenshot-2018-5-30 Publik local de Fred - démarches de l_usager.png ajouté
Les textes ne sont pas alignés non plus avec ce dernier patch; cf capture, le numéro se trouve un pixel trop bas.
Mis à jour par Frédéric Péters il y a presque 6 ans
Les textes ne sont pas alignés non plus avec ce dernier patch; cf capture, le numéro se trouve un pixel trop bas.
Je me suis en fait trouvé à réappliquer le patch initial plutôt que le patch corrigé :/ Là le problème est effectivement corrigé.
Mis à jour par Frédéric Péters il y a presque 6 ans
Sur une cellule affichant une liste de démarches, limitées à un certain nombre, le "+" qui sert à étendre la vue était centré, ne l'est plus.
--- a/static/includes/_cells.scss +++ b/static/includes/_cells.scss @@ -182,6 +182,7 @@ div.wcscurrentformscell ul { padding: 0.5rem 1rem; cursor: pointer; text-align: center; + display: block; } /* make sure first/last children don't overflow the cell radius */
Mis à jour par Frédéric Péters il y a presque 6 ans
Côté combo, la modification fait échouer un test,
tests/test_wcs.py::test_current_drafts_cell_render_logged_in FAILED [ 98%]
Mis à jour par Anonyme il y a presque 6 ans
- Fichier 0001-list-of-forms-wrap-the-title-into-a-span-23977.patch 0001-list-of-forms-wrap-the-title-into-a-span-23977.patch ajouté
- Fichier 0001-cells-avoid-lists-of-forms-contents-to-overlap-23977.patch 0001-cells-avoid-lists-of-forms-contents-to-overlap-23977.patch ajouté
Frédéric Péters a écrit :
Côté combo, la modification fait échouer un test,
- test corrigé, c'était à cause du re.findall
pour lister les URL des listes de démarches qui n'était pas adaptée à la présence du nouveau <span class="form-title">
- côté cells.scss : j'ai suivi la recommandation et le + est bien centré
Mis à jour par Frédéric Péters il y a presque 6 ans
- Statut changé de En cours à Résolu (à déployer)
J'ai arrangé un peu le message de commit côté combo et j'ai poussé les deux, merci.
commit 1dc4add1cf97d0e0366e3648d34dd05c76cb78af Author: Elias Showk <eshowk@entrouvert.com> Date: Tue May 22 18:13:46 2018 +0200 wcs: wrap title into a span in list of forms (#23977)
Mis à jour par Frédéric Péters il y a plus de 5 ans
Ça cassait le rendu à Nancy à cause des puces mises en ::before.
commit 690274e476643db2a217ff8e51ebda3c68cd2f55 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Jul 27 09:15:06 2018 +0200 nancy-2017: adjust cell list item style (broken by #23977)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
cells: avoid lists of forms contents to overlap (#23977)