Projet

Général

Profil

Bug #7996

Le menu latéral n'apparait pas si un des jsonp de COMBO_KNOWN_SERVICES n'est pas récupérable

Ajouté par Benjamin Dauvergne il y a plus de 8 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
03 août 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Club:
Non

Description

Dans ce code:

      $.ajax({url: element.backoffice_menu_url,
            xhrFields: { withCredentials: true },
            async: true,
            dataType: 'jsonp',
            crossDomain: true,
            success: function(data) { element.data = data; check_all_done(); },
            error: function(error) { console.log('bouh', error); element.data = Array(); check_all_done(); }
           }
       );

le calback error n'est jamais appelé car jQuery ne gère pas les erreurs sur du jsonp cross-domain (voir http://api.jquery.com/jQuery.ajax/ sur le paramètre error et http://stackoverflow.com/questions/19035557/jsonp-request-error-handling#answer-19207855 pour des contournements).

Coté le w.c.s. le bug n'est pas visible car w.c.s. semble construire son menu latéral tout seul sans passer par create_menu_items() de publik.js (un breakpoint ne s'arrête jamais dedans pourtant je vois bien un menu).

Je me dis qu'on pourrait tenter une construction progressive du menu, en laissant les entrées en cours de chargement avec un logo pulsatile (comment traduire throbber ? qui n'existe même pas dans un dico d'anglais) puis développer l'entrée quand c'est terminé.


Fichiers

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 8 ans

  • Assigné à mis à Frédéric Péters
#2

Mis à jour par Frédéric Péters il y a plus de 8 ans

#4

Mis à jour par Thomas Noël il y a plus de 8 ans

Ça m'a l'air bon mais j'ai pas testé en vrai, na.

#5

Mis à jour par Frédéric Péters il y a plus de 8 ans

  • Statut changé de En cours à Résolu (à déployer)
commit de36956369124b119510431835c3d57bbb1d52bf
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Sep 16 18:52:15 2015 +0200

    gadjo: add progressive rendering of publik menu (#7996)
#6

Mis à jour par Frédéric Péters il y a plus de 8 ans

#7

Mis à jour par Frédéric Péters il y a environ 5 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF