Produits Entr'ouvert » w.c.s.

(la branche en erreur)

Pour les liens de pagination on fait aussi < et > donc je dirais que ça passe ici aussi (côté combo on a pour la navigation entre pages des icônes via FontAwesome mais il n'y existe pas de << ou |< pour le retour au premier).

Je garderais cependant tout le temps les 4 boutons présents, en marquant comme désactivé ceux ne menant pas sur une version différente, ce genre :

         if snapshot.id != snapshot.first:
-            r += htmltext(' <a href="../../%s/view/">&Lt;</a>') % (snapshot.first)
+            r += htmltext(' <a class="button" href="../../%s/view/">&Lt;</a>') % (snapshot.first)
+        else:
+            r += htmltext(' <a class="button disabled" href="../../%s/view/">&Lt;</a>') % (snapshot.first)

(mais c'est peut-être lisible de faire tenir sur une ligne, note que tu peux mettre le % à l'intérieur de l'htmltext vu qu'on sait que la variable est safe)

(et je me rends compte que le style n'existe pas → j'ai créé et mis un patch dans #50302)

Je serais aussi pour centrer les boutons,

     if snapshot.previous or snapshot.next:
-        r += htmltext('<p>')
+        r += htmltext('<p class="snapshots-navigation">')

et

+++ b/wcs/qommon/static/css/dc2/admin.scss

+p.snapshots-navigation {
+       text-align: center;
+}

        if snapshot.id != snapshot.first:
            r += htmltext(' <a class="button" href="../../%s/view/">&Lt;</a>' % (snapshot.first))
            r += htmltext(' <a class="button" href="../../%s/view/">&LT;</a>' % (snapshot.previous))
        else:
            r += htmltext(' <a class="button disabled" href="../../%s/view/">&Lt;</a>' % (snapshot.first))
            r += htmltext(' <a class="button disabled" href="../../%s/view/">&LT;</a>' % (snapshot.first))

Ça m'a l'air bizarre.

Ce que j'imaginais :

@@ -66,11 +66,19 @@ def snapshot_info_block(snapshot):
         r += htmltext('<p>')
         if snapshot.id != snapshot.first:
             r += htmltext(' <a href="../../%s/view/">&Lt;</a>') % (snapshot.first)
+        else:
+            r += htmltext(' <a href="../../%s/view/" class="disabled">&Lt;</a>') % (snapshot.first)
         if snapshot.previous:
             r += htmltext(' <a href="../../%s/view/">&LT;</a>') % (snapshot.previous)
+        else:
+            r += htmltext(' <a href="../../%s/view/" class="disabled">&LT;</a>') % (snapshot.previous)
         if snapshot.next:
             r += htmltext(' <a href="../../%s/view/">&GT;</a>') % (snapshot.next)
+        else:
+            r += htmltext(' <a href="../../%s/view/" class="disabled">&GT;</a>') % (snapshot.next)
         if snapshot.id != snapshot.last:
             r += htmltext(' <a href="../../%s/view/">&Gt;</a>') % (snapshot.last)
+        else:
+            r += htmltext(' <a href="../../%s/view/" class="disabled">&Gt;</a>') % (snapshot.last)
         r += htmltext('</p>')
     return r.getvalue()

(et j'ai tenté de faire tenir sur une ligne mais j'ai trouvé ça moins lisible,

+        r += htmltext(' <a href="../../%s/view/" %s>&Lt;</a>' % (
+            snapshot.first, 'class="disabled"' if not snapshot.first else ''))
</pe>

En fait c'est pareil: si snapshot.id == snapshot.first, alors snapshot.previous vaut None
de même pour last et next

De plus:

if snapshot.previous:
    r += htmltext(' <a href="../../%s/view/">&LT;</a>') % (snapshot.previous)
else:
    r += htmltext(' <a href="../../%s/view/" class="disabled">&LT;</a>') % (snapshot.previous)

Ok je trouve ça étonnamment compliqué :/

S'il y a un snapshot précédent, il y a nécessairement un snapshot.first, et donc il ne correspondra pas au snapshot actuel, et donc il y aura toujours first et previous actifs.

S'il n'y a pas de snapshot précédent, c'est qu'on est nécessairement sur le premier snapshot, et donc il y aura toujours first et previous inactifs.

Peut-être dans les else remplir les href avec "#" pour éviter cette impression d'erreur dans la répétition, et ajouter un commentaire type "# currently browsing the first snapshot, display links as disabled" ?