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

Fix weird ALT label press behavior in embedded images #5751

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

gpp-0
Copy link
Contributor

@gpp-0 gpp-0 commented Oct 14, 2024

On posts with more than one embedded image, pressing on the ALT label on any of the images registers a press on the rest of the post instead of on the image, which is weird, see video. This doesn't happen with a single embedded image

a.mp4

EDIT: This is what happens in posts with a single embedded image (added ripple for visibility). It looks like it does register a press on the actual image but it's kinda funky (it starts at the top right corner instead of at the point of contact), so that should probably be fixed too

b.mp4

I disabled pointer events on the badges and now it works fine i think

this actually doesn't make a visible difference at this point which is
why i didn't change it before, but the behavior is still technically
weird without it. pressing on the badge does register a press on the
embedded image like expected, but it's not at the point of contact. this
is visible when an android ripple is added to the pressable enclosing
the image
@gpp-0 gpp-0 force-pushed the fix-weird-alt-label-touch-event branch from ed5f9f6 to 681a77b Compare November 30, 2024 00:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant