P@t a écrit:
vincent3569 a écrit:
pendant qu'on y est, je rajoute quelque fonctions à paramétrer :
- menu catégorie
- le nombre d'image dans chaque catégorie
- le nombre total d'imagePourquoi pas... je préfèrerai que ce soit en $conf dans le fichier de configuration.
Ou alors, encore mieux, géré par AMM.
c'est pas vraiment une personnalisation liée aux menus mais plutôt une personnalisation tout cours.
si c'est en variable de conf (bien documentée), ça me va bien aussi
Hors ligne
je vais faire mon monsieur plus de chez bahlsen etma demande concerne un ajout de fonctionnalité plutôt q'une suppression :-)
serait possible de prévoir un paramétrage de personnalisation pour rajouter un lien "notification" dans le pied de page avec une icone RSS qui va avec ?
et si pas possible, ajouter une icone RSS en face du lien notification du menu ?
au vu de ce que je vois sur le web, c'est une pratique assez généralisée de mettre le lien RSS dans le footer (et ça m'arrangerait vu que je veux faire alléger au max le menu de ma galerie (c'est ce que j'ai fait en template perso)
Hors ligne
P@t a écrit:
Et voila, les propriétés de la photo sont maintenant activables/désactivables à volonté.
C'est-y pas beautiful tout ca?
du coup, tu as embarqué une de mes demandes d'évol [Bugtracker] ticket 1513
je ne sais pas comment la faire passer à résolu sur mantis, si tu peux t'en charger...
Hors ligne
P@t a écrit:
Et voila, les propriétés de la photo sont maintenant activables/désactivables à volonté.
C'est-y pas beautiful tout ca?
P@t,
J'ai 2 remarques:
1. Le display de rating n'est pas automatiquement caché si on decoche l'option rating for all ?
2. Je pense que j'aurais mis dans la base de données et dans l'interface plutot "Cacher XXX" que "Afficher XXX" car par defaut on affiche tout. Si on veut maintenant ajouter une nouvelle option dans la base on n'a pas a se compliquer la vie pour la rajouter dans le array de la table config et par defaut ca sera activé. Tu penses quoi ?
Hors ligne
rvelices a écrit:
1. Le display de rating n'est pas automatiquement caché si on decoche l'option rating for all ?
Si tu parles de l'average rate, oui, elle est automatiquement caché si on décoche l'option rating de la configuration générale.
Mais j'ai pensé qu'un utilisateur peut vouloir se contenter d'afficher la note sur la page des catégories, et pas sur la page de l'image...
rvelices a écrit:
2. Je pense que j'aurais mis dans la base de données et dans l'interface plutot "Cacher XXX" que "Afficher XXX" car par defaut on affiche tout. Si on veut maintenant ajouter une nouvelle option dans la base on n'a pas a se compliquer la vie pour la rajouter dans le array de la table config et par defaut ca sera activé. Tu penses quoi ?
La, je suis pas d'accord, si on veut afficher quelque chose, on coche, et si on veut cacher quelque chose, on décoche. Je ressortirai ce que m'a dit pierrick un jour: "si possible, une variable ne doit pas représenter quelque chose de négatif" (comme par exemple l'option "no_fallback" de la fonction load_language). Pour la base de donnée, cela pourrait se justifier, mais ca obligerait quand meme à coder:
if (!isset($conf['new_param']) or !$conf['new_param'])
Au lieu de:
if (!isset($conf['new_param']) or $conf['new_param'])
C'est kifkif, non?
Hors ligne
vincent3569 a écrit:
serait possible de prévoir un paramétrage de personnalisation pour rajouter un lien "notification" dans le pied de page avec une icone RSS qui va avec ?
et si pas possible, ajouter une icone RSS en face du lien notification du menu ?
au vu de ce que je vois sur le web, c'est une pratique assez généralisée de mettre le lien RSS dans le footer (et ça m'arrangerait vu que je veux faire alléger au max le menu de ma galerie (c'est ce que j'ai fait en template perso)
En fait, c'est pas tant icône/pas icône ou menu vs/footer qui me gêne, mais plutôt le fait que le lien affiche une autre page, sur laquelle il faut ensuite re-cliquer pour souscrire à l'un ou l'autre flux RSS.
Je pense que ces 2 liens devraient être directement accessibles en page principale, un peu comme sur mon blog, par exemple.
Hors ligne
P@t a écrit:
La, je suis pas d'accord, si on veut afficher quelque chose, on coche, et si on veut cacher quelque chose, on décoche.
+1
Il faut rester cohérent sur l'ensemble des paramètres, sinon les utilisateurs vont se planter à tous les coups.
Hors ligne
P@t a écrit:
rvelices a écrit:
2. Je pense que j'aurais mis dans la base de données et dans l'interface plutot "Cacher XXX" que "Afficher XXX" car par defaut on affiche tout. Si on veut maintenant ajouter une nouvelle option dans la base on n'a pas a se compliquer la vie pour la rajouter dans le array de la table config et par defaut ca sera activé. Tu penses quoi ?
La, je suis pas d'accord, si on veut afficher quelque chose, on coche, et si on veut cacher quelque chose, on décoche. Je ressortirai ce que m'a dit pierrick un jour: "si possible, une variable ne doit pas représenter quelque chose de négatif" (comme par exemple l'option "no_fallback" de la fonction load_language). Pour la base de donnée, cela pourrait se justifier, mais ca obligerait quand meme à coder:
if (!isset($conf['new_param']) or !$conf['new_param'])
Au lieu de:
if (!isset($conf['new_param']) or $conf['new_param'])
C'est kifkif, non?
Oui c'est kifkif... Seulement que si je dois rajouter un jour une option dans la base, ca me gonfle d'ecrire un xx-database.php pour lire le array de la base et rajouter une cle dans la map pour le reecrire apres ...
Concernant la negation - je suis d'accord avec Pierrick, mais il y a toujours des exceptions. Pour prendre ton exemple de no_fallback. C'est toujours discutable de ce qui est plus lisible:
if ( ! @$options['no_fallback'] )
ou
if ( !isset($options['fallback']) or $options['fallback']==true )
Hors ligne
rvelices a écrit:
Oui c'est kifkif... Seulement que si je dois rajouter un jour une option dans la base, ca me gonfle d'ecrire un xx-database.php pour lire le array de la base et rajouter une cle dans la map pour le reecrire apres ...
Concernant la negation - je suis d'accord avec Pierrick, mais il y a toujours des exceptions. Pour prendre ton exemple de no_fallback. C'est toujours discutable de ce qui est plus lisible:Code:
if ( ! @$options['no_fallback'] )ou
Code:
if ( !isset($options['fallback']) or $options['fallback']==true )
Oulalalalala.... un @ dans le code! Pas bien! Je me suis fait engueuler pour beaucoup moins que ca... (cf [Subversion] r5283)
Par contre, un truc faisable (merci pierrick), c'est de rajouter les $conf dans le config_default, et surcharger en bdd.
Hors ligne
P@t a écrit:
vincent3569 a écrit:
page profile, lancée par le menu identification/custom
- permettre la personnalisation ou pas (afficher ou pas le bloc préférence)
=> permettrait de forcer le look de la galerie pour tous les visiteurs, connectés ou pasC'est une bonne idée... ca reviendrait à un status intermédiaire entre generic et visiteur.
EDIT: ca c'est fait: [Subversion] r5328
c'est sur le trunck ?
je ne vois rien qui y ressemble...
Hors ligne
P@t a écrit:
Par contre, un truc faisable (merci pierrick), c'est de rajouter les $conf dans le config_default, et surcharger en bdd.
+1
Simple et de bon goût...
Hors ligne
P@t a écrit:
Oulalalalala.... un @ dans le code! Pas bien! Je me suis fait engueuler pour beaucoup moins que ca... (cf [Subversion] r5283)
Qui c'est qui est méchant comme ça ?
Hors ligne
plg a écrit:
P@t a écrit:
Oulalalalala.... un @ dans le code! Pas bien! Je me suis fait engueuler pour beaucoup moins que ca... (cf [Subversion] r5283)
Qui c'est qui est méchant comme ça ?
Regarde dans ta boîte mail : toute l'équipe l'a vu passer ...
Hors ligne
plg a écrit:
P@t a écrit:
Oulalalalala.... un @ dans le code! Pas bien! Je me suis fait engueuler pour beaucoup moins que ca... (cf [Subversion] r5283)
Qui c'est qui est méchant comme ça ?
Pour une fois que ce n'est pas moi.
;-)
PS: Je ne dénoncerai pas le responsable d'autant qu'il n'avait pas forcément tort.
Hors ligne
VDigital a écrit:
PS: Je ne dénoncerai pas le responsable d'autant qu'il n'avait pas forcément tort.
Je rapelle que mon @ était en partie admin, uniquement en cas de changement de date de création...
Dans le cas présent, c'est en partie publique que rvelices veut en mettre un... Booooouuuuuh!
Hors ligne