[FAIT] "zones couvertes" dans le bandeau trop pauvre

Message par **sly** » 16 oct. 2013, 22:35

Il serait bien de faire en sorte que le menu "zones couvertes" soit généré dynamiquement et rempli avec toutes nos "zones couvertes" à l'image de la page d'accueil (en dessous de la vignette qui montre les alpes)

Message par **NicoM** » 14 sept. 2014, 13:08

effectivement çà serait bien

Message par **Dominique** » 14 sept. 2014, 14:17

Bonne remarque. ça me rappelle vaguement quelque chose (pourquoi je n'avais pas fait comme ça à l'époque - il y avait un os) mais je ne sais plus.
Je vais re-tenter le coup

Message par **NicoM** » 03 déc. 2015, 15:51

c'est possible?

Message par **Dominique** » 03 déc. 2015, 16:18

Oui.
Je finis un gros truc qui n'a rien à voir et je reprend toutes les suggestions non abouties.

Message par **NicoM** » 03 déc. 2015, 16:29

merci, désolé de t'embêter pour un détail mais je trouve que ça fait plus propre et cohérent!

Message par **sly** » 03 déc. 2015, 17:05

Dominique a écrit :je reprend toutes les suggestions non abouties.

Whaa, t'as du temps libre ? Motivé le dominique ;-)

Message par **Dominique** » 03 déc. 2015, 19:16

sly a écrit :
Dominique a écrit :je reprend toutes les suggestions non abouties.
Whaa, t'as du temps libre ? Motivé le dominique

*** EDIT *** Je reprend QUELQUES suggestions non abouties et bugs mineurs que j'ai noté au fil du temps.

Message par **Dominique** » 02 févr. 2016, 22:11

sly a écrit :Il serait bien de faire en sorte que le menu "zones couvertes" soit généré dynamiquement et rempli avec toutes nos "zones couvertes" à l'image de la page d'accueil (en dessous de la vignette qui montre les alpes)

2 ans 1/2 plus tard, voilà qui est fait

Dominique a écrit :ça me rappelle vaguement quelque chose (pourquoi je n'avais pas fait comme ça à l'époque - il y avait un os) mais je ne sais plus.
Je vais re-tenter le coup

Ben voilà le loup: il y a du code de recherche dans la base à factoriser pour TOUTES les pages.
Je l'ai bien mis dans generales.routes mais ça ne marche que pour les URL converties en MVC (moins le forum et de vieux trucs tel la gestion)

Message par **sly** » 03 févr. 2016, 00:41

Dominique a écrit :Ben voilà le loup: il y a du code de recherche dans la base à factoriser pour TOUTES les pages.

Je vois le problème, comme la petite étoile qui indique un besoin de modération pour lequel j'ai écris :

FIXME : Dans une logique de rangement parfait, ça ne devrait pas être ici, mais dans chaque contrôleur qui a besoin (ou pas) de modifier le bandeau avec l'étoile, mais la factorisation a eu raison de moi ;-)
Si quelqu'un veut le bouger, il a mon feu vert -- sly

Dans la même logique de "flemme" ce que tu as fais est finalement très acceptable. C'est plus dur de se rappeler dans quel fichier on a ce code, mais ça réduit grandement les répétitions.

Dominique a écrit :moins le forum et de vieux trucs tel la gestion

Voui, là c'est pénible ce menu est vide quand on est dans le forum.

Pour /gestion bon, dès qu'un courageux s'y colle, zou ça sera réglé, pour le forum (et la gestion le temps que), bon ben, y'a qu'a mettre ton générateur de zone dans une fonction et l'appeler juste avant l'inclusion du bandeau.

Aller hop, je ne sais toujours pas où il faudrait ranger ça, donc comme l'étoile dans includes/autoconnexion.php, en attendant une prochaine tempête de réorganisation des trucs qui ne devraient pas être là où ils sont.