wcs: add id to cell body div (#83250) #191

bdauvergne · 2023-11-08T21:31:14+01:00

bdauvergne commented

2023-11-08 21:31:14 +01:00

No description provided.

bdauvergne added 1 commit 2023-11-08 21:31:14 +01:00

gitea/combo/pipeline/head This commit looks good Details

d5a4825548 wcs: add id to cell body div (#83250 )

fpeters commented

2023-11-09 08:38:27 +01:00

Dans mes tests, ça ne fonctionne pas systématiquement; j'ai imaginé que ça pouvait échouer quand la cellule était chargée de manière asynchrone mais ça ne semble pas ça. (cf fichier attaché)

Aussi, quand ça fonctionne, ou sur une page déjà chargée, ça met à hauteur du contenu alors que souvent il me semble on préférerait que ça se place au niveau du titre.

Dans mes tests, ça ne fonctionne pas systématiquement; j'ai imaginé que ça pouvait échouer quand la cellule était chargée de manière asynchrone mais ça ne semble pas ça. (cf fichier attaché) Aussi, quand ça fonctionne, ou sur une page déjà chargée, ça met à hauteur du contenu alors que souvent il me semble on préférerait que ça se place au niveau du titre.

out.ogv

5.9 MiB

bdauvergne commented

2023-11-13 10:24:41 +01:00

Dans mes tests, ça ne fonctionne pas systématiquement; j'ai imaginé que ça pouvait échouer quand la cellule était chargée de manière asynchrone mais ça ne semble pas ça. (cf fichier attaché)

La vidéo est un peu rapide, mais de ce que je comprends des premiers tests pour parking-5, sur une même page et un même fragment des fois ça marche et des fois ça ne marche pas, bien que ce soit chargé de manière asynchrone la première fois et que les suivantes non car venant du cache ?

Aussi, quand ça fonctionne, ou sur une page déjà chargée, ça met à hauteur du contenu alors que souvent il me semble on préférerait que ça se place au niveau du titre.

Oui tu as raison ce serait mieux sur le titre.

> Dans mes tests, ça ne fonctionne pas systématiquement; j'ai imaginé que ça pouvait échouer quand la cellule était chargée de manière asynchrone mais ça ne semble pas ça. (cf fichier attaché) La vidéo est un peu rapide, mais de ce que je comprends des premiers tests pour parking-5, sur une même page et un même fragment des fois ça marche et des fois ça ne marche pas, bien que ce soit chargé de manière asynchrone la première fois et que les suivantes non car venant du cache ? > Aussi, quand ça fonctionne, ou sur une page déjà chargée, ça met à hauteur du contenu alors que souvent il me semble on préférerait que ça se place au niveau du titre. Oui tu as raison ce serait mieux sur le titre.

fpeters commented

2023-11-13 10:27:20 +01:00

Dans mes tests, ça ne fonctionne pas systématiquement; j'ai imaginé que ça pouvait échouer quand la cellule était chargée de manière asynchrone mais ça ne semble pas ça. (cf fichier attaché)

La vidéo est un peu rapide, mais de ce que je comprends des premiers tests pour parking-5, sur une même page et un même fragment des fois ça marche et des fois ça ne marche pas, bien que ce soit chargé de manière asynchrone la première fois et que les suivantes non car venant du cache ?

Oui asynchrone ou pas parfois ça marche parfois pas.

> > Dans mes tests, ça ne fonctionne pas systématiquement; j'ai imaginé que ça pouvait échouer quand la cellule était chargée de manière asynchrone mais ça ne semble pas ça. (cf fichier attaché) > > La vidéo est un peu rapide, mais de ce que je comprends des premiers tests pour parking-5, sur une même page et un même fragment des fois ça marche et des fois ça ne marche pas, bien que ce soit chargé de manière asynchrone la première fois et que les suivantes non car venant du cache ? Oui asynchrone ou pas parfois ça marche parfois pas.

bdauvergne force-pushed wip/83250-Avoir-un-id-stable-sur-les-cellu from d5a4825548 to b799d77b2e

2023-11-23 02:55:49 +01:00

Compare

bdauvergne commented

2023-11-23 02:57:18 +01:00

J'ai implémenté le focus sur le #hash en JS, je n'ai pas trouvé plus simple. Je n'ai pas réussi à faire un chargement synchrone des fiches (le cache à 5 secondes ne semble pas fonctionner ou je lis mal le code).

--- a/combo/public/static/js/combo.public.js
+++ b/combo/public/static/js/combo.public.js
@@ -43,6 +43,10 @@ function combo_load_cell(elem) {
               } else {
                 $elem.removeClass('empty-cell')
               }
+              // Scroll to currently selected anchor
+              if (window.location.hash) {
+                $('html').scrollTop($(window.location.hash).offset().top);
+              }

J'ai implémenté le focus sur le #hash en JS, je n'ai pas trouvé plus simple. Je n'ai pas réussi à faire un chargement synchrone des fiches (le cache à 5 secondes ne semble pas fonctionner ou je lis mal le code). ``` --- a/combo/public/static/js/combo.public.js +++ b/combo/public/static/js/combo.public.js @@ -43,6 +43,10 @@ function combo_load_cell(elem) { } else { $elem.removeClass('empty-cell') } + // Scroll to currently selected anchor + if (window.location.hash) { + $('html').scrollTop($(window.location.hash).offset().top); + } ```

bdauvergne force-pushed wip/83250-Avoir-un-id-stable-sur-les-cellu from b799d77b2e to 0d6796ecc9

2023-11-23 10:09:25 +01:00

Compare

bdauvergne force-pushed wip/83250-Avoir-un-id-stable-sur-les-cellu from 0d6796ecc9 to 5df055bddf

2024-02-26 13:14:24 +01:00

Compare

gitea/combo/pipeline/head This commit looks good

Details

This pull request can be merged automatically.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 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/combo#191