Development #59873
ne pas marquer cliquables par défaut les lignes de tableau
0%
Description
Actuellement on change le curseur au survol des lignes, et ça donne l'impression qu'on peut cliquer et ouvrir la ligne.
Fichiers
Demandes liées
Révisions associées
general: allow text selection in clickable-rows (#59873)
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-general-add-new-clickable-rows-class-for-rows-that-s.patch 0001-general-add-new-clickable-rows-class-for-rows-that-s.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ajout d'une classe clickable-rows pour avoir le changement de curseur, + incorporation du code de w.c.s. pour qu'un clic sur la ligne soit ~équivalent à un clic sur un lien de la ligne. (code tiré de wcs.listings.js).
C'est fait dans l'immédiat pour être utilisé dans combo,
--- a/combo/apps/wcs/templates/combo/wcs/care_forms.html +++ b/combo/apps/wcs/templates/combo/wcs/care_forms.html @@ -3,7 +3,7 @@ {% for slug, forms in care_forms.items %} <h2>{% trans "Forms to process" %} - {{ forms.title }}</h2> {% if forms.data %} - <table id="listing" class="main"> + <table id="listing" class="main clickable-rows">
mais ça pourra servir ailleurs j'imagine.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
On peut effectivement cliquer et ouvrir la ligne dans authentic, donc ce patch casse un comportement attendu là bas (et dans authentic le changement des curseur est désactivé explicitement quand on en a pas besoin).
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 2 ans
Si l'objectif est de rendre cliquable la ligne d'un tableau lorsqu'un lien s'y trouve, vraiment pas besoin de JS pour ça.
Quelque chose comme
table.main.clickable-rows { tr { position relative } tr a:before { content: ""; display: block; position: absolute; top: 0; right: 0; bottom: 0; left: 0; } }
devrait faire le job
Mis à jour par Frédéric Péters il y a plus de 2 ans
On peut effectivement cliquer et ouvrir la ligne dans authentic, donc ce patch casse un comportement attendu là bas
ok côté authentic il y aurait donc à anticiper la disparition du "cursor: pointer".
devrait faire le job
Presque, ça casse le rendu :hover des liens. À l'usage ça empêche aussi la sélection de texte ce qui n'est pas top.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 2 ans
Presque, ça casse le rendu :hover des liens. À l'usage ça empêche aussi la sélection de texte ce qui n'est pas top.
Ok (j'ai pas regardé + que le code du patch). Mais le rendu des :hover peut p-e être adapté au fait que toute la tr soit cliquable, et la selection du texte n'est p-e pas vraiment possible non plus avec le JS à cause de l'action lancé en mouseup : on selectionne le texte et pouf la page est redirgée : potentiellmeent encore + frustrant que de ne pas pouvoir selectionner.
Mais encore une fois, je n'ai pas tester.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
Thomas Jund a écrit :
Ok (j'ai pas regardé + que le code du patch). Mais le rendu des :hover peut p-e être adapté au fait que toute la tr soit cliquable, et la selection du texte n'est p-e pas vraiment possible non plus avec le JS à cause de l'action lancé en mouseup : on selectionne le texte et pouf la page est redirgée : potentiellmeent encore + frustrant que de ne pas pouvoir selectionner.
Mais encore une fois, je n'ai pas tester.
Oui je confirme que la sélection ne marche pas vraiment (ni dans a2 ni dans w.c.s. je ne sais pas si c'est le même code maintenant, je suppose que non à la lecture du ticket) et que c'est super chiant, tu sélectionnes et pim ça charge le lien. L'ergonomie pour les agents qui ne font que cliquer me parait quand même être la priorité (mais ce serait bien d'avoir une solution pour les deux en même temps, dans un monde idéal).
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Lié à Bug #59959: pouvoir sélectionner à la souris du texte des tableaux de traitement ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Thomas Jund a écrit :
Mais encore une fois, je n'ai pas tester.
Vu sur #59959, on peut avoir un code JS qui ne bloque pas la sélection, +1 pour le JS par rapport au CSS. Il faudrait adapté le code ici de la même manière, j'ouvre un ticket a2 pour migrer vers ce code.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Lié à Development #59991: manage: migrer vers .clickable-rows (gadjo) pour le clic sur les lignes de tableaux ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 68e236a120512e9e2dbd5a79a304f16520eaacbc Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Dec 17 18:31:11 2021 +0100 general: add new clickable-rows class for rows that should be clickable (#59873)
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
J'ai poussé pour pouvoir avancé coté Toulouse, je suppose qu'il faut aussi un patch dans combo.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Assigné à changé de Frédéric Péters à Benjamin Dauvergne
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Fichier 0001-general-allow-text-selection-in-clickable-rows-59873.patch 0001-general-allow-text-selection-in-clickable-rows-59873.patch ajouté
- Statut changé de Résolu (à déployer) à Solution proposée
Pas vu que le changement dans #59959 n'était pas répercuté ici.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Lié à Development #60309: utiliser la nouvelle classe .clickable-rows dans les tableaux ajouté
Mis à jour par Thomas Noël il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Je valide parce que j'ai lu #59959
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit bfdbab4bcab6063e5bc77f570fc3506d13a51a36 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Wed Jan 5 17:49:34 2022 +0100 general: allow text selection in clickable-rows (#59873)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
general: add new clickable-rows class for rows that should be clickable (#59873)