We have the “downloaded local copies of images” setting enabled, and just realized that in some cases this causes the “edited” (pencil) icon to display at the top of the post.
In our experience the “edited” icon is useful for seeing at a glance when a user has actually changed the substance of a post.
For behind the scenes housekeeping stuff like making image URLs local, invisible to most users, I think this indicator should be suppressed. Maybe some cases where a system edit should be visible, but for this local image thing in particular, or similar invisible edits, seems ideal to not display it as “edited”.
Ah thanks Jay. Just checked and we actually do have that setting enabled already!
Maybe that setting isn’t being respected for showing the edit icon as well (though I think it should be). Example here, shows the edited / pencil icon for system edit:
Sorry, I was looking at the Oneboxed link here. When I went to your site I did see the revision. The videos themselves were highlighted to show something was changed in the data by the system. All text is identical - no changes. The system did something when downloading the video, but no edit pencil should be showing.
And yes, Troy has a shield after his name.
Je voulais faire un suivi car cela continue d’être une légère nuisance. Il semble très clair que les modifications système ne devraient pas afficher l’indicateur d’édition (icône de crayon). Particulièrement lorsque « désactiver les notifications de modification système » est activé, mais peut-être même dans tous les cas, indépendamment de ce paramètre.
Nous trouvons cela perturbant de voir cette icône orange « modifié » alors qu’il n’y a aucun changement réel de contenu, juste une réécriture d’URL en arrière-plan.
(Aussi, je ne suis pas sûr de savoir à quelle fréquence cela se produit, car ces modifications système ne semblent pas ajouter d’élément de journal dans Actions du personnel > « édition de message », sauf si je passe à côté de quelque chose !)
Équipe Discourse, avez-vous des avis ? Soutenez-vous la modification suggérée ici ?
Je pense que vous savez ce que je veux dire ! Oui, il y a un changement dans la diff littérale des messages bruts. Non, il n’y a aucun changement dans le contenu final affiché du message.
Si quelque chose entraîne un changement essentiellement invisible, il n’est pas utile d’y attirer l’attention. Ce n’est même pas une image différente, c’est simplement « cette image est maintenant hébergée à un autre endroit » — un changement sans signification pour tout lecteur, qui ne peut qu’ajouter de la confusion.
Je suis du même avis ! Je n’aime pas être attiré par l’icône de crayon orange, légèrement anxiogène, juste pour voir ceci :
À mon avis, nous ne devrions pas afficher le crayon du tout et gérer cela en interne (maintenance). Si un utilisateur s’intéresse aux URL, il peut apprendre automatiquement que Discourse récupérera les images en lien direct. Si cela représente un changement trop important, nous devrions au moins changer la couleur par défaut de ces icônes d’orange à gris, mais cela pourrait ne pas être facile à mettre en œuvre car les couleurs sont probablement déterminées par la date de la dernière modification
Attendez, l’intention n’a jamais été de désactiver la révision. Le paramètre concerne à 100 % la désactivation de la notification, comme suit :
Cela fonctionne, j’ai testé en local :
Si nous voulons étendre ce paramètre du site pour qu’il le soit, je suppose que nous pourrions ajouter :
hide system edit revisions : Masquer les révisions créées par le système lors du téléchargement d’images hotlinkées.
À vous de voir @codinghorror si vous voulez vous amuser avec cela, mais tel qu’il est, le crayon est à 100 % conçu ainsi ; ce qui est supprimé, c’est la notification que l’utilisateur reçoit concernant la révision.
Oh non, je voulais dire que je n’aimais pas la révision d’édition orange sur le message. Je peux confirmer que je n’ai pas été notifié, donc cela fonctionne comme prévu – désolé pour la confusion !
Ma demande ci-dessus et celles qui la précèdent concernent le comportement « masquer les révisions d’édition système » que vous avez mentionné, et elles n’ont rien à voir avec les notifications.
