Pas mal l'auteur ton script, mais malheureusement, je n'en télécharge aucun, déjà que mon chrome n'est pas très performant, je crois que lui rajouter des scripts ne sera pas très judicieux.
Alors ton module est parfait, je ne le trouvais pas très abouti dans les versions précédentes, mais la il remplit tout ce que je demande pour un module jvc ! Sauf que... le script est un peu lent chez moi (windows 8 et firefox), je pense que ceci est dû au fait que tu n'ai pas fais le script agissant de manire "asynchrone", c'est à dire qu'il faut attendre que toutes les fonctions du module chargent pour pouvoir continuer à naviguer normalement de manière fluide et sans que cela rame. J'espère ce que ce léger défaut assez contraignant sera corrigé dans les prochaines versions !
Bon alors ça va un peu moins, c'est beaucoup plus rapide mais c'est quand même un peu lent quelques fois
Merci de m'aider à améliorer le script.
Je compte travailler sur la lenteur générale du script juste après l'amélioration des citations.
Le script ne travaille pas de manière asynchrone mais toutes les requêtes sont lancées en même temps. J'ai plusieurs pistes d'amélioration pour ça :
- Ne récupérer qu'une seule fois les données d'un auteur (pour l'instant, si l'auteur a posté 5 fois dans la page, les données sont récupérées 5 fois)
- Ne faire qu'une seule requête pour tous les auteurs, mais comme l'API de JVC ne fonctionne pas comme ça, je serai obligé de passer par mon serveur pour compiler tous les résultats en une seule réponse.
- Mettre en cache les données de l'auteur, dans ce cas il faut que je songe à un système de péremption au cas où l'auteur change d'avatar
- Charger certains modules en JSONP seulement quand on en a besoin. Cette solution sera appliquée en dernier recours parce qu'elle risque d'être difficile à mettre en place.
Du coup, ça devrait aller mieux dans la version 1.6.3
(En attendant, ça semble fonctionner correctement sous Chrome donc si vous tenez absolument à utiliser SpawnKill de manière fluide, vous pouvez via ce navigateur)
Je poste ici un échantillon de quelques scripts de citation pour m'aider au développement, désolé pour le flood :
------------------
et moi le second vu que je viens de l avoir lol
BEATRICE
╭ dalfinus, le 18 janvier 2009 à 23:39:02
┊ https://www.jeuxvideo.com/forums/1-16371-71-1-0-1-0-prems.htm#message_79
┊┄┄┄
┊ et moi le second vu que je viens de l avoir lol
╰┄┄┄
SpawnKill
| https://www.jeuxvideo.com/forums/1-16371-71-1-0-1-0-prems.htm#message_79
| Ecrit par « dalfinus », 18 janvier 2009 à 23:39:02
| « et moi le second vu que je viens de l avoir lol »
JVC Master
| dalfinus - le 18 janvier 2009
| https://www.jeuxvideo.com/forums/1-16371-71-1-0-1-0-prems.htm#message_79
| et moi le second vu que je viens de l avoir lol
TurboForum
Je préfère le style de citation de jvc master perso
Au pire tu laisse le choix à l'utilisateur via le panneau de config de choisir parmi des style de citations multiple
Les citations TurboForum et JVC Master sont sympa mais il n'y a pas assez d'espace entre l'entête et le message, à mon avis.
Du coup, ça force à réfléchir quand on lit la citation.
BEATRICE, j'aime pas du tout le permalien sous le message.
Mise à part ça, j'aime pas trop le chevron pour indiquer la réponse mais c'est un avis perso. je pense que le cadre permet de bien séparer l'entête de la citation, le texte de la citation et la réponse.
Si vous avez des pistes d'amélioration, hésitez pas
╭ Saberdream, le 8 août 2014 à 15:41:22
┊ https://www.jeuxvideo.com/forums/1-1000021-2267708-11-0-1-0-script-jvc-spawnkill-avant-respawn.htm#message_2277769
┊┄┄┄
┊ Au pire tu laisse le choix à l'utilisateur via le panneau de config de choisir parmi des style de citations multiple
╰┄┄┄
C'est prévu, mais pas immédiatement. De toute façon, dans la version qui arrive, toutes les citations seront stylisées à la manière de celles de JVC Master. Je supporterai les quatre formats de citation ci-dessus.
Et pour ceux qui n'auraient pas envie de citation stylisée a la jvc master par exemple on pourra désactiver la mise en forme ou on aura pas le choix ?
Si je me suis fait chier à faire un panneau de configuration complexe, c'est pour l'utiliser
Bien-sûr qu'on aura le choix
Ok tant mieux, tu me rassures. C'est vrai que ça serait un peu bête d'avoir codé un panneau de configuration mais de ne l'utiliser que pour certaines fonctions... Bref ton module a un bel avenir je le sens, même avec l'arrivée de respawn si tu veux mon avis. Tu pourrais peut-être même en faire une api à part entière, indépendante de gm, qui sait.
Si j'ai le courage de m'y mettre et que c'est pas trop compliqué à maintenir, je ferai peut-être une extension pour navigateur à part entière (qui fonctionne sans greaseMonkey).
Ah ça serait cool ça, un genre d'extension "réseau social" du genre de l'extension qu'un gars a créé pour twitter
Je suis en train de faire les regex pour matcher tous les types de citation. C'est passionnant
Pour ceux que ça intéresse :
BEATRICE (sans lien, impossible à matcher, pas de délimitation de fin)
/^# (.*)\n^# Posté le (\d{2}) ([^\s]*) (\d{4}) à (\d{2}:\d{2}:\d{2})\n((?:.|[\n\r])*?)\n^# (http.*)/m
$1 : pseudo
$2 : jour
$3 : mois
$4 : année
$5 : heure
$6 : message
$7 : permalien
SpawnKill
/^╭ ([^,]*), le (\d{2}) ([^\s]*) (\d{4}) à (\d{2}:\d{2}:\d{2})\n^┊ (http.*)\n┊┄┄┄\n((?:.|[\n\r])*)\n^╰┄┄┄/m
$1 : pseudo
$2 : jour
$3 : mois
$4 : année
$5 : heure
$6 : permalien
$7 : message (à épurer en retirant le cadre)
JVC Master
/(?:(?:^\| (http.*)\n)|())^\| Ecrit par « ([^\s]*) »(?:[^\d]*)(\d{2}) ([^\s]*) (\d{4}) à (\d{2}:\d{2}:\d{2})\n\| « ((?:.|[\n\r])*?) »(?:(?:[\s]*)> )*/m
$1 : permalien (peut être vide)
$2 : pseudo
$3 : jour
$4 : mois
$5 : année
$6 : heure
$7 : message (à épurer en retirant le cadre)
Turboforum
/^\| ([^\s]*)[^-]*-[^l]*le (\d{2}) ([^\s]*) (\d{4})(?:\n\| (http.*)|())\n((?:(?:\n*^\|.*)*)*)(?:(?:[\s]*)> )*/m
$1 : pseudo
$2 : jour
$3 : mois
$4 : année
$5 : permalien (peut être vide)
$6 : message (à épurer en retirant le cadre)
Intéressant, moi qui suis passionné de langages web toutes ces lignes de code ne me déplaisent pas
La regex de jvc master qui est la plus longue alors qu'elle parait bien plus simple que Turboforum
Go remettre le bouton pour la cdv à coté du pseudo comme avant
Il y est déjà, t'as dû désactiver l'option
J'avance doucement mais sûrement
(Ne vous inquiétez pas, les couleurs sont juste là pour m'aider, c'est provisoire)
Je dois encore :
- régler quelques problèmes de reconnaissance des citations,
- faire le design des citations
- gérer les citations imbriquées