TangoMan75
commented
6 years ago Salut Thibault, désolé de répondre si tard, je viens juste de voir ton post.
J'ai honte, je te présente mes excuses, je dis de la merde, excuse moi. Je suis vraiment désolé de t'avoir vexé.
Licence
Je crois pas que j'ai enfreint les termes de la licence.
J'ai juste changé les namespaces parceque j'avais préféré regrouper tous mes bundles dans le même vendor.
J'ai conservé ton nom et ton contact dans la liste des "authors" dans le composer.json.
Et en plus j'ai pas dupliqué le dépôt, j'ai fork, donc je crois pas qu'il n'y ait aucune ambiguité que tu es l'auteur original.
Merge
Si tu veux je te propose de merge mais nos versions ont peut être trop divergées... Par exemple je comprend pas pourquoi tu as ajouté le parametre get ici :
public function paginationFunction(Paginator $paginator, $get = 'page') {
// ...Ce qui implique que je dois adapter tous mes templates.
Configuration
Et aussi :
OK, c'est une option supplémentaire de passer le chemin du template par défaut dans le config.yml.
Mais j'avais imaginé que ça serait mieux de passer le nom ou le chemin du template à utiliser comme un paramètre de la fonction :
{{ pagination(listUser, 'smart') }}De façon à ce qu'on puisse utiliser plusieurs paginations sur la même page avec des templates différents.
Qu'est ce que tu en dis ?
Pagination "intelligente"
Et qu'est ce que tu penses de la pagination intelligente ?
Utilisation de rel="next" et rel="prev"
Ah, j'ignorais que les rel="next" et rel="prev" pouvaient être inclus dans le body.
Dans la doc de google ils donnent cet exemple :
Dans la section
<head>de la première page (http://www.example.com/article-partie1.html), ajoutez une balise de lien redirigeant vers la page suivante de la séquence, comme ceci :
Et dans le cas où on aurait plusieures paginations sur la même page, comment le robot va distinguer le balisage qui se rapporte à chacune des différentes paginations ?
Qu'est ce que tu suggères ?
rel="canonical"
Le rel="canonical" c'était une demande pour l'ADM.
Perso, je trouve pas délirant de l'inclure dans le bundle de pagination, dans la mesure où ça serait un overkill de créer un bundle juste pour gèrer ce truc.
Bises
Yo, pour info, j'ai fait la mise à jour nécessaire sur le repo original : Tiloweb/SymfonyPaginationBundle
Tu peux maintenant ajouter un fichier twig personnalisé depuis
config.yml.Concernant le SEO, les
rel="prev"etrel="next"peuvent tout à fait être inclus dans le body, directement dans la pagination, surtout si on arrive dans le cas où il y a plusieurs pagination dans la page.La pagination n'a pas vocation à gérer le
rel="canonical"qui a un comportement propre.Libre à toi de continuer sur ton propre repo, mais merci de respecter la licence originale (et d'arrêter de dire en soirée que ce que je fais c'est de la merde <3).
Bise !