Balises et safari - Page 1

J'ai un petit problème : jsuis sous safari, et lorsque je clique sur l'une des balises sur le côté droit (url, img, etc...), rien ne se passe ... Quelqu'un a une idée d'où ça peut venir ?

Oui, moi ... de safari ^^

tsss

Aucun moyen de résoudre ça (outre utiliser firefox), donc ? :/

utiliser IE

Utiliser Konqueror.
Et oui, sous Konqueror ça marche, on se demande ce qu'ils ont fait avec KHTML chez Apple pour que ça ne marche plus.

...

kim (./7) :
...

pas mieux

Et c'est vraiment chiant à corriger ?

Corriger un bug que je n'ai pas et que j'ai pas moyen d'avoir ... oui c'est chiant Surtout que c'est pour une minorité mineure d'une minorité de personnes

yAro: tu as aucune excuse, safari est dispo sous windows

Mais a-t-il le même bug ?

En tout cas, le bug reste sur la dernière nightly de Safari :/

(oui il doit avoir le meme "bug" c'est lié au moteur, par au rendu ça...) Et a mon avis c'est lié a la gestion de la DOM plus qu'autre chose. L'inclusion dans un champs texte, du moins de la facon dont ça avait été fait sur yN il y a quelque année est plus du hack que de la manipulation de la DOM propre.

C'était surtout la seule méthode compatible avec tous les navigateurs du moment ...

C'est clair qu'à ce niveau, la gestion des différents navigateurs doit être une horreur... pas un qui ait une implémentation correcte du JS :/

Faut voir, avec le DHTML ça doit commence (pour ce genre de choses) a se standardiser un peu plus....

(oupa)

Elle fonctionne la fonction que j'utilise sur http://prettyprint.free.fr ? Je n'ai jamais testé sous Safari.

Thepro > non

Le probleme, c'est que vous testez (toi et yAro) "Konqueror" dans le UserAgent, mais safari ne s'affiche pas comme konqueror, et le code qui va etre utilisé est le hack fait pour IE (createRange toussa)

Tu penses qu'il suffirait de changer juste ça ?
yAro > si tu as la flemme de changer, j'ai ça comme User Agent : Mozilla/5.0 (Macintosh; U; Intel Mac OS X; fr-fr) AppleWebKit/523.6 (KHTML, like Gecko) Version/3.0.3 Safari/523.6

Ben au pire deja ça insererais les balises a la fin du post comme le fait yAronet pour konqueror. je sais pas si la fonction mozWrap pourrais marcher sur Safari (a tester)

edit: apriori oui safari supporte document.getSelection (ou window.getSelection). Donc le code de mozWrap devrais marcher

o, tu peut mettre dans ton premier test de la fonction "fonc" ça corrigerais le probleme, au moins pour que le fallback du pire des cas marche (a savoir prettyprint_textarea.value += a + b + cje peut pas tester en ligne, mais thepr) mais apriori c'est mozWrap qui est pris en compte.

if ((navigator.userAgent.indexOf("Konqueror") > -1) || (navigator.userAgent.indexOf("WebKit") > -1))

edit mieux :

Voila comment améliorer fonc :

function fonc(a,b,c) {
   var prettyprint_textarea = document.getElementById("textarea-equ");
   if (navigator.userAgent.indexOf("Konqueror") > -1) {
         if (document.getSelection) {
            mozWrap(prettyprint_textarea, a, c, b);
         } else
            prettyprint_textarea.value += a + b + c;
   } else if ((document.getSelection) || (window.getSelection)) {
         mozWrap(prettyprint_textarea, a, c, b);
   } else {
         var strsel = document.selection.createRange().text;
         if (strsel == "") str = b;
         else str = strsel;
         prettyprint_textarea.focus();
         var sel = document.selection.createRange();
         str = a + str + c
         sel.text = str;
         if (strsel != "") {
            sel.moveStart('character', -str.length);
            sel.select();
         }
   }
   prettyprint_textarea.focus();
}

et je pige pas pourquoi IE est en fallback, ça devrait plutot etre un truc genre :

function fonc(a,b,c)
{
   var prettyprint_textarea = document.getElementById("textarea-equ");
   if (navigator.userAgent.indexOf("IE") > -1) // Should not work as-is
   {
      var strsel = document.selection.createRange().text;

      if (strsel == "") 
         str = b;
      else 
         str = strsel;

      prettyprint_textarea.focus();

      var sel = document.selection.createRange();
      str = a + str + c
      sel.text = str;
      if (strsel != "")
      {
         sel.moveStart('character', -str.length);
         sel.select();
      }
   }
   else if ((document.getSelection) || (window.getSelection))
   {
         mozWrap(prettyprint_textarea, a, c, b);
   }
   else 
   {
      prettyprint_textarea.value += a + b + c;
   }
   prettyprint_textarea.focus();
}

Godzil (./19) :
Le probleme, c'est que vous testez (toi et yAro) "Konqueror" dans le UserAgent, mais safari ne s'affiche pas comme konqueror, et le code qui va etre utilisé est le hack fait pour IE (createRange toussa)

Donc en fait, la solution, c'est s/Konqueror/KHTML/g tout simplement?

Godzil (./21) :
et je pige pas pourquoi IE est en fallback, ça devrait plutot etre un truc genre :

C'est vrai, je viens de changer

maintenant ça marche

Cool

Avec un peu de chance, ça sera assez facile sur yAronet également

C'est à peu près la même fonction pour yAronet : v31/gfx/js/scripts2.js
Donc oui, c'est facile à corriger

\o/

Ce n'est pas la peine de modifier tout ça, il suffit de remplacer "Konqueror" par "KHTML", qui est dans la chaîne UA de Konqueror et de Safari.

Oui mais non. Je vois pas l'interet de tester konqueror, il n'y a que IE a tester et sinon tester si le navigateur propose window.getSelection ou document.getSelection, si cela marche on utilise mozWrap sinon si IE on utilise la technique a IE, sinon le fallback qui fait que ajouter a la fin du texte. C'est ce qui se passe que Konqueror actuellement, et sachant que safari supporte mieux que ça, j'en vois pas l'interet.