Projet

Général

Profil

Development #26804

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

Ajouté par Frédéric Péters il y a 6 mois. Mis à jour il y a 16 jours.

Statut:
Solution déployée
Priorité:
Normal
Assigné à:
Début:
27 sept. 2018
Echéance:
% réalisé:

0%

Patch proposed:
Oui

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 Voir (2,59 ko) Benjamin Dauvergne, 28 sept. 2018 12:39

0001-do-not-refresh-table-when-tab-is-invisible-or-user-i.patch Voir (2,6 ko) Benjamin Dauvergne, 26 fév. 2019 17:42

0001-do-not-refresh-table-when-tab-is-invisible-or-user-i.patch Voir (2,96 ko) Benjamin Dauvergne, 04 mar. 2019 09:40

0001-do-not-refresh-table-when-tab-is-invisible-or-user-i.patch Voir (1,77 ko) Benjamin Dauvergne, 04 mar. 2019 15:23

Révisions associées

Révision 6c0f72fc (diff)
Ajouté par Benjamin Dauvergne il y a 18 jours

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

Historique

#1 Mis à jour par Benjamin Dauvergne il y a 6 mois

  • Assigné à mis à Benjamin Dauvergne

#2 Mis à jour par Benjamin Dauvergne il y a 6 mois

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

#3 Mis à jour par Benjamin Dauvergne il y a 4 mois

Patch s'appliquant toujours sur master.

#5 Mis à jour par Frédéric Péters il y a 18 jours

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 Mis à jour par Benjamin Dauvergne il y a 18 jours

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 Mis à jour par Frédéric Péters il y a 18 jours

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 Mis à jour par Frédéric Péters il y a 18 jours

  • Statut changé de Solution proposée à Solution validée

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

#11 Mis à jour par Benjamin Dauvergne il y a 18 jours

  • Statut changé de Solution validée à 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 Mis à jour par Frédéric Péters il y a 16 jours

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF