Journal DLFPToolbar 1.0-beta1

Posté par  (site web personnel) .
Étiquettes : aucune
0
7
juin
2005
Voila une beta de la toolbar dont vous reviez tous. Les petits malins noterons que ca ressemble furieusement a une version que j'avais posté ya quelque temps deja, mais qui faisait crasher konqueror 3.x.y (je n'ai plus les x et y en tete).

Tout a été ré-écrit ou presque, et il y a fort a parier que ca ne marche que sous gecko (et encore, j'ai pas testé deer park :), mais je compte bosser sur la compatibilité avec les autres navigateurs qui ont plus de mal avec le DOM plus tard. (Patches bienvenus, comme d'hab :)

Donc ca se passe la:
http://virgule.info/linuxfr/1.0-beta1/(...)

Vous pouvez changer le seuil, la feuille de style (vous avez plutot interet a la faire vu que celle par défaut ne style pas la toolbar), naviguer dans les nouveaux commentaires, ca marche a peu pres comme la toolbar actuelle, sauf que:
  • C'est 3 fois plus rapide (voire plus :)
  • C'est pas sur que tout marche
  • On peut replier tout un thread en double cliquant sur le petit [+]/[-]
  • On peut faire un systeme de blacklist d'utilisateur, pour automatiquement planquer les trolls. Ca sera dans la prochaine béta, pas le temps de finir le truc la. (cf le paragraphe mentionnant les patches bienvenus tout ca, le boulot est pré-maché)
  • Ya une vague tentative de rassembler dans une seule stylesheet la toolbar et le reste du site, et d'autre part le choix d'une feuille de style remplace l'actuelle au lieu d'etre simplement rajoutée (les auteurs de feuilles de styles apprécieront)
  • Ya pas tellement de doc pour le moment, mais le code est beaucoup plus clair et commenté.
  • C'est pas encore intégrable au site. Mettons que ca sera le cas dans genre 1 mois, d'ici la je compte faire 2 ou 3 beta supplémentaires, avec notamment un test sur des gros articles (deja fait en local, mais ca me lourde de re-uploader la :)

