une petite erreur dans pxSingleCatListPages

[Melody Nelson]

salut,

je voulais juste signaler qu'il manquait un espace après
class="current" dans la fonction qui génère les numéros de pages
ça me bloquait la validation XHTML stricte, je l'ai modifié sur mon
site (dans manager/inc/lib.frontend.php) mais au cas où ça puisse
servir à quelqu'un d'autre je le poste ici

ligne 361 :
if ($i == $activepage) $class = 'class="current"';

modification apportée :

if ($i == $activepage) $class = 'class="current" ';

[Jojaba]

Melody Nelson a écrit :

salut,

je voulais juste signaler qu'il manquait un espace après
class="current" dans la fonction qui génère les numéros de pages
ça me bloquait la validation XHTML stricte, je l'ai modifié sur mon
site (dans manager/inc/lib.frontend.php) mais au cas où ça puisse
servir à quelqu'un d'autre je le poste ici

ligne 361 :
            if ($i == $activepage) $class = 'class="current"';

modification apportée :

            if ($i == $activepage) $class = 'class="current" ';
--~--~---------~--~----~------------~-------~--~----~

  

Merci pour ce signalement Melody.
Je viens de modifier ça dans le dépôt pour la prochaine version ;-)
Ceci dit, j'ai préféré modifier la ligne 364 :
                 $pages[]='<a '.$class.'href="'.$url.'">'.$i.'</a>';
Modification apportée (espace après '$class.') :
                $pages[]='<a '.$class.' href="'.$url.'">'.$i.'</a>';

[Jojaba]

Oups !
Mauvais choix ! Désolé d'avoir proposé quelque chose qui n'est pas bon
(puisque si on adopte la modification que je propose, on obtiendra un
espace en trop dans la balise dans le cas où la page visionnée n'est as
la page courante ! Je viens de resoumettre ta proposition dans le svn.
Merci encore.