Double lien urls lors du collage sur des balises bbcode `[url`

Continuant la discussion de Veuillez ne pas double-lier les URL si l’on colle par-dessus une URL en markdown :

Hier, cela a été corrigé sur mon site, mais ce matin, cela refait un double lien lorsque je veux coller par-dessus un lien existant.

Pouvez-vous reproduire cela sur meta ? Je me demande si vous avez trouvé un nouveau cas limite ?

1 « J'aime »

J’avais un lien qui commençait par « ## [url= \u003clink\u003e [/url] »

[## url=

Ensuite, quand j’ai essayé de copier/coller le lien, j’obtiens le résultat ci-dessus. Il le place complètement en dehors du code [url= [/url].

Et oui, je viens de faire ça ici sur Meta

1 « J'aime »

Oh, ce sont des liens BBCODE, nous ne les utilisons pas très couramment, c’est pourquoi nous n’avons pas créé d’heuristique pour cela.

Nous allons régler cela, peut-être que @gwwar peut envoyer une PR ?

Y a-t-il une raison pour laquelle vous privilégiez les liens BBCODE par rapport aux liens Markdown ? Principalement par habitude ?

Oui, plus de 10 ans d’habitude. J’ai essayé de changer, mais mes doigts n’écoutent pas.

1 « J'aime »

Bien sûr, je peux voir s’il existe une solution décente pour cela.

1 « J'aime »

Voyons, pour ma propre référence, les balises bb url et email fonctionnent comme suit. Elles peuvent également prendre en charge les liens magiques, où le protocole URI n’a pas besoin d’être spécifié.

[url=][/url]
[url=https://www.phpbb.com/]Visitez phpBB ![/url]
[url]https://www.phpbb.com/[/url]
[email]example@example[/email]

https://www.phpbb.com/community/help/bbcode

En termes de comportement, je pense qu’il serait raisonnable de tester si une sélection de texte contient une balise bbcode email ou url. Selon ce que les gens rencontrent, nous pourrions également élargir le test pour qu’il utilise le comportement de collage par défaut si nous détectons des balises bbcode.

3 « J'aime »

J’ai ajouté une PR ici. Faites-moi savoir si ce n’est pas le cas d’utilisation auquel les gens pensaient.

4 « J'aime »

Désolé de soulever ce point à nouveau, mais y a-t-il un calendrier pour la résolution de ce problème ? Je sais que cela prend parfois quelques jours, mais cela se produit toujours ce matin.

La PR a été approuvée, mais nécessite qu’un membre de l’équipe Discourse la fusionne. @eviltrout attendions-nous d’autres commentaires ?

@Slowhand puis-je juste vérifier, quelle partie de la balise [url mettez-vous en surbrillance avant de coller ? Si je comprends bien, je pense que la PR de @gwwar ne couvre que l’option 1 ci-dessous. Cela résoudra-t-il le problème pour vous ?


Option 1 :

<pre>
This is a sentence with <mark>[url=discourse.org]A link[/url]</mark>
</pre>

Option 2 :

<pre>
This is a sentence with [url=<mark>discourse.org</mark>]A link[/url]
</pre>

Option 3 :

<pre>
This is a sentence with [url=discourse.org]<mark>A link</mark>[/url]
</pre>

L’option 2 est celle avec laquelle j’ai des difficultés.

Pouvez-vous poster un exemple de point de départ ? Avec l’Option 2 exactement comme je l’ai posté ci-dessus, cela semble déjà bien fonctionner pour moi, même sans le nouveau PR.

discourse.org est détecté comme une URL existante, et donc le comportement de collage fantaisiste est contourné :thinking:

Pour l’option 2, nous avons des liens vers les promotions quotidiennes d’un site d’anime, donc je copie/colle généralement les liens des jours précédents dans un nouveau message, puis je les modifie, ce qui m’évite de taper tout ce code.

Exemple : ce matin, j’ai copié les offres d’hier

et j’ai collé les liens avec les offres d’aujourd’hui

ce qui a provoqué ceci :
image

Si je sélectionne ceci

[url=<mark>https://www.rightstufanime.com/Some-Link</mark>]Text[/url]

Et que je colle

https://www.rightstufanime.com/Deca-Dence-Blu-ray

J’obtiens :

[url=https://www.rightstufanime.com/Deca-Dence-Blu-ray]Text[/url]

Pourriez-vous voir si vous pouvez reproduire le problème que vous rencontrez ici sur Meta ? (notez que les balises [url ne s’affichent pas réellement ici, mais vous devriez toujours pouvoir essayer le comportement du compositeur)

D’après votre dernière capture d’écran, je pense que vous sélectionnez toute la ligne, y compris le point et l’enveloppe [url=blah]test[/url] ? (c’est-à-dire presque la même chose que « Option 1 » ci-dessus) ?

Je ne peux pas reproduire le comportement ici sur Meta.

Je ne pense pas sélectionner toute la ligne car je ne veux pas avoir à retaper tout le code. Je suppose que je pourrais accidentellement cliquer sur toute la ligne de temps en temps, mais j’ai été très prudent dernièrement à cause de ce problème.

Mais, laissez-moi vous demander ceci, si je sélectionne TOUT, est-ce le comportement approprié ? Au lieu de simplement insérer ce que je colle, cela l’ajoute à la fin et laisse ce que j’avais initialement sélectionné ?

Permettez-moi de m’excuser encore une fois de ne pas être très calé en technologie, c’est un peu au-delà de mes compétences.
Si j’ai vraiment besoin de modifier ma façon de faire, je le ferai.

1 « J'aime »

Ok, c’est super intéressant ! Juste pour confirmer, pourriez-vous le reproduire une fois de plus sur votre propre forum ? S’il y a une différence, veuillez nous en informer afin que nous puissions aller au fond des choses.

Je dirais « non », ce n’est pas le comportement approprié. Nous devrions absolument le corriger, et le PR de @gwwar le fait. Je voulais juste comprendre exactement quel était le cas d’utilisation ici pour m’assurer que nous avions tous les angles couverts.

Je vais fusionner ce PR maintenant, il devrait donc être déployé sur votre site dans les prochains jours @Slowhand

Merci d’avoir supporté mes questions ! :sweat_smile:

Non, même chose.

Merci beaucoup pour toute votre aide !

1 « J'aime »

Notez que l’option 2 devrait en théorie être gérée par

Si des utilisateurs rencontrent encore des problèmes avec des cas particuliers supplémentaires, il pourrait être judicieux d’ajouter un paramètre de site ou d’utilisateur pour activer ou désactiver le comportement.

3 « J'aime »

Ce sujet a été automatiquement fermé après 10 jours. Les nouvelles réponses ne sont plus autorisées.