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

[$500] Chat - Link is highlighted when right clicking on it on Safari #31597

Closed
2 of 6 tasks
kbecciv opened this issue Nov 20, 2023 · 9 comments
Closed
2 of 6 tasks

[$500] Chat - Link is highlighted when right clicking on it on Safari #31597

kbecciv opened this issue Nov 20, 2023 · 9 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors

Comments

@kbecciv
Copy link

kbecciv commented Nov 20, 2023

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 1.4.1.4
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:

Issue found when executing PR #31465

Action Performed:

. Open any report
2. Send a link
3. Long press or right-click on the link

Expected Result:

No highlight style appears on the link

Actual Result:

On Web/Safari the link is highlighted and on mobile Safari the whole chat is being highlighted when right clicking or tapping and holding on a link

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6284762_1700524432645.Recording__501.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01c01e3d3e075b8c26
  • Upwork Job ID: 1726751687719890944
  • Last Price Increase: 2023-11-20
@kbecciv kbecciv added External Added to denote the issue can be worked on by a contributor Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Nov 20, 2023
@melvin-bot melvin-bot bot changed the title Chat - Link is highlighted when right clicking on it on Safari [$500] Chat - Link is highlighted when right clicking on it on Safari Nov 20, 2023
Copy link

melvin-bot bot commented Nov 20, 2023

Triggered auto assignment to @adelekennedy (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Copy link

melvin-bot bot commented Nov 20, 2023

Job added to Upwork: https://www.upwork.com/jobs/~01c01e3d3e075b8c26

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Nov 20, 2023
Copy link

melvin-bot bot commented Nov 20, 2023

Bug0 Triage Checklist (Main S/O)

  • This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
  • This bug is not a duplicate report (check E/App issues and #expensify-bugs)
    • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
  • This bug is reproducible using the reproduction steps in the OP. S/O
    • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
    • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
  • This issue is filled out as thoroughly and clearly as possible
    • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
  • I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Copy link

melvin-bot bot commented Nov 20, 2023

Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak (External)

@abzokhattab
Copy link
Contributor

not sure if this is a bug https://discussions.apple.com/thread/2165827 https://www.reddit.com/r/MacOS/comments/k8cipw/safari_right_clicking_google_links_highlights_too/

@erquhart
Copy link
Contributor

Agreed, Safari doesn't just select links, it selects any text when you right-click. Try clicking something that's just text, same thing. Feature not a bug.

@Tony-MK
Copy link
Contributor

Tony-MK commented Nov 21, 2023

Proposal

Please re-state the problem that we are trying to solve in this issue.

Chat - Link is highlighted when right clicking on it on Safari

What is the root cause of that problem?

The root cause of the problem is that, by default, Safari selects the text in the link when it is right-clicked. This is most likely not a bug, but different platforms, like Chrome, don't highlight the text in this situation. Therefore, it is more about improving the consistency of the app across multiple platforms.

What changes do you think we should make in order to solve the problem?

Use css attributes -webkit-user-select: none;, user-select: none; and possibly -ms-user-select: none; for consistency with other browsers. Apply them to the link elements in the chat message. This will ensure that the links do not get highlighted across all platforms. You can read more over here: User Selection

{
  -webkit-user-select: none; /* Safari */
  -ms-user-select: none; /* IE 10 and IE 11 */
  user-select: none; /* Standard */
}

@Tony-MK
Copy link
Contributor

Tony-MK commented Nov 21, 2023

@adelekennedy Could you please ask @eVoloshchak for a second opinion before we close, especially when I have new findings on this prospal. It opposes what the other contributors say, and I would just like another opinion. Sorry, but thank you.

@eVoloshchak
Copy link
Contributor

Agree with closing this one out
This is a normal behavior for Safari (not only Safari, system apps like Contacts, Notes, Mail, TextEdit, Dictionary all select a word when you right-click it)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors
Projects
None yet
Development

No branches or pull requests

6 participants