Open pascalchevrel opened 12 years ago
JeremiePat
commented
12 years ago Oui, pas de problème, vas-y :)
Si quelqu'un utilise le plugin en l'état, il peut déjà bénéficier de HJS 7 s'il choisi de passer par Yandex :)
Pour la MAJ, à moins qu'il y ai de gros changements dans la structure des feuilles de style, ça devrait être assez trivial à faire.
pascalchevrel
commented
12 years ago Pour Yandex non, tu as des URLs en dur dans le code (http://yandex.st/highlightjs/6.2/), ça fait partie des choses à màj pour n'avoir qu'à changer le numéro de version à un seul endroit.
J'ai vu qu'il y avait aussi des définitions de langage dans le code et une boucle sur leur nombre (45), il y en a maintenant 51, je ferais plutôt une boucle sur le count() des langages dans une variable statique en fait.
Pour les feuilles de style j'ai rien vu de nouveau dans la structure mais il y en a deux de plus.
Pour la page d'admin, si on choisit une version personnalisée, tous les langages sont saisis en dur dans le html, ça serait bien de boucler un tableau des langages pour pas avoir à le mettre à jour manuellement non plus.
Je pense qu'on peut avoir une mise à jour crade en 2h, mais (pour mon niveau de code), ça me prendra plus de temps de nettoyer de manière à ce que ce soit plus maintenable avec le temps.
pascalchevrel
commented
12 years ago aussi, je sais pas encore si je te ferai des pull request petit à petit ou bien tout d'un coup :)
il y a eu une grosse màj de HighlightJS cette semaine, ça serait bien de l'intégrer.
J'ai commencé à regardé en local et ça serait aussi l'occasion de nettoyer du code pour faciliter les màj des versions suivantes.
Je suis volontaire pour essayer cet été (màj vers la 7.0 + nettoyage code), c'est la première fois que je travaille sur un plugin dotclear donc c'est intéressant pour moi de voir si j'y arrive.