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

selecting screenshots from the gallery are sometimes sent immediately without staging #2544

Closed
r10s opened this issue Jan 22, 2025 · 4 comments · Fixed by #2546
Closed

selecting screenshots from the gallery are sometimes sent immediately without staging #2544

r10s opened this issue Jan 22, 2025 · 4 comments · Fixed by #2546
Labels

Comments

@r10s
Copy link
Member

r10s commented Jan 22, 2025

on iphone7, when selecting a screenshot from the gallery, that is sent immediatey, without staging

this is the (questionable) behaviour we use for stickers, however, the screenshot is sent correctly as image (and not as sticker)

i could reproduce that on iphone7 with ios15 only - not on iphone13 with ios18

cc @Amzd - that may be introduced by recent changes - but maybe it is also in since "forever", i do not use the iphone7 that often

@r10s r10s added the bug label Jan 22, 2025
@Amzd
Copy link
Collaborator

Amzd commented Jan 22, 2025

Can you attach a screenshot like that? It must be recognized as a sticker, so if we had an example I could strengthen that detection.

@r10s
Copy link
Member Author

r10s commented Jan 22, 2025

here is a video, recorded on iphone7 - note the missing staging and the final opening of "info" that shows, the screenshot is sent as image, not sticker

RPReplay_Final1737558186.mov

@r10s
Copy link
Member Author

r10s commented Jan 22, 2025

it does not happen with all screenshot, but it happens with the attached one (1.135.694 bytes) ...
... so maybe not related to iPhone7

Image

@r10s
Copy link
Member Author

r10s commented Jan 22, 2025

yip, with that ^^ screenshot i have the same behavior on iphone13 with ios18

@Amzd Amzd linked a pull request Jan 22, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants