Project

General

Profile

Development #26804

après un certain temps diminuer la fréquence de rafrachissement des tableaux

Added by Frédéric Péters 9 months ago. Updated 4 months ago.

Status:
Solution déployée
Priority:
Normal
Start date:
27 Sep 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Diminution à décider, l'idée étant que ça ne sert trop à rien de continuer à rafraichir l'écran d'agents qui ne sont visiblement pas devant leur ordi, qui l'ont laissé allumer pour la nuit.

0001-do-not-refresh-table-when-tab-is-invisible-or-user-i.patch View (2.59 KB) Benjamin Dauvergne, 28 Sep 2018 12:39 PM

0001-do-not-refresh-table-when-tab-is-invisible-or-user-i.patch View (2.6 KB) Benjamin Dauvergne, 26 Feb 2019 05:42 PM

0001-do-not-refresh-table-when-tab-is-invisible-or-user-i.patch View (2.96 KB) Benjamin Dauvergne, 04 Mar 2019 09:40 AM

0001-do-not-refresh-table-when-tab-is-invisible-or-user-i.patch View (1.77 KB) Benjamin Dauvergne, 04 Mar 2019 03:23 PM

Associated revisions

Revision 6c0f72fc (diff)
Added by Benjamin Dauvergne 4 months ago

backoffice: do not refresh table when user is idle (#26804)

History

#1 Updated by Benjamin Dauvergne 9 months ago

  • Assignee set to Benjamin Dauvergne

#2 Updated by Benjamin Dauvergne 9 months ago

J'en ai profité pour faire mumuse avec l'API de visibilité.

#3 Updated by Benjamin Dauvergne 7 months ago

Patch s'appliquant toujours sur master.

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

Particulièrement pour du js qui complexifie les choses, je ferais attention au nommage, et donc vis(), fan, plutôt on_visibility_change(); et peut-être reprendre/réunir les commentaires pour avoir une explication globale du comportement que ça amène.

 /* refresh every 30 seconds (idle_id)
  * watch for inactivity (longtime_id)
  * on inactivity, stop refreshing (clear idle_id)
  */

+ quelque chose sur la visibilité mais à écrire la description je me demande si c'est vraiment nécessaire d'aller taper du js bizarre pour ça, que ça pourrait juste rester sur le $(window).on('mousemove keydown mousedown touchstart'.

#6 Updated by Benjamin Dauvergne 4 months ago

Frédéric Péters a écrit :

Particulièrement pour du js qui complexifie les choses, je ferais attention au nommage, et donc vis(), fan, plutôt on_visibility_change(); et peut-être reprendre/réunir les commentaires pour avoir une explication globale du comportement que ça amène.

Ok.

+ quelque chose sur la visibilité mais à écrire la description je me demande si c'est vraiment nécessaire d'aller taper du js bizarre pour ça, que ça pourrait juste rester sur le $(window).on('mousemove keydown mousedown touchstart'.

Tu veux dire ne pas utiliser l'API de visibilité des onglets ?

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

Tu veux dire ne pas utiliser l'API de visibilité des onglets ?

Oui, parce que je trouve que ça ajouter du js que personne n'osera toucher, et qu'on a assez à juste suivre l'(in)activité.

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

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

Ok avec mise à jour du message git pour retirer les références à l'invisibilité.

#11 Updated by Benjamin Dauvergne 4 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 6c0f72fc6b214079f347ae9f74c38a45c2f7c6bb (HEAD -> master, origin/master, origin/HEAD)
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri Sep 28 12:36:36 2018 +0200

    backoffice: do not refresh table when user is idle (#26804)

#12 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