[PAYÉ] Corrections de bugs et améliorations de Discourse Chat

Je cherche de l’aide pour 4 problèmes concernant Discourse Chat. Si vous pensez pouvoir aider avec l’un d’entre eux, veuillez me contacter :

1. Bug : Impossible de sélectionner du texte dans le compositeur de chat sur iOS

Dans la fenêtre du compositeur de chat, si vous tapez un message et sélectionnez du texte, un mot sera mis en surbrillance mais vous ne pourrez pas ajuster votre sélection en déplaçant les points aux extrémités du mot.

2. Bug : Les miniatures vidéo n’apparaissent pas dans le chat sur iOS

Sur iOS, lorsque des fichiers MP4 ou d’autres fichiers vidéo sont téléchargés, aucune miniature n’apparaît.

3. Fonctionnalité : Canaux de chat en lecture seule

Dans les paramètres du canal, créez un nouveau paramètre pour marquer un canal comme « Lecture seule ». Une fois défini, seul le personnel pourra publier dans le canal.

Si possible, il serait bon de restreindre la publication tout en permettant à quiconque dans le canal de répondre à la publication dans un fil de discussion.

4. Fonctionnalité : Désactiver les e-mails lorsque @all est utilisé sur le chat

Lorsque @all est utilisé, nous voulons empêcher l’envoi d’un e-mail, mais nous souhaitons que les notifications push continuent de fonctionner comme elles le font.


Le budget pour ces éléments en dollars américains est négociable.

6 « J'aime »

Envoi d’un message privé :+1:

7 « J'aime »

Je vais travailler sur cela dans les prochaines semaines, nous avons un problème similaire dans les sujets également.

10 « J'aime »

C’est une excellente nouvelle et merci beaucoup. Nous avons entendu parler de ce bug par quelques-uns de nos utilisateurs et ce serait formidable de le corriger pour tous les utilisateurs de Discourse.

2 « J'aime »

@j.jaffeux, il me semble qu’il s’agit d’un bug dans le navigateur Safari ?

Lorsqu’il est ouvert dans Firefox ou Chrome, avec la même pièce jointe vidéo, la balise vidéo s’affiche correctement avec « un aperçu », êtes-vous d’accord ? (entre guillemets car je ne pense pas que cela dépende du traitement des miniatures de l’application Discourse).

Cela suggère qu’il n’y a rien que nous puissions faire côté application ?

1 « J'aime »

@blake en sait beaucoup plus que moi sur ce sujet

2 « J'aime »

L’approche discutée ici est-elle une option ?

Vous pouvez le voir en action sur cette page si vous la chargez sur votre iPhone dans Safari :

2 « J'aime »

Je note que le problème ne se produit que dans le chat, qui utilise la balise HTML <video> avant la lecture.

Sur les Posts, où l’aperçu est correctement rendu sur Safari, Discourse semble utiliser un <div class="video-placeholder-container">… et un faux espace réservé pour l’affiche vidéo avant que vous ne cliquiez, moment auquel il passe à la version <video>.

Peut-être y avait-il une raison à cela…

… oui, il y en avait une :

Donc justifié pour une raison différente ? Faire d’une pierre deux coups ?

S’il y a de l’intérêt, je suis prêt à proposer une version de l’approche Post ?

3 « J'aime »

Pour information, nous n’avons jamais implémenté de placeholders vidéo pour le chat, il s’appuie simplement sur les fonctionnalités de la balise vidéo HTML5 pour fonctionner. Nous avons implémenté notre propre logique de placeholder à l’origine pour les sujets/messages car les vidéos “téléchargeaient” automatiquement des données à chaque chargement de sujet, même si elles n’étaient pas lues, ce qui augmentait les coûts CDN sur les sites populaires.

Je ne pense pas que nous ayons besoin de réimplémenter la même logique de placeholder que nous avons pour les sujets dans le chat pour le moment, mais il se peut qu’il y ait encore quelques ajustements iOS/Safari que nous devrons faire pour que les choses fonctionnent sur iOS. Je pensais que le simple fait d’avoir l’attribut preload="metadata" suffirait, mais il semble qu’il soit déjà présent.

<video class="chat-video-upload" preload="metadata" height="150" controls="">
        <source src="https://actual-url.MOV">
      </video>
7 « J'aime »

Cette correction sera apportée par ce commit :

Notez qu’il y a une chance que je doive annuler cela et trouver une solution différente, mais au moins je connais maintenant la cause profonde.

11 « J'aime »

Des nouvelles à ce sujet ?

3 « J'aime »

Salut @blake - je voulais aussi faire un suivi sur celui-ci. Dans la mesure où cela a déjà été corrigé ailleurs et que cela peut être appliqué au chat, ce serait formidable.

Nous avons beaucoup de personnes qui partagent des vidéos dans notre communauté et, lorsqu’il n’y a pas de miniatures, de nombreux utilisateurs pensent que la fonctionnalité vidéo est cassée car sur toutes les autres plateformes (SMS, Slack, Discord, etc.), vous voyez toujours une miniature lorsqu’une vidéo est publiée.

Toute aide est très appréciée.

3 « J'aime »

Voyons si cela fonctionne

4 « J'aime »

:partying_face: Ça s’annonce très, très bien !

4 « J'aime »

Merci beaucoup @blake. Très apprécié. Je vais vérifier ça.

3 « J'aime »