Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Footer overlaps message if image only #4310

Closed
nicodh opened this issue Nov 2, 2024 · 6 comments
Closed

Footer overlaps message if image only #4310

nicodh opened this issue Nov 2, 2024 · 6 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists

Comments

@nicodh
Copy link
Contributor

nicodh commented Nov 2, 2024

Footer in message with image only, overlaps the image and reactions are displayed on the right

Happens for incoming and outgoing messages

image

image

@nicodh nicodh added the bug Something isn't working label Nov 2, 2024
@Simon-Laux
Copy link
Member

Bildschirmfoto 2024-11-02 um 17 15 50

can't reproduce on main: 5eef453

also the overlay metadata is intended behaviour, I think on android and iOS they even overlay the name when it is not a forward message.

@iequidoo
Copy link

iequidoo commented Nov 2, 2024

Thanks, then there's nothing to fix it seems. I'm using 1.46.8, waiting for 1.47.1 with fixed deb package name

@link2xt
Copy link
Collaborator

link2xt commented Nov 2, 2024

Looks like this on my side:
em

Using commit d619d2d

@iequidoo
Copy link

iequidoo commented Nov 2, 2024

Btw, i checked Telegram and Whatsapp, they don't put reactions onto image, only datetime. EDIT: But i checked Android apps, not desktop versions. EDIT: Not putting reactions onto image makes sense, otherwise if there are many reactions, the bottom of the image is almost hidden. And on desktop there is more screen space usually than on mobile devices.

@WofWca
Copy link
Collaborator

WofWca commented Nov 4, 2024

The original post is a duplicate of #4235, which has been recently resolved. Please confirm that you're on 1.47.1 or main.

For what @link2xt there is a WIP MR: #4176

@WofWca WofWca added the duplicate This issue or pull request already exists label Nov 4, 2024
@nicodh
Copy link
Contributor Author

nicodh commented Nov 8, 2024

Closed and moved to #4322

@nicodh nicodh closed this as completed Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

5 participants