Mathieu Pillard a écrit 2873 commentaires

Pour naviguer entre les nouveaux commentaires, je trouve quand meme la nouvelle methode bien plus propre. (Ya meme un raccourci clavier fait pour, pas besoin de rechercher, c'est quand meme plus pratique non ?)

Pour la couleur, j'ai pas trouvé mieux. L'idee etait d'avoir un truc moins moche et tout de meme simple a remarquer.

Avec safari tu n'aurais rien du tout alors bon :)

Parceque j'ai pas eu le temps de bosser pour des hacks pour que ca marche pour konqueror, donc la toolbar est desactivée chez lui. Idem pour safari et IE. En fait tout ce qui ne supporte pas getComputedStyle (gecko et opera le supportent)

Ce qu'il manque surtout, c'est une pref pour desactiver la toolbar, qui fait plusieurs tours de la page et qui devrait pas. Bref, patience, ca va s'ameliorer, et yaura bientot une pref aussi.

En l'occurence, ya pas grand chose, et c'est pour faire une manip temporaire en attendant mieux, rien de plus.

En fait c'est pas un bug, c'est une feature qui a mal tourné. (l'idee est de laisser ceux qui ont desactivé js avoir la feuille de style). Patience, ma toolbar peut facilement laisser le choix, mais rome ne s'est pas faite en un jour.

Non il n'y a pas bug, ce n'est pas une feuille de style utilisateur au sens propre du terme.
Sinon, le fichier est encore a part pour diverses raisons, en attendant, tu peux utiliser !important.

Oui, et c'est "normal". Lire la demi douzaine de journaux a ce sujet.

Cela dit, c'est interessant, parceque une procédure est prevue, ils peuvent pas se refugier derriere les arguments classiques du genre "c'est pas nous que ca regarde", etc...

J'ai _peut etre_ un fix pour ce probleme.
Au lieu de
var styleURL = getcookie("style", "css/style.css");
add_stylesheet(styleURL, 1, 0);

Mettre:

var styleURL = getcookie("style", "css/style.css");
// hack pour que ca marche dans khtml
document.write('<link rel="stylesheet" type="text/css" href="' + styleURL + '" id="csspersolink">');

Pas testé, c'est moche, mais je pense que ca devrait resoudre le probleme. Sinon, on pourrait aussi appliquer le style onload, mais ca impliquerait de laisser la page sans style pendant qu'elle se charge, ce qui est moche. (le probleme de konqueror vient du fait que son arbre dom n'est pas construit et que on essaye d'ajouter un element dedans, et ca, il aime pas khtml)

En fait avant je les laissais mais on m'a fait remarqué que ca servait a rien de voir des commentaires comme nouveaux si ils sont en dessous du pallier...

Ca peut se patcher facilement, ya meme le cas de prevu pour la feuille de style,
la toolbar leur affecte la classe "nouveaudessous". Donc eventuellement tu peux leur affecter un style avec une feuille alternative, ou en poster une meilleure, ou trouver un compromis qui satisfasse tout le monde...

Tu penses bien que si c'etait juste ca le probleme ca serait trop simple... La toolbar ne se resume pas a 3 lignes de codes pour planquer ou non un div.
Par ailleurs ton code est assez crade :)

C'est lié. Me demande pas pourquoi par contre.

Sinon pour forcer l'activation, il faut obliger konqueror a repondre autre chose que false ou undefined au test sur window.getComputedStyle, ou demander a ce que le script soit changé :)

- replier je trouve ca bien, et replier tout un thread oui, mais ca serait couteux, et je vois pas tres bien ou caser un bouton pour :)
- tu peux faire ca en 2 touches: soit en recherchant > ou <, soit avec les acceskey sur la meme touche, avec alt+ > ou < donc.
- normalement il est stocké dans un cookie, je n'ai pas cherché d'ou venait le probleme mais a priori il vient pas de mon code (quoique :)

Il gere un peu le DOM, mais pas suffisamment. Patches bienvenus, je n'ai pas de IE sous la main pour tester.

Tu peux utiliser !important dans ta css non ?
Le probleme de l'integration auto c'est que quand on pourra desactiver la toolbar, on se chargera quand meme la feuille de style, ce qui fait quelques ko en plus... comment ca je chipote ? :)

Ce n'est pas juste un probleme de position de la toolbar.
C'est un probleme du au fait que khtml fait n'importe quoi avec le DOM.

Oui, c'est du au position:fixed, et je n'ai pas tellement de solution. C'etait pire quand j'avais mis la toolbar en haut :)

C'est le genre de commentaires que j'aurais bien aimé voir avant, quand je faisais mes tests :)

Je vais tenter de trouver un moyen pour que ca perturbe pas l'historique. En attendant, tu peux trouver toi meme comment faire une feuille de style alternative qui ne cache pas ces horribles !!N!! :)

Tout est possible... mais bon, ya aussi le temps, la motivation, les competences des gens qui rentrent en ligne de compte...

http://linuxfr.org/comments/405037,1.html(...) , encore plus rapide, pas a appuyer sur entrée :)

Bah, la feuille de style de base de linuxfr est _tres_ chargée, inutilement d'ailleurs, donc ca m'etonne pas trop. Je suis quand meme surpris que la différence soit si flagrante, mais pareil que pour le reste, j'ai pas le temps de trop regarder ca la.

Tu peux changer la couleur du truc en quelque chose de plus voyant, voir carrement remettre les !!!N!!! avec juste quelques lignes de CSS. J'ai pas trop le temps de faire ca en ce moment, donc faudra trouver tout seul ce genre de choses ou prier pour que quelqu'un de l'equipe linuxfr l'implemente :)

- Pour le envoyer, c'est fastoche, je le ferais si jamais CMoi a besoin d'aide la dessus.
- Pour la feuille de style, ca peut etre bien pratique de l'afficher, si tu n'en veux pas... propose une feuille de style alternative le cachant :)

Je te conseille fortement de changer de navigateur, mozilla 1.0 est vieux, buggé par rapport a ce qui se fait, et niveau secu on a mieux chez mozilla.org. Notamment ma toolbar use et abuse de plein de trucs avec lesquels il a du mal :)