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

Problème/bug rencontré sur le site, évolution/amélioration à proposer
Avatar du membre
sly
Messages : 3757
Enregistré le : 29 févr. 2004, 18:59
Localisation : Chambéry - Savoie

[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)

NicoM
Messages : 415
Enregistré le : 29 juin 2013, 16:39
Localisation : strasbourg

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

effectivement çà serait bien

Avatar du membre
Dominique
Messages : 2621
Enregistré le : 08 avr. 2006, 21:58
Localisation : Chaville 92
Contact :

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

NicoM
Messages : 415
Enregistré le : 29 juin 2013, 16:39
Localisation : strasbourg

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

c'est possible?

Avatar du membre
Dominique
Messages : 2621
Enregistré le : 08 avr. 2006, 21:58
Localisation : Chaville 92
Contact :

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

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

NicoM
Messages : 415
Enregistré le : 29 juin 2013, 16:39
Localisation : strasbourg

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

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

Avatar du membre
sly
Messages : 3757
Enregistré le : 29 févr. 2004, 18:59
Localisation : Chambéry - Savoie

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

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

Avatar du membre
Dominique
Messages : 2621
Enregistré le : 08 avr. 2006, 21:58
Localisation : Chaville 92
Contact :

Message par Dominique » 03 déc. 2015, 20: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. :oops:

Avatar du membre
Dominique
Messages : 2621
Enregistré le : 08 avr. 2006, 21:58
Localisation : Chaville 92
Contact :

Message par Dominique » 02 févr. 2016, 23: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)

Avatar du membre
sly
Messages : 3757
Enregistré le : 29 févr. 2004, 18:59
Localisation : Chambéry - Savoie

Message par sly » 03 févr. 2016, 01: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.

Répondre