this post was submitted on 09 May 2024
90 points (93.3% liked)

Technology

59630 readers
3989 users here now

This is a most excellent place for technology news and articles.


Our Rules


  1. Follow the lemmy.world rules.
  2. Only tech related content.
  3. Be excellent to each another!
  4. Mod approved content bots can post up to 10 articles per day.
  5. Threads asking for personal tech support may be deleted.
  6. Politics threads may be removed.
  7. No memes allowed as posts, OK to post as comments.
  8. Only approved bots from the list below, to ask if your bot can be added please contact us.
  9. Check for duplicates before posting, duplicates may be removed

Approved Bots


founded 1 year ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
[–] [email protected] 1 points 6 months ago (4 children)

Thanks! I'll go ask Tiff about getting reddthat updated later.

BTW, is there a community for discussion of mlmym itself somewhere on lemmy? I can't participate on GitHub, but those aren't the only issues I've found. (e.g. there's also ?format=jpg&thumbnail=96 on non-pictrs links and a text handling issue with angle brackets...)

[–] nnrx 2 points 6 months ago* (last edited 6 months ago) (3 children)

no lemmy community for mlmym yet, that's good idea.

do you have an example of that thumbnail issue?

[–] [email protected] 1 points 6 months ago* (last edited 6 months ago) (1 children)

I ran into an example of the thumbnail issue again today -- this time on a post from kbin: https://old.reddthat.com/post/19193476

The thumbnail looks like this in the HTML:

<div class="thumb">
  <a class="url"
     href="https://media.kbin.social/media/60/a4/60a45b8ff88b1b2e3a0f77b701feb323c5bbfb7ceeb75154ea7df5d6eea15ef8.jpg"
     >
    <div  style="background-image: url(https://media.kbin.social/media/60/a4/60a45b8ff88b1b2e3a0f77b701feb323c5bbfb7ceeb75154ea7df5d6eea15ef8.jpg?format=jpg&amp;thumbnail=96)"></div>
  </a>
</div>

Note that it's making a request to kbin.social with ?format=jpg&thumbnail=96 parameters in the CSS -- which results in the full image being loaded since kbin doesn't run pictrs.

The versions in use on reddthat (according to the settings page) are:

lemmy: 0.19.4-beta.7

mlmym: 0.0.44

[–] nnrx 2 points 6 months ago

This kinda looks like an issue between lemmy and kbin. The issue is present on the lemmy ui as well. The ?format=jpg&thumbnail=96 parameter being there doesn't have an effect here. The issue is the thumbnail_url is reported wrong. You can see that here. The thumbnail_url should be: https://kbin.social/media/cache/resolve/entry_thumb/60/a4/60a45b8ff88b1b2e3a0f77b701feb323c5bbfb7ceeb75154ea7df5d6eea15ef8.jpg We could try and use some regex to lookout for kbin images and rewrite the thumbnail_url, but it should probably just be fixed upstream.

Found this github issue which might be related: https://github.com/LemmyNet/lemmy/issues/3775

load more comments (1 replies)
load more comments (1 replies)