Development #29362
pwa : ajouter une section de configuration de la navigation
0%
Description
Maintenant qu'il y a une page de configuration (#25496), il faut y ajouter une section permettant de configurer les éléments qu'on veut retrouver dans la navigation. (étant donné qu'on a vu qu'imposer une unique navigation desktop/mobile ne tenait pas).
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-pwa-add-management-of-navigation-entries-29362.patch 0001-pwa-add-management-of-navigation-entries-29362.patch ajouté
- Fichier Screenshot_2018-12-28 Services en ligne.png Screenshot_2018-12-28 Services en ligne.png ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Rien de bien fancy ici, simple liste d'éléments pour la navigation, à exploiter ensuite dans publik-base-theme.
Mis à jour par Thomas Noël il y a plus de 5 ans
Sur le get_url :
def get_url(self):
if self.url:
return utils.get_templated_url(self.url)
return self.link_page.get_online_url()
j'inverserais la logique, c'est link_page qui passe en premier. Comme ça, on n'oublie pas de la retirer quand on veut forcer une URL externe (l'idée m'est venue de vérifier ça parce que on_delete=CASCADE, quand la link_page est supprimée, l'entrée aussi).
En fait ça pourrait aussi être contraint dans form_valid, si y'a un form.instance.url alors il ne peut pas y avoir de form.instance.link_page_id.
Est-ce qu'on veut aussi contraindre, autant que possible, le format du fichier icon ? À ce stade je dirais que non, mais je me posais la question parce que, en fait, moi même je ne sais pas ce qui est attendu : peut-être qu'on pourrait avoir un verbose_name qui précise qu'il faut du png ou svg (ou autre) ? Quoi qu'il en soit, un verbose_name i18né serait bienvenu je pense.
En dehors de ça, tout me semble bel et bon. Et ça donne très envie d'avoir la même chose pour définir des entrées supplémentaires dans le menu latéral du portail agent.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-pwa-add-management-of-navigation-entries-29362.patch 0001-pwa-add-management-of-navigation-entries-29362.patch ajouté
j'inverserais la logique, c'est link_page qui passe en premier.
Oui et c'est d'ailleurs ce qui est fait dans la cellule Lien, je pensais avoir regardé pour faire pareil.
En fait ça pourrait aussi être contraint dans form_valid, si y'a un form.instance.url alors il ne peut pas y avoir de form.instance.link_page_id.
En effet, ajouté au code et au test.
Est-ce qu'on veut aussi contraindre, autant que possible, le format du fichier icon ? À ce stade je dirais que non, mais je me posais la question parce que, en fait, moi même je ne sais pas ce qui est attendu : peut-être qu'on pourrait avoir un verbose_name qui précise qu'il faut du png ou svg (ou autre) ? Quoi qu'il en soit, un verbose_name i18né serait bienvenu je pense.
Pour le moment on a du PNG (Metz) et du SVG (Quimper); j'ai ajouté le verbose_name oublié en mettant juste _('Icon') pour le moment.
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Impec.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit d9367275de8540a881135f3d5ff3637b08874b91
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Thu Dec 27 15:26:21 2018 +0100
pwa: add management of navigation entries (#29362)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
pwa: add management of navigation entries (#29362)