lzear / LeekWars_Kikimeter

5 stars 3 forks source link

Options de configuration (colonnes du tableau) et stockage par cookies #13

Closed abourdin closed 9 years ago

lzear commented 9 years ago

Wah! Vous faites tous des trucs trop cools ! :+1: C'est vraiment l'effervescence aujourd'hui. Bravo à vous (Lark, Corshclaw, Foudge). Vous avez vraiment transformé le plugin en un outil super balèze ! J'ai pas trop la possibilité de travailler sur Kikimeter ces jours-ci, mais je serai probablement actif à nouveau dans environ 2 semaines.

Je pense à une petite amélioration qui pourrait servir : placer dataReceiverURL dans un cookie.

abourdin commented 9 years ago

Ouaip yLark, c'était une première itération pour poser les bases, qui va pouvoir servir à configurer tout ce qu'on a à configurer.

Pour le faire de déplacer la config dans les settings, vu qu'un userscript est lié à une page, il faudrait exporter ça dans un second script dédié à la config et attaché à /settings non ? Après, comme je pense qu'il va être assez facile de déclencher des re-display des différentes parties, je pense que garder la config ici pour modifier dynamiquement l'affichage de la page peut être intéressant.

J'ai hésité à ajouter le dataReceiverURL dans un cookie, parce qu'une fois le cookie set, pour changer sa valeur, il fallait la changer via la console javascript. Mais je pense pouvoir trouver une meilleure solution : si la chaîne dans le script est vide ou différente de la valeur stockée dans le cookie, on met le cookie à jour. Ca devrait faire l'affaire :)

yLark commented 9 years ago

Bien évidemment, Rome ne s'est pas faite en un jour !

Pour placer la config dans les settings, il suffit d'ajouter une ligne @match http://leekwars.com/settings dans l'entête du fichier. La conséquence, c'est qu'il faudra mettre des if pour exécuter les bouts de code suivant l'url de la page.

Pourquoi passer par la console ? On ne peut pas le faire avec une textbox ?

abourdin commented 9 years ago

Si bien sûr, encore faut-il avoir ajouté la textbox ^^