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.

Sign up for GitHub

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

Jump to bottom

Don't open menu when long clicking a link #4324

Closed
chagai95 opened this issue Oct 23, 2021 · 2 comments · Fixed by #4620
Closed

Don't open menu when long clicking a link #4324

chagai95 opened this issue Oct 23, 2021 · 2 comments · Fixed by #4620
Assignees
@ouchadam
Labels
A-Timeline O-Occasional Affects or can be seen by some users regularly or most users rarely S-Tolerable Low/no impact on users T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
Issue triage Android App Team

Comments

@chagai95
Copy link
Contributor

Steps to reproduce

  1. long clicking a link to copy it opens the menu meant for the long click on a message

Intended result and actual result

What did you expect?

The menu should not open

What happened instead?

The menu opened

Your phone model

Galaxy A51

Operating system version

One UI 3.1, Android 11

Application version and app store

1.3.3 [40103032] (G-b3874)

Homeserver

matrix.org

Will you send logs?

No
@chagai95 chagai95 added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Oct 23, 2021
@RiotRobot RiotRobot added this to Incoming in Issue triage Oct 23, 2021
@chagai95
Copy link
Contributor Author

Here's a screen cap:

Screen_Recording_20211023-195756_Matrix.Chat.mp4

@ouchadam ouchadam added A-Timeline O-Occasional Affects or can be seen by some users regularly or most users rarely S-Tolerable Low/no impact on users labels Oct 25, 2021
@ouchadam ouchadam moved this from Incoming to Triaged in Issue triage Oct 25, 2021
@ouchadam ouchadam self-assigned this Dec 2, 2021
@ouchadam ouchadam added this to In progress in Android App Team Dec 2, 2021
@ouchadam ouchadam mentioned this issue Dec 2, 2021
Merged
@ouchadam ouchadam moved this from In progress to In Code Review in Android App Team Dec 2, 2021
Issue triage automation moved this from Triaged to Closed Dec 2, 2021
Android App Team automation moved this from In Code Review to Merged Dec 2, 2021
@bmarty
Copy link
Member

bmarty commented Dec 2, 2021

In EA 1.3.10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ouchadam ouchadam

Labels
A-Timeline O-Occasional Affects or can be seen by some users regularly or most users rarely S-Tolerable Low/no impact on users T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
Android App Team
Merged
Issue triage
Closed
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Avoiding duplicate message long press events element-hq/element-android
3 participants
@ouchadam @bmarty @chagai95