Demande de fonctionnalité : lien vers le titre et non le corps lors du partage sur discourse PWA

Ce serait vraiment bien si les liens étaient partagés vers le titre au lieu de (juste) le corps du message, de sorte que le titre soit rempli automatiquement comme il le fait si vous le faites sur un site.

Y a-t-il une raison pour laquelle le corps du message est la cible ?

1 « J'aime »

Accepteriez-vous une demande de fusion ici ou y a-t-il une raison pour laquelle cela a été fait de cette façon ?

@Falco, quelle demande de fusion a introduit cette fonctionnalité (j’ai regardé mais je n’ai pas pu la retrouver) ?

C’était il y a trèèèèès longtemps, mais cela ne va-t-il pas casser lorsque vous ne partagez pas un lien mais un contenu textuel ?

Au moins, vous devriez détecter si le texte partagé est un lien et l’envoyer au bon champ.

1 « J'aime »

Oui, c’est sans doute plus difficile car vous ne faites pas que récupérer une URL, par exemple https://blah.com/new-topic? ... ?

Mais je suis heureux d’y jeter un œil.

1 « J'aime »

Si la plupart des sites n’envoient plus qu’une URL de nos jours, je serais content avec ce changement.

1 « J'aime »

J’ai trouvé le code :detective: et je vais voir si nous pouvons gérer les deux… mais je suis d’accord, les URL sont probablement envoyées 99 % du temps, donc cela vaut la peine d’y réfléchir…

1 « J'aime »

OK, j’ai trouvé comment prendre en charge les deux :

  • les partages de texte iront dans le corps
  • les titres iront dans le titre
  • les partages de liens iront dans le titre (et seront ensuite rendus comme un titre entièrement développé et dans le corps une jolie onebox lors de l’instanciation avec tout ce qui l’accompagne, y compris les aperçus de la liste des sujets)

Actuellement, la mise à jour est dans un plugin fonctionnel :

Je soumettrai cela comme une PR car c’est vraiment une évidence.

J’ai testé cela sur les PWA Android et Windows (Edge) et cela semble fonctionner parfaitement.

1 « J'aime »

La PR est ici :

Petit rappel @Falco

2 « J'aime »

Fusionné, merci pour la PR
bsp!

2 « J'aime »

merci pour la revue !

2 « J'aime »

Confirmé que cela fonctionne sur la dernière reconstruction

Une autre raison de considérer Android plutôt qu’iOS pour votre prochain téléphone :mobile_phone: si vous êtes accro à Discourse-land :sweat_smile:

Je pense que nous pouvons marquer cela comme Terminé ?

3 « J'aime »