Enjoy, et les suggestions, cheques, dons en nature, choppes de biere et encouragements sont tout aussi bienvenu(e)(s) que les patches.
  • # Commentaire supprimé

    Posté par  . Évalué à 3.

    Ce commentaire a été supprimé par l’équipe de modération.

    • [^] # Re: Bien joué, merci

      Posté par  (site web personnel) . Évalué à 3.

      Les accesskey, ya deja < et > pour les nouveaux messages (le truc a droite de la toolbar). J'avais mis ceux la parceque les touches étaient a coté, c'etait simple a utiliser du coup :)

      Le N rouge, il peut etre mis via CSS. (cf generated content dans la recommandation CSS), et pour la barre qui se planque ou pas, c'est aussi uniquement du CSS, donc c'est pas mon probleme :-)
    • [^] # Re: Bien joué, merci

      Posté par  . Évalué à 2.

      Je confirme que sous Konqueror 3.3.2, la barre ne se cache pas automatiquement. Je rajoute que sur Opera 8, elle n'apparait pas du tout. En tout cas, bonne chance pour ce boulot, et merci.
      • [^] # Re: Bien joué, merci

        Posté par  (site web personnel) . Évalué à 2.

        Idem sous konqueror 3.4.1
        qui reporte le bogue? :)
        • [^] # Re: Bien joué, merci

          Posté par  (site web personnel) . Évalué à 2.

          En meme temps... elle marche, a part ca, sous konqueror ? Et le dépliage/repliage aussi ?
          • [^] # Re: Bien joué, merci

            Posté par  (site web personnel) . Évalué à 2.

            J'ai même plus les plus/moins...
            Juste un [+] pour les < 0
            On peut changer le seuil mais ca change rien
            On dirait qu'elle est pas chargée la!
            tout les commentaires sont marqués deja lu donc je peux pas essayer la navigation
            • [^] # Re: Bien joué, merci

              Posté par  (site web personnel) . Évalué à 2.

              Ok, donc c'est juste que le support DOM/Js de konqueror ne suffit pas pour cette nouvelle version, comme je m'en doutais. Patches bienvenus en attendant ...
              • [^] # Re: Bien joué, merci

                Posté par  (site web personnel) . Évalué à 2.

                Update: j'ai trouvé ce qui ne va pas dans cette version avec KHTML / Webcore. Une nouvelle version fixant ce probleme devrait arriver demain. (et finalement, c'est pas le support DOM/JS, c'est juste une interprétation un rien différente, mais qui se défend aussi, au niveau des elements script dans le DOM)
  • # cool.

    Posté par  (site web personnel) . Évalué à 2.

    Sympa de t'y recoller :)

    Une remarque sur le double clique pour tout replier/déplier, je pense que cela pourrait être bien si c'était sur la barre de titre de chaque commentaire plutot que sur le [+].

    Pour la tester en direct, il n'est pas possible d'en faire une version Greasemonkey? Il y a un empechement technique? Sinon, j'veux bien m'y essayer.

    Bon courage pour la suite.
    • [^] # Re: cool.

      Posté par  (site web personnel) . Évalué à 3.

      Pour greasemonkey, Ya un empechement technique, en fait ya une modif a faire dans les fonctions de linuxfr (dans javascriptxxxx.js) qui 'taggent' un nouveau commentaire. C'est possible de contourner, mais ca serait un poil lent (en fait c'est une des modifs qui permettent d'accelerer cette version). Cela dit, c'est possible, ca serait juste un poil plus chiant.

      Pour le double clic, j'y ai pensé, mais certaines personnent double cliquent pour selectionner le texte du titre, et donc c'est ptet un peu moyen... Si quelqu'un a une meilleure suggestion...
  • # Bien

    Posté par  (site web personnel) . Évalué à 1.

    Ta feuille de style me donne un linuxfr sobre et fonctionnel. Le pliage des fils me plait bien.

    Merci :-)
    • [^] # Re: Bien

      Posté par  . Évalué à 1.

      >C'est 3 fois plus rapide (voire plus :)

      Un grand merci pour cette amélioration. Les journaux sur le TCE m'avaient obligé à désactiver la toolbar qui provoquait des messages alarmistes de firefox ( "A script on this page is causing mozilla to run slowly") sur mon duron 700 :s
    • [^] # Re: Bien

      Posté par  (site web personnel) . Évalué à 2.

      Petite précision, ce n'est pas ma feuille de style (kaiska.css, tu peux deja l'utiliser actuellement sur le site, d'ailleurs ca doit etre celle par défaut) :)
      • [^] # Re: Bien

        Posté par  . Évalué à 4.

        Pardon, je ne sais pas comment installer ton toolbar.
        • [^] # Re: Bien

          Posté par  . Évalué à 3.

          y 'a rien à installer.
          C'est un élément html
          Tu vas sur le lien qui a été indiqué puis sur la page html et tu vas voir en bas de ta fenêtre de navigation une petite barre, tu passe le curseur dessus et la barre se déploit.
          Puis dans le reste de la page il y a de nouvelle fonctionnalités, pas vraiment besion de la barre pour ça.
          Sinon dlfp possède déjà une barre de ce genre. Tu vas dans "modifier vos préférences" puis tu coches "activer la toolbar".

          voilà :)
  • # Bravo

    Posté par  . Évalué à 2.

    En tout cas, bravo pour l'amélioration des performances ! C'était le seul gros reproche que je pouvais faire à la toolbar. Là, ça va carrément plus vite. On va pouvoir troller encore plus et mieux grâce au repliage des threads.

    Bravo !
  • # Le problème de la molette

    Posté par  . Évalué à 6.

    Il semble que le problème de la molette (déjà présent sur la version actuelle et signalé là: http://linuxfr.org/tracker/180.html(...) ) subsiste. Comment répéter:

    -utiliser firefox
    -mettre le pointeur de la souris sur la toolbar
    -cliquer sur "»"
    -mettre le pointeur sur la zone des messages ( jusque là tout va bien)
    -faire tourner la molette
    -retourner sur la toolbar elle apparaît vide

    Sinon bravo et merci pour tout ce bon boulot.
    • [^] # Re: Le problème de la molette

      Posté par  . Évalué à 1.

      s/messages/commentaires/ mais vous aviez compris.
    • [^] # Re: Le problème de la molette

      Posté par  (site web personnel) . Évalué à 4.

      Ah, bien joué, je cherchais ce bug depuis longtemps, et je n'avais aucune idée de comment le reproduire (j'avoue ne pas regarder trop le systeme de suivi :)

      Je vais chercher d'ou ca vient, j'ai ma petite idée, il est fort possible que ca soit uniquement CSS, du fait que le contenu de la toolbar dépasse, et du coup il est possible de scroller dedans. D'ailleurs en tournant la molette dans l'autre sens quand tu es sur la toolbar, elle devrait ré-apparaitre.
      • [^] # Re: Le problème de la molette

        Posté par  . Évalué à 3.

        Yep, si tu épaissis légèrement la toolbar (au repos) tu peux voir le contenu scroller joyeusement. Le problème, c'est qu'il faut lui redonner le focus pour scroller dans l'autre sens si elle l'a perdu, et que c'est pas évident une fois le contenu caché (mais ça le fait avec les access keys).

        Perso, je serais pour la garder tout simplement toujours visible : c'est ce que je fais avec ma css perso (juste qlqs modifs sur kaiska), et quand elle tient sur une seule ligne on peut vraiment pas dire que ça bouffe de la place. Je trouve ça plus agréable pour pouvoir aller viser à la souris droit sur là où est le machin que tu veux cliquer.
      • [^] # Re: Le problème de la molette

        Posté par  (site web personnel) . Évalué à 3.

        oui, ca doit venir de la css car avec celle que j'utilise, je n'arrive pas à faire la même chose... Ou alors je n'ai pas bien compris :)
  • # Nouvelle Beta

    Posté par  (site web personnel) . Évalué à 4.

    http://virgule.info/linuxfr/1.0-beta2/(...)

    ChangeLog:
    * Passage du double click sur le titre, parceque c'est quand meme plus simple
    * Marche sous le dernier safari sous tiger
    * Marche ptet sous konqueror ou opera, a vous de me le dire
    * Fix d'un probleme qui faisait que on naviguait entre les mauvais commentaires dependant du seuil choisi (la, on ne navigue que entre les nouveaux commentaires au dessus du seuil, et ca marche vraiment :)
    * Des trucs cosmetiques, notamment j'ai vaguement retouché kaiska.css pour afficher mieux la toolbar

    Pour ceux que ca interesse, pour que ca marche sous safari j'ai du:
    * Changer les Node.prototype en Object.prototype
    * Changer document.documentElement.scrollTop en document.body.scrollTop (en fait j'ai mis les 2 au cas ou)
    * Faire quelques petites modifs cosmetiques
    • [^] # Re: Nouvelle Beta

      Posté par  . Évalué à 2.

      Allez zou, pour pas que tu te reposes sur tes lauriers, une autre petite suggestion :
      Quand on plie puis déplie un thread complet par double clic, ça serait sympa qu'il se restaure dans l'état où il était avant qu'on l'ait plié (au lieu d'avoir la totalité des commentaires qui sont dépliés).
      Bon ceci dit, de ce que je comprends du code (m'enfin j'suis vraiment pas fluent en JS), ça m'a pas l'air évident : là les commentaires ont deux états (déplié / plié), et il en faudrait trois (déplié / plié avec le thread / vraiment bien plié).

      Bon, c'est vraiment si t'as le temps/envie hein, c'est pas essentiel comme feature :)


      Hmmm... puis en y réflechissant, ça tuerait un peu une utilisation du double clic qui est de déplier complètement un gros troll bien moinsé sans changer son seuil, donc c'est p-ê pas terrible comme idée en fait... Bon ouais nan alors, j'ai rien dit.
      • [^] # Re: Nouvelle Beta

        Posté par  (site web personnel) . Évalué à 2.

        C'est la premiere chose a laquelle j'ai pensé en faisant ca, mais c'est chiant a faire, comme tu le dis il faudrait un autre état. ou alors tricher sur l'attribut class que je mets, en mettant genre 'replie old-was-replie' pour se souvenir de l'etat dans lequel le message était avant d'y toucher. Je vais voir :)
    • [^] # Re: Nouvelle Beta

      Posté par  (site web personnel) . Évalué à 2.

      Toujours pas sur konqueror 3.4.1 :'(
      • [^] # Re: Nouvelle Beta

        Posté par  (site web personnel) . Évalué à 2.

        Bon bah dans ce cas, c'est clairement plus un probleme de KHTML, vu que ca marche aussi dans safari (c'est le moment de profiter de l'opensourcage de webkit :-). Je serais interessé par un point de vue d'un développeur de KHTML sur l'histoire, en attendant je vais installer un kde quelquepart pour tester.
    • [^] # Re: Nouvelle Beta

      Posté par  (site web personnel) . Évalué à 2.

      Ah dans les modifs j'ai oublié le truc le plus important: Safari laissait les [script] dans le DOM, alors que visiblement gecko non.
      Je saurais pas trop dire qui a raison, mais du coup, j'ai du changer de technique pour changer le score: celui ci n'a pas d'id specifique, donc jusqu'ici je le recuperais parceque je savais que c'etait en gros le xieme element en partant du haut, 2eme a droite.
      La, j'ai fait un peu plus générique, ca parcourt tous les elements a partir du début, jusqu'a trouver celui contenant le score. Résultat, cette version marche mieux partout, mais est un poil plus lente. (enfin, ca doit rester de l'ordre de 3 fois plus rapide qu'avant, au moins sous gecko, qui a vue de nez est lui meme 3 fois plus rapide que safari :)

      (Dites le si ce que je dis est incomprehensible :)
    • [^] # Un bug

      Posté par  . Évalué à 1.

      Il y a un bug quand on ne saisit rien en seuil.

      Testé sous Firefox dernière version, que sur mon poste (ça vient p'tre de chez moi)

      Etape 1: Clicker sur seuil
      Etape 2: Valider, sans rien saisir
      Etape 3: Mettre le curseur sur la bar
      Etape 4: Magie ! Il n'y a plus de lien pour changer le seuil !

Suivre le flux des commentaires

Note : les commentaires appartiennent à celles et ceux qui les ont postés. Nous n’en sommes pas responsables.