112

Append attachments to comments by flamingo-cant-draw ?? Pull Request #5143 ?? LemmyNet/lemmy

top 17 comments
sorted by: hot top controversial new old
[-] cabbage@piefed.social 37 points 2 days ago

So when images are attached to comments from Mbin (without markdown), they will finally show up in Lemmy? Is that what this means?

[-] KLISHDFSDF@lemmy.ml 19 points 2 days ago

yeah, someone dumb it down for us plebs

[-] Aatube@kbin.melroy.org 20 points 2 days ago

basically what cabbage said. same goes for mastodon posts in subs

theres a manual fix for this. in mbin, when you attach the image in a post and save, it is uploaded and shown. you can then grab the image link (right click image, copy link), edit the post, and add the link using the image tag, and save, it then shows in-line in lemmy.

yeah, it sucks but at least it works until this is implemented. ive been usin this method for over a year now..

[-] DarkThoughts@fedia.io 3 points 2 days ago

Speaking of which... When can we finally ctrl+v paste them in from our clipboard? :(

[-] Aatube@kbin.melroy.org 3 points 2 days ago

speaking of which... when will the "upload from URL" button actually work‽
oh wait, they removed it?

...yay?

[-] DarkThoughts@fedia.io 2 points 1 day ago

Upload from URL is pointless since you can just markdown that URL directly.

[-] Aatube@kbin.melroy.org 1 points 1 day ago
[-] DarkThoughts@fedia.io 2 points 1 day ago

Even then pasting would also solve that issue since you can copy images from websites.

[-] Blaze@feddit.org 15 points 2 days ago
[-] andrew_s@piefed.social 6 points 2 days ago

Do they work the other way around btw? If someone on Lemmy uses the Markdown for an inline image do they show up on MBIN? I don't they do on Mastodon.

[-] Aatube@kbin.melroy.org 5 points 2 days ago

This PR doesn't add that, but anecdotally I'm fairly sure it already works for Mastodon... Not a Mastodon user, so I can't find any examples though. They definitely show on Mbin tho.

[-] andrew_s@piefed.social 8 points 2 days ago* (last edited 2 days ago)

I just tried with Masto - maybe there's different versions, but it didn't work with the one I tried.

Screenshot:

It's probably for the best that this PR doesn't also convert inline Markdown into an attachment to send out for Mastodon's benefit, because then there would be the danger of apps that understand both showing two images. It'd be better if Mastodon did the translation when receiving stuff, but Mastodon doesn't seem as good as MBIN when it comes to co-operating with Lemmy.

(edit: how that screenshot shows on MBIN is a bit disappointing though - at least looking at on the web)

[-] flamingos@feddit.uk 4 points 2 days ago* (last edited 2 days ago)

This is Mastodon's HTML sanitiser, you can see they stipe out <img> tags.

How does Piefed handle image attachments, btw?

[-] andrew_s@piefed.social 3 points 2 days ago

How does Piefed handle image attachments, btw?

For comments: not at all. If a Mastodon user tried to do what I did, with the inline image, nothing would show.

We could do what I think you've done, and regex the details of the attachment into ! [] () Markdown and add it to the text. There's also a DB relationship between comments and images that isn't used, but could be, I suppose.

I've never actually seen a Mastodon user try to add an image to something that ended up as a Lemmy comment, tbh, so it's not something I've thought too much about.

[-] flamingos@feddit.uk 4 points 2 days ago* (last edited 2 days ago)

We could do what I think you’ve done, and regex the details of the attachment into ! [] ()

To be clear, this pull request doesn't use regex, it's just JSON deserialisation and string interpolation.

I’ve never actually seen a Mastodon user try to add an image to something that ended up as a Lemmy comment, tbh, so it’s not something I’ve thought too much about.

The pull request actually includes one, the main KDE account tags !kde@lemmy.kde.social and includes pictures in their threads regularly. It's just hard to tell from our side as you can't see what's missing.

[-] andrew_s@piefed.social 4 points 2 days ago

Ah, great, thank you. It's been added as an Issue for PF now, with a link to this post, so that'll be handy.

(I was likely misusing the term 'regex').

this post was submitted on 02 Nov 2024
112 points (100.0% liked)

Fediverse

28220 readers
147 users here now

A community to talk about the Fediverse and all it's related services using ActivityPub (Mastodon, Lemmy, KBin, etc).

If you wanted to get help with moderating your own community then head over to !moderators@lemmy.world!

Rules

Learn more at these websites: Join The Fediverse Wiki, Fediverse.info, Wikipedia Page, The Federation Info (Stats), FediDB (Stats), Sub Rehab (Reddit Migration), Search Lemmy

founded 1 year ago
MODERATORS