[BBcode] Insérer une balise [url]

[Le Grand Duc]

Bonjour,

Lorsque dans l'éditeur on ajoute certaines balises, comme pour le gras ou l'italique, le texte en sélection dans le message est entouré par les balises d'ouverture et de fermeture.
Mais quand on veut entrer un lien, le texte sélectionné en toute logique devrait être reproduit automatiquement dans le champ « Description ».
Ou disons, si le champ description reste vide, le texte sélectionné devrait être conservé et entouré des balises.
Au lieu de ça il est effacé et remplacé par le texte du lien, ce n'est pas très pratique.

[Mat]

Totalement pour

J'avais remarque ceci il y a quelques semaines 

[Le Grand Duc]

La chose serait certainement possible, puisque si on va sur le site de l'éditeur (SCeditor), c'est exactement de la façon que j'ai décrite qu'il fonctionne à savoir, la zone de texte en sélection est entourée par les balises URL.
Si le champ description est rempli, celui-ci effacera et remplacera la zone sélectionnée.
La façon dont l'éditeur se comporte sur FA est donc due soit à un bogue, soit plus vraisemblablement à une ancienne version.

[Kallindra]

Perso je n'ai pas de soucis avec cette balise. Quand mon éditeur est bien sur le mode d'édition permettant ed voir les balises j'ai bien ceci :

Code:

[url="urltest"]Test[/url]

qui apparait dans mon éditeur.

J'ai donc voté contre car ce n'est, a priori pas un défaut venant de FA.

[Le Grand Duc]

Avez-vous entré "Test" dans le champ Insérer un lien ?
Si oui, tout est normal.
Le problème est quand vous avez Test en texte dans le message, vous le sélectionnez et vous entrez urltest dans champ URL. Une fois terminé, ce n'est pas le mot Test qui est entouré des balises url, c'est urltest qui l'a remplacé.
Ça fait la même chose dans ce forum ici, mais chez SCeditor, tout fonctionne normalement.

[Kallindra]

Non ce que je vous ai mis c'est pour pas vous mettre l'url que j'ai utilisé en vrai. Mais moi j'ai bien la balise url qui apparaît et pas le lien direct ^^

[Le Grand Duc]

Non ce que je vous ai mis c'est pour pas vous mettre l'url que j'ai utilisé en vrai. Mais moi j'ai bien la balise url qui apparaît et pas le lien direct ^^

... Après avoir sélectionné dans le texte déjà entré ce qui doit apparaître entre les balises, ou avoir entré le texte dans le champ Description ?
C'est important, c'est dans le premier cas seulement qu'il y a un problème.
Si vous n'avez pas de problème, c'est sans doute parce que votre forum utilise une version plus récente de l'éditeur.

[Kallindra]

Je viens de re-test :
Alors si je mets une url (appelons là urltest) + une description (appelons la test) :

Code:

[url=urltest]test[/url]

si je ne mets qu'une url (AKA urltest) et pas de description :

Code:

[url=urltest]urltest[/url]

Voilà ce que mon éditeur me sort ^^
Donc a priori les balises s'affichent bien dans les 2 cas pour moi.

[Mat]

Je viens de re-test :
Alors si je mets une url (appelons là urltest) + une description (appelons la test) :

Code:

[url=urltest]test[/url]

si je ne mets qu'une url (AKA urltest) et pas de description :

Code:

[url=urltest]urltest[/url]

Voilà ce que mon éditeur me sort ^^
Donc a priori les balises s'affichent bien dans les 2 cas pour moi.

Vous n'avez pas bien compris la suggestions je pense
Si j'écrit par exemple le mot test :

Test

Que je le surligne

Puis que je clique sur le bouton insérer un lien
On dois de nouveau taper la "Description"
Alors, on voudrait que la description sois automatiquement ajouter selon le mot qui est surligné

Je sais pas si je suis clair dans mes explications

[Kallindra]

Non mais là c'est du chipotage OO
La balise mise à disposition par FA remplit parfaitement son rôle.
Si vraiment c'est ça la suggestion je vote encore plus contre. Du coup cette suggestion ça fait vraiment suggestion "poil dans la main" pardonnez moi l'expression.

[Le Grand Duc]

Non mais là c'est du chipotage OO

Du chipotage ? Mais c'est comme ça que fonctionnent toutes les autres fonctions, que fonctionne la fonction URL sur le site de SCEditor et devrait fonctionner la fonction sur FA.
Voyons don !
Je pense que le problème est sans doute que notre version de SCEditor n'est pas à jour.
Est-ce qu'on a la possibilité de faire la mise à jour nous mêmes ?
Devrais-je reformuler une autre demande pour qu'on installe la dernière version de SCEditor ?

[Mat]

Je ne considère pas ça comme du chipotage, mais comme un ajout qui devrait déjà être présent depuis bien longtemps 

[Outis]

Totalement pour.
C'est d'autant plus gênant quand la description est composé d'une phrase +/- explicative sur l'objet du lien
source. Ou d'un texte avec plusieurs sources qu'on ajoute ultérieurement.
Tous les liens ne se composent pas d'un mot simple dans un texte simple.

Et c'est encore pire si on veut utiliser une image comme lien, qui est beaucoup moins évident à recopier.

[Mat]

Et c'est encore pire si on veut utiliser une image comme lien, qui est beaucoup moins évident à recopier.

Ha oui 

[Le Grand Duc]

Bon, j'ai trouvé comment corriger le code pour le rendre conforme :
1º éditon en mode Wysiwysi, dans le fichier sceditor-commands.js, commande $.sceditor.command.set("link", ... éliminer la source du problème :

Code:

        /* éliminer if(!description)
          {
          description=src // <=== Bug !!!
          }*/


C'est là que la description est remplacée par l'adresse si elle est vide.
Un peu plus bas :

Code:

          if(values['src'])
            {
            // remplacer editor.wysiwygEditorInsertHtml('<a href="'+values['src']+'">'+values['desc']+'</a>')
            // par :
            if(values['desc']) editor.wysiwygEditorInsertHtml('<a href="'+values['src']+'">'+values['desc']+'</a>')
            else editor.wysiwygEditorInsertHtml('<a href="'+values['src']+'">','</a>')
            }


C'est là que la partie de texte sélectionnée est remplacées par la description. L'idée c'est que si la description est vide, elle doit le rester, et ne par remplacer la zone sélectionnée qui est la vraie descripton.

2º En mode d'édition BBcode : dans le fichier sceditor-commands-bbcode.js, commande .set('link', {txtExec:function(caller) :

Code:

      if(values['src']&&values['src']!=="http://")
        // remplacer editor.insertText('[url='+values['src']+']'+values['desc']+'[/url]')
        // par
        if (values['desc']) editor.insertText('[url='+values['src']+']'+values['desc']+'[/url]');
        else editor.insertText('[url='+values['src']+']', '[/url]'); //


J'ai fait des essais en mode local, et tout fonctionne normalement.
Par contre, je n'ai pas encore trouvé un moyen pour corriger le problème simplement en ajoutant un script.
Ces fichiers ne sont pas les originaux de l'éditeur SCeditor mais ont été modifiés par FA pour les besoins spéciaux.
Alors je convertis ma proposition d'amélioration en requête pour corriger un problème.
À mon avis, il s'agit d'une erreur de conception, sinon un bogue.

[laBelleBella]

Ce n'est certainement pas du "chipotage" que de demander qu'une fonction fonctionne de façon conforme à toutes les autres.
Il semble que FA utilise une version modifiée de l'éditeur, et si la version d'origine fonctionne normalement, c'est manifestement que FA y a introduit le défaut.

[:?.cool.?:]

L'idée ne peut que simplifier et rendre la fonction plus "normale", je suis donc favorable .
Je ne vois pas comment on pourrait dire non alors que c'est déjà le cas un peu partout et même si il semble que nous puissons corriger ça nous même, tout le monde n'a pas les moyens de le faire.

[Le Grand Duc]

L'idée ne peut que simplifier et rendre la fonction plus "normale", je suis donc favorable .
Je ne vois pas comment on pourrait dire non alors que c'est déjà le cas un peu partout et même si il semble que nous puissons corriger ça nous même, tout le monde n'a pas les moyens de le faire.

Vous avez raison, mais je n'ai fait qu'identifier le problème dans le code de FA, je n'ai pu le corriger qu'en faisant tourner une version de l'éditeur copiée localement sur mon ordinateur. Je n'ai pas de solution pour le corriger dans les forums, c'est manifestement un bogue.

[Le Grand Duc]

Bon, j'ai fini par mettre au point une solution de secours permettant de faire fonctionner la balise URL normalement.
Mais ça reste une « patch », en attendant que le code de FA soit corrigé :

Code:

function corrigeURL(ev) 
  {
  if ($.sceditor)
    {
    $.sceditor.command.set("link",
      {
      _dropDown:function (editor,caller,handler)
        {
        var content=$('<div><label for="linksrc">'
          + editor._("URL:")
          + '</label> <input type="text" id="linksrc" class="url" placeholder="http://" value="" /></div><div><label for="linktitle">'
          + editor._("Description (optional):")
          + '</label> <input type="text" id="linktitle" value="" /></div><div><input type="button" class="button" value="'
          + editor._("Insert")+'" /></div>');
        content.find('.button').click
          (
          function(e)
            {
            var values=new Array(),
            src=content.find("#linksrc").val(),
            description=content.find("#linktitle").val();
            values['src']=src;
            values['desc']=description;
            if (editor.inSourceMode())
              {
              if (values['desc']) editor.insertText('[url='+values['src']+']'+values['desc']+'[/url]');
              else editor.insertText('[url='+values['src']+']', '[/url]');
              }
            else
              {
              if(values['desc']) editor.wysiwygEditorInsertHtml('<a href="'+values['src']+'">'+values['desc']+'</a>');
              else editor.wysiwygEditorInsertHtml('<a href="'+values['src']+'">','</a>');
              }
            editor.closeDropDown(true);
            e.preventDefault()
            }
          );
        editor.createDropDown(caller,"insertlink",content)
        }
      }
      ); 
    }
  }
function addEventHandler(elem,eventType,handler) {
  if (elem.addEventListener) elem.addEventListener (eventType,handler,false);
  else if (elem.attachEvent) elem.attachEvent ('on'+eventType,handler);}
addEventHandler(window, "load", corrigeURL);

[Outis]

Fonction intéressante, mais je n'ai pas compris où il faut l'installer. D'autres seront probablement comme moi.
De plus, cela fonctionnera-t-il avec toutes les version des forums ?

[Mat]

Fonction intéressante, mais je n'ai pas compris où il faut l'installer. D'autres seront probablement comme moi.
De plus, cela fonctionnera-t-il avec toutes les version des forums ?

Dans la "Gestion des javascripts"

Fonctionne parfaitement sur PhpBB2

[Cloo]

Je l'ai aussi installé sur mon forum test en phpBB3, ça semble très bien fonctionner!

Mais le mieux serait bien sûr que FA corrige ce bug, sans que nous ayons besoin d’installer ce module Javascript.

[Le Grand Duc]

Mais le mieux serait bien sûr que FA corrige ce bug, sans que nous ayons besoin d’installer ce module Javascript.

Tout à fait, J'ai identifié le problème dans cette suggestion, mais plus j'y pense et plus je suis convaincu qu'il s'agit plus d'une requête pour corriger un défaut qu'une suggestion d'amélioration.
C'est pourquoi j'ai déposé une requête pour corriger un problème ici.

[Outis]

Je suis d'accord que c'est une correction qui doit être faite par FA puisque tous les forums sont concernés.

[Le Grand Duc]

Je suis d'accord que c'est une correction qui doit être faite par FA puisque tous les forums sont concernés.

... reste à savoir combien de temps ça pourra prendre, on n'a pas vu beaucoup de réaction jusqu'à présent

[no_way]

Bonsoir, un grand merci à Le Grand Duc pour son codage (+ 1 cœur), un "patch" était déjà en cours de mon côté, il permet également de mettre dans les balises youtube l'url de partage d'une vidéo faisant partie d'une liste voir suggestion :

Code:

/* FA7UP>SCEditor - patch (ver 0.1): */
$(function(){
    if(!$.sceditor)return;
    /*
    * Linking
    */
    var link_commandSet={
    _dropDown:function(editor,caller,handler){
        var values=[],content=$('<div><label for="linksrc">'+editor._("URL:")+'</label> <input type="text" id="linksrc" class="url" placeholder="http://" value="" /></div><div><label for="linktitle">'+editor._("Description (optional):")+'</label> <input type="text" id="linktitle" value="" /></div><div><input type="button" class="button" value="'+editor._("Insert")+'" /></div>');
        /*tentative de récupération de la sélection*/
        selection=null;
        if(editor.inSourceMode()){
            if((sNode=window.getSelection())&&(sNode=sNode.anchorNode)&&(sNode=sNode.children[2])&&(sdirection=sNode.selectionDirection)&&sNode.selectionStart!=sNode.selectionEnd)selection=editor.val().substring(sNode.selectionStart,sNode.selectionEnd);
        }else{
            r=editor.getRangeHelper().selectedRange();
            if(r&&r.startContainer===r.commonAncestorContainer){
                selection=r.commonAncestorContainer.hasChildren?null:r.toString();
            }
        }
        content.find("#linktitle").val(selection);
        /*fin*/
        content.find('.button').click(function(e){
        e.preventDefault();
        values['src']=$('#linksrc').val();
        values['desc']=$('#linktitle').val();
        handler(values);
        editor.closeDropDown(true)
        });
        editor.createDropDown(caller,"insertlink",content)
    },
    tooltip:"Insert a link"
    };
    link_commandSet.txtExec=link_commandSet.exec=function(caller){
        var editor=this;
        $.sceditor.command.get('link')._dropDown(editor,caller,function(v){
            if(v['src']){
                range=editor.getRangeHelper().selectedRange();
                v['desc']?editor.insert('[url='+v['src']+']'+v['desc']+'[/url]'):(range.collapsed?editor.insert('[url='+v['src']+']'+v['src']+'[/url]'):editor.insert('[url='+v['src']+']','[/url]'));
            }
        })
    };
    $.sceditor.command.set("link",link_commandSet);
    /*
    * Youtube
    */
    $.sceditor.command.set("youtube",{
    _dropDown:function(editor,caller,handleIdFunc){
        var matches,content=$('<div><div><label for="youtubeurl">'+editor._("URL:")+'</label> <input type="text" id="youtubeurl" class="url" placeholder="http://" value="" /></div><div><input type="button" value="'+editor._("Insert")+'" class="button"></div>');
        content.find('.button').click(function(e){
        var val=content.find("#youtubeurl").val().replace(/^https?:\/\//i,"");
        if(val!==""){
            matches=val.match(/(?:youtu\.be|youtube\.com\/embed)\/([0-9a-z]{11}\?list=.+)$/i);
            if(matches){
                handleIdFunc(matches[1]);
            }else{
                matches=val.match(/(?:v=|v\/|embed\/|youtu.be\/)(.{11})/);
                if(matches)val=matches[1];
                if(/^[a-zA-Z0-9_\-]{11}$/.test(val)){
                    handleIdFunc(val)
                }else{
                    alert(editor._('Invalid YouTube video'))
                }
            }
        }
        editor.closeDropDown(true);
        e.preventDefault()});
        editor.createDropDown(caller,"insertlink",content)},
        exec:function(caller){
        var editor=this;
        $.sceditor.command.get('youtube')._dropDown(editor,caller,function(id){
            editor.insert('[youtube]'+id+'[/youtube]')}
        )},
        tooltip:"Insert a YouTube video"
    });
});
/* FA7UP_fin */


Il ne supporte pas les anciens navigateurs par contre et lorsque vous êtes en mode source le texte sélectionné étant versé dans le champ description, il est assez aisé de faire n'importe quoi ...

Il s'agit là d'un code clairement améliorable mais ça peut peut-être dépanner en attendant que le comportement soit vraiment corrigé.

[Mat]

Bonsoir, un grand merci à Le Grand Duc pour son codage (+ 1 cœur), un "patch" était déjà en cours de mon côté, il permet également de mettre dans les balises youtube l'url de partage d'une vidéo faisant partie d'une liste voir suggestion .

C'est super gentil
Je vais le tester 

[Pinguino]

Bonjour,

La suggestion a été développée et est à présent en ligne sur tous les forums Forumactif

Je déplace donc ce sujet.