https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342020-04-20T13:27:18ZRedmine Entr’ouvertw.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=2287262020-04-20T13:27:18ZSerghei Mihai
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-2 status-5 priority-4 priority-default closed" href="/issues/41720">Development #41720</a>: thème pour le département de la Dordogne</i> ajouté</li></ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=2287502020-04-20T13:57:25ZThomas Jundtjund@entrouvert.com
<ul></ul><p>Dans l'état, il faudrait déconseiller de mettre l'option `(grow)` sur le bouton `previous` qui n'est pas systématiquement présent (mise à jour de la doc).</p>
<p>Sinon, comme propositions :</p>
<ul>
<li>faire en sorte que previous soit toujours présent dans le code, mais avec un attribut `disabled` dessus (et son container du coup) et masquer ou non ce bouton à l'aide d'un `display:none`. Et si option (grow) est placé dessus, mettre à la place un visibility: hidden, </li>
<li>soit trouver une solution pur CSS que je n'entrevois pas encore.</li>
</ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=2287582020-04-20T14:09:14ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Il me semblait que de l'autre côté il y avait un contournement qui marchait ?</p> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=2288132020-04-20T15:08:04ZThomas Jundtjund@entrouvert.com
<ul></ul><blockquote>
<p>Il me semblait que de l'autre côté il y avait un contournement qui marchait ?</p>
</blockquote>
<p>Dans un cas spécifique, oui. Et c'était une mauvaise idée, conseiller la modification de $buttons-alignment aurait été préférable :</p>
<p>Je récapitule pour flex-start et flex-end, <br />si l'option (grow) est placé sur le bouton previous, quand il n'est pas présent, la position des boutons qui l'entourent peuvent être modifiés</p>
<p>1/ l'alignement des boutons est en `flex-start`</p>
<p>Quand le bouton previous est en 1ère ou 2e position</p>
<p>previous (grow), cancel, submit<br />ou<br />cancel, previous (grow), submit</p>
<p>Alors le⋅s bouton⋅s placés APRÈS `previous` perdent leur positionnement à droite quand previous n'est pas présent et repassent à gauche</p>
<p>1/ l'alignement des boutons est en `flex-end`</p>
<p>Alors c'est l'inverse<br />Quand le bouton previous est en 2e ou 3e position<br />Alors le⋅s bouton⋅s placés AVANT `previous` perdent leur positionnement à gauche quand previous n'est pas présent et passent à droite.</p>
<hr />
<p>Exemple 1. Previous position 1.<br />Dans le cas de la Dordogne où l'affichage doit être</p>
<p>previous --> cancel Submit</p>
<p>Il faut donc utiliser<br /><pre>
$buttons-order: previous, cancel (grow), submit;
$buttons-alignment: flex-end
</pre></p>
<p>ET pas<br /><pre>
$buttons-order: previous (grow), cancel, submit;
$buttons-alignment: flex-start // default
</pre></p>
<p>Exemple 2, previous position 2.<br />Dans le cas où on souhaite</p>
<p>cancel previous --> Submmit</p>
<p>Idem, La solution est d'aligner à droite et de faire un grow sur Submit</p>
<pre>
$buttons-order: cancel, previous, submit (grow);
$buttons-alignment: flex-end
</pre>
<hr />
<p>Après avoir détaillé tout ça, pour moi, déconseiller l'utilisation de l'option grow sur previous devrait suffire.<br />Par contre, que le bouton soit présent dans le code avec l'option 'visibity;:hidden' permettrait quand même de pouvoir 'conserver' l'emplacement du bouton dans d'autres configurations d'alignement (center, space-between) et rendrait les options de placement moins confus.</p> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=2288172020-04-20T15:18:36ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Je ne comprends toujours pas, mais c'est sans doute parce que ce ticket a été créé trop générique.</p>
<p>Du cas Dordogne, qui posait : $buttons-order: previous (grow), cancel, submit;</p>
<p>À ça on dit "dans ce cas précis il faut également poser telle ou telle option".</p>
<p>Et j'imaginais ce ticket comme étant mettre dans le code le "dans ce cas précis également poser telle ou telle option".</p>
<blockquote>
<p>(...) déconseiller l'utilisation de l'option grow sur previous devrait suffire</p>
</blockquote>
<p>En l'état elle ne produit pas, seule, un comportement convenable.</p>
<p>Est-ce qu'il y a des moments où elle produit un comportement qui serait souhaité ?</p>
<p>Si oui, mais en combinaison avec d'autres options, a-t-on de quoi les poser ?</p>
<p>Si non, a-t-on de quoi interdire le previous (grow) ?</p> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=2290762020-04-21T13:45:03ZThomas Jundtjund@entrouvert.com
<ul><li><strong>Sujet</strong> changé de <i>consérver l'ordre des boutons même si un des boutons n'est pas affché</i> à <i>conserver l'ordre des boutons même si un des boutons n'est pas affiché</i></li></ul><blockquote>
<p>Je ne comprends toujours pas<br />Du cas Dordogne, qui posait : $buttons-order: previous (grow), cancel, submit;<br />À ça on dit "dans ce cas précis il faut également poser telle ou telle option".</p>
</blockquote>
<p>Moi j'ai proposé (<a class="external" href="https://dev.entrouvert.org/issues/41720#note-5">https://dev.entrouvert.org/issues/41720#note-5</a>), pour conserver le même design, de déplacer l'option grow sur le bouton cancel et de modifier l'alignement. Exactement l'exemple 1 présenté au dessus.</p>
<p>Poser un flex-grow sur un flex-child qui disparaît est en effet une mauvaise idée.<br />Et pour me répéter encore, de mon point de vue :</p>
<ul>
<li>Soit on garde le code en l'état qui permet toutes les possibilités de layout mais on ne permet plus (on déconseille) l'utilisation de l'option grow sur previous. Ce qui oblige l'intégrateur à jouer avec les options d'alignement un peu plus souvent. Et on peu en effet éviter d'appliquer un flex-grow sur previous</li>
<li>Soit une solution du coté du code HTML : ne pas supprimer 'previous' du code. Qui a largement ma préférence.</li>
</ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=3403792021-11-30T11:28:01ZThomas Jundtjund@entrouvert.com
<ul><li><strong>Tracker</strong> changé de <i>Support</i> à <i>Bug</i></li><li><strong>Sujet</strong> changé de <i>conserver l'ordre des boutons même si un des boutons n'est pas affiché</i> à <i>conserver le positionnement des boutons même si le bouton previous n'est pas affiché</i></li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>En cours</i></li><li><strong>Assigné à</strong> mis à <i>Thomas Jund</i></li></ul><p>re-tombé sur la fonction index() de sass sur la doc qui permet de récupérer l'index et repenser à cette problématique</p>
<p>1.<br /><pre>
@if (index($buttons-order, previous grow)) {
@error("$buttons-order : #{$buttons-order} / grow option is not allowed for previous button");
}
</pre></p>
<p>Avec ça on peut interdire la pose de l'option grow sur le bouton previous. Mais l'analyse de l'existant m'a permis de dégager 2 contournements possibles en récupérant l'index du bouton previous</p>
<pre>
$previous-is-grow: index($buttons-order, previous grow);
</pre>
<p>Si `$previous-is-grow 1` alors un `justify-content: flex-end` fait le job<br />et si `$previous-is-grow 2` il faut alors un `justify-content: space-between`.</p> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=3403832021-11-30T11:34:33ZThomas Jundtjund@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/59573">0001-scss-keep-correct-buttons-order-value-when-previous-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/59573/0001-scss-keep-correct-buttons-order-value-when-previous-.patch">0001-scss-keep-correct-buttons-order-value-when-previous-.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=3643652022-03-14T17:07:55ZNicolas Roche (absent jusqu'au 3 avril)
<ul></ul><p>Pour moi c'est bon, mais je trouve ça compliqué à tester (je sais pas comment déboguer du code scss).<br />Et surtout très spécifique.<br />Je pense comme toi qu'une solution du coté du code HTML serait plus adaptée.</p> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=3645732022-03-15T10:06:37ZThomas Jundtjund@entrouvert.com
<ul></ul><blockquote>
<p>je sais pas comment déboguer du code scss</p>
</blockquote>
<p>Dans le code SCSS tu peux poser des `@debug()`: <a class="external" href="https://sass-lang.com/documentation/at-rules/debug">https://sass-lang.com/documentation/at-rules/debug</a></p>
<blockquote>
<p>Je pense comme toi qu'une solution du coté du code HTML serait plus adaptée.</p>
</blockquote>
<p>N'hésite pas à proposer un patch côté WCS :)</p> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=4501262023-03-10T10:42:03ZRobot Gitea
<ul><li><strong>Assigné à</strong> changé de <i>Thomas Jund</i> à <i>Corentin Séchet</i></li></ul><p>Corentin Sechet (csechet) a ouvert une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/196">https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/196</a></li>
<li>Titre : scss: keep correct $buttons-order value when previous button is missing (<a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: conserver le positionnement des boutons même si le bouton previous n'est pas affiché (Fermé)" href="https://dev.entrouvert.org/issues/41889">#41889</a>)</li>
<li>Modifications : <a class="external" href="https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/196/files">https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/196/files</a></li>
</ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=4501312023-03-10T10:46:19ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Serghei Mihai (smihai) a approuvé une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/196">https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/196</a></li>
</ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=4501572023-03-10T11:33:01ZCorentin Séchet
<ul><li><strong>Projet</strong> changé de <i>Intégrations graphiques Publik</i> à <i>w.c.s.</i></li></ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=4501612023-03-10T11:37:18ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Solution proposée</i></li></ul><p>Corentin Sechet (csechet) a ouvert une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/wcs/pulls/163">https://git.entrouvert.org/entrouvert/wcs/pulls/163</a></li>
<li>Titre : forms: add hidden previous button on first form page (<a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: conserver le positionnement des boutons même si le bouton previous n'est pas affiché (Fermé)" href="https://dev.entrouvert.org/issues/41889">#41889</a>)</li>
<li>Modifications : <a class="external" href="https://git.entrouvert.org/entrouvert/wcs/pulls/163/files">https://git.entrouvert.org/entrouvert/wcs/pulls/163/files</a></li>
</ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=4523452023-03-20T13:02:30ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>En cours</i></li></ul><p>Frédéric Péters (fpeters) a relu et demandé des modifications sur une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/wcs/pulls/163">https://git.entrouvert.org/entrouvert/wcs/pulls/163</a></li>
</ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=4618112023-05-02T07:10:14ZRobot Gitea
<ul><li><strong>Tracker</strong> changé de <i>Bug</i> à <i>Development</i></li><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li></ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=4618122023-05-02T07:10:15ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Frédéric Péters (fpeters) a approuvé une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/wcs/pulls/163">https://git.entrouvert.org/entrouvert/wcs/pulls/163</a></li>
</ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=4620062023-05-02T12:16:06ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><p>Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/wcs/pulls/163">https://git.entrouvert.org/entrouvert/wcs/pulls/163</a></li>
<li>Titre : forms: add hidden previous button on first form page (<a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: conserver le positionnement des boutons même si le bouton previous n'est pas affiché (Fermé)" href="https://dev.entrouvert.org/issues/41889">#41889</a>)</li>
<li>Modifications : <a class="external" href="https://git.entrouvert.org/entrouvert/wcs/pulls/163/files">https://git.entrouvert.org/entrouvert/wcs/pulls/163/files</a></li>
</ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=4620762023-05-02T14:14:47ZTransition automatique
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul> w.c.s. - Development #41889: conserver le positionnement des boutons même si le bouton previous n'est pas affichéhttps://dev.entrouvert.org/issues/41889?journal_id=4763192023-07-02T02:42:01ZTransition automatique
<ul></ul><p>Automatic expiration</p>