Project

General

Profile

Bug #36003

Les flèches pour se déplacer restent actives même

Added by Marie Kuntz 4 months ago. Updated 4 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Les flèches à gauche et à droite des créneaux restent actives même si le défilement n'est plus possible, c'est source de confusion

fleches.png View - flèche active et cliquable (10 KB) Marie Kuntz, 10 Sep 2019 03:52 PM

0001-templates-disable-previous-next-meetings-buttons-on-.patch View (1.06 KB) Frédéric Péters, 17 Sep 2019 08:17 AM

37184

Associated revisions

Revision 69c826c1 (diff)
Added by Frédéric Péters 4 months ago

templates: disable previous/next meetings buttons at edges (#36003)

History

#1 Updated by Serghei Mihai 4 months ago

Il s'agit du rendu dans wcs, non?

#2 Updated by Frédéric Péters 4 months ago

  • Project changed from Chrono to Intégrations graphiques Publik

#3 Updated by Frédéric Péters 4 months ago

#4 Updated by Thomas Noël 4 months ago

Je ne suis pas assez à l'aise avec le code, mais j'ai l'impression que c'est inversé, non ? Il faut mettre le disabled à null quand is:visible est vrai et pas l'inverse ?

    if (go_prev.next().is(':visible')) {
      go_prev.prop('disabled', true);      <-- plutôt null ?
    } else {
      go_prev.prop('disabled', null);      <-- plutôt true ?
    }

#5 Updated by Frédéric Péters 4 months ago

Non, quand la première colonne d'heures est visible (c'est ça qui est ddans go_prev.next()), on veut désactiver le bouton "précédent".

#6 Updated by Thomas Noël 4 months ago

  • Status changed from Solution proposée to Solution validée

Bien bien... Je ack pour la forme, le fond m'échappe.

#7 Updated by Frédéric Péters 4 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 69c826c168aefe6bc81da14dcbee564dcd4f900f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Sep 17 08:16:25 2019 +0200

    templates: disable previous/next meetings buttons at edges (#36003)

#8 Updated by Frédéric Péters 4 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF