les boutons de partage cassent les liens contenu dans le texte · Issue #246 · seenthis/seenthis_squelettes · GitHub

Je vote pour…

Mon minuscule petit doigt me susurre avec pincettes qu’il suppose que ça pourrait peut être éventuellement sans excès de confiance probablement se dérouler là : ▻https://github.com/seenthis/seenthis_squelettes/blob/master/noisettes/head_message.html#L22

Il semble y avoir eu un ticket pour les RSS aussi : ►https://github.com/seenthis/seenthis_squelettes/issues/195

Faut peut être ouvrir un ticket là bas ? @b_b zaussi peut savoir

Tu parles de quel compte?
►https://twitter.com/SeenthisNet ?

@marcimat merci pour le pointage, il y a un truc qui m’étonne dans ce code, la fonction supprimer_titre() ne semble accepter que deux argument et là on lui en passe trois oO (il semble donc qu’on limite à 256 caractères à ce jour et non 60)

@davduf quelle longueur souhaites-tu qu’on envoie sur twitter ?

Ah tiens tu as raison @b_b … je me suis arrêté sur ce code en supposant que ça coupait à 256 caractères, mais il n’en est rien ; tu peux regarder la source d’un message seenthis actuellement, la <meta name="twitter:description" ... /> est renseignée avec l’ensemble du texte (sans le titre)... Ça doit pas être ça du coup le souci.

Sur ▻https://developer.twitter.com/en/docs/tweets/optimize-with-cards/overview/markup ça indique que le champ est limité à 200 caractères. C’est peut être donc twitter qui limite, simplement.

@b_b j’aurais tendance à occuper tout l’espace possible (280 caractères) avec priorité sur le titre, et si possible placer la vignette du message, quand vignette il y a... merci ,))

La passerelle automatique vers Twitter n’est pas dans les métadonnées de chaque page, mais dans l’export conçu pour ça :
►https://seenthis.net/people/davduf/feed_tw
qui est lui-même passé par IFTT comme passerelle vers Touiteur.

Heu, je parle de twitter un Seenthis vers Twitter (l’option petit oiseau à droite des posts :-)

@marcimat haha bien vu tout le texte doublonné dans deux metas, encore un truc énergivore à nettoyer dans seenthis.

@davduf d’après la source citée par @marcimat twitter limite à 200 caractères en entrée, et à ce jour on envoie la complète, c’est donc bien chez eux que c’est limité, on n’y peut rien donc.

Reste à fixer le bug des metas gonflées dans seenthis, ça fera ça de moins pour la pachamama...

Et hop ▻https://github.com/seenthis/seenthis_squelettes/pull/244

Pour en revenir au sujet initial @davduf c’est déjà en place depuis 2017 cf ►https://github.com/seenthis/seenthis_squelettes/issues/195 ^^

Hé bien, @b_b teste ici même (le message tout en haut), tu verras que le tweet proposé fait... 139 caractères (et non 240) :-)

Bonjour @seenthis, tu crois qu’on pourrait avoir 140 signes de plus dans la passerelle #seenthis #twitter (quand… ▻http://seen.li/i3oc

C’est pas le texte du meta qui pose problème, c’est le lien hypertexte qui est dans le bouton de partage.

Ah tiens ! Comme j’ai toujours uBlock activé, je voyais pas ces boutons ; j’ignorais même leur existence !

Alors pour le coup, ça semble le JS là : ▻https://github.com/seenthis/seenthis_squelettes/blob/master/javascript/seenthis/seenthis_partage.js#L37

Ah, ba, nous v’la bien ,-) Je aprle bien du bouton Twitter à droite des messages...

Merci @marcimat c’est en place ▻https://github.com/seenthis/seenthis_squelettes/commit/b4c6edcebdb800c6891e7aa7b85d6d32d8400bd4 @davduf

Ah, super ! Mais désormais on a deux liens (quand le post SeenThis reprend un article publié ailleurs) : le lien d’origine et le lien SeenThis. Que diriez vous si on ne conservait (et si possible) que le lien SeenThis ?

Tu veux supprimer le lien qui pointe vers la source originale ? Si c’est ça, je ne pense pas que ça soit une bonne idée.

Je le supprimerais du tweet, oui, pour en garder que le lien SeenThis. Non ?

J’utilise fréquemment ce bouton, et, oui, je supprimerais la mention de l’url initial qui est souvent mass média, histoire de pas enrichir la ferme à clic pub (le fais fréquemment à la main, quitte à citer la source si besoin)... et pour garder de la place pour une accroche texte (d’autant que cela fait cliquer sur des paywall dans pas mal de cas, ce qui bloque la lecture). Mais peut-être est-ce compliqué techniquement, ou pas souhaitable selon d’autres ici. (si par exemple cela facilité l’arrivée de robots ici, comme hier à propos de m @ s q u € s)

Désolé de vous embêter à nouveau... Mais il semblerait que le lien d’origine soit cassé quand on tweete le post Seenthis.

Exemple : ▻https://twitter.com/davduf/status/1255605249630486528?s=20 + ▻https://twitter.com/MandadeleD/status/1255606908884574208

Je suis sûr qu’on va y arriver :-)

@davduf je le vois bien, et c’est un autre bug qui vient cette fois de la fonction texteArticle() ▻https://github.com/seenthis/seenthis_squelettes/blob/master/javascript/seenthis/seenthis_partage.js#L17 qui récupère le texte de l’article avant de générer le contenu à poster sur twitruc ou fb.

En effet, cette fonction récupère le texte du lien court affiché qui comme son nom l’indique est court... (et donc tronqué).

Bref, encore un bug qu’il faudra traiter dans un autre ticket cf ▻https://github.com/seenthis/seenthis_squelettes/issues/246

Je suis vraiment désolé ,-) Courage !

Quand j’essaie d’utiliser le bouton twitter :

403 Forbidden: The server understood the request, but is refusing to fulfill it.

Pareil pour vous ?

@davduf ça devrait être corrigé maintenant ;) (enfin une fois que le cache sera actualisé côté js)