FixTweet icon indicating copy to clipboard operation
FixTweet copied to clipboard
verified publisher icon dangeredwolf

Mosaic behavior in readme inverted from production behavior

Open ZenithRogue opened this issue 8 months ago • 1 comments

The image mosaic behavior, as stated in the readme, is inverted from production behavior

Looks like this is a holdover from the changes made in https://github.com/FxEmbed/FxEmbed/commit/129e26c6a9468370a8e1eb5e5b75a8fde18a20b7. I am not sure which of these is actually intended to be the default behavior

Demo:

fxtwitter.com m.fxtwitter.com
Image Image

Snippet from [readme.md]:

Combine multiple images into one

By default, FxEmbed will combine multiple images into one, allowing you to see all images on all platforms. You can also opt in to have images displayed using Discord's native multi-image so you can view individual, full-quality images (but, it doesn't work on iOS :( )

  • https://m.fxtwitter.com/example/status/123456789123456789
  • https://m.fixupx.com/example/status/123456789123456789
  • https://m.fxbsky.app/profile/example.com/post/3lkord123abc

ZenithRogue avatar Apr 12 '25 21:04 ZenithRogue

Considering the iOS client now (finally) has native multi image (https://bsky.app/profile/rcombs.me/post/3lmslvz4vqk24), the mosaic default should probably be reverted back rather than fixing the readme.

consoleSkunk avatar Apr 15 '25 18:04 consoleSkunk

Native multi-image is now the default, I gave it some extra time to settle in

dangeredwolf avatar Sep 08 '25 06:09 dangeredwolf