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

Triple clicking text /sometimes/ selects pills in preference to paragraph #9770

Open
michaelkaye opened this issue May 20, 2019 · 1 comment
Labels
A-Pills A-Timeline P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@michaelkaye
Copy link
Contributor

Description

Triple clicking a message containing a leading pill selects the pill rather than the entire paragraph containing the pill.

Steps to reproduce

  • Send a message, for instance @michaelkaye how are you?, where @michaelkaye is a pill
  • Triple click "are"
  • @michaelkaye is selected

But:

  • Send another message, like hello there @michaelkaye how are you?
  • Triple click "are"
  • how are you? is selected
  • Triple click "hello"
  • hello there is selected

But:

  • Send another message, like ok, @michaelkaye, where should this go?
  • Triple click "ok"
  • ok, is selected
  • Triple click "should"
    @michaelkaye is selected

Describe how what happens differs from what you expected.

This seems very inconsistent - i'm not quite sure what does and doesn't cause it to select the whole message, but it's not /just/ pills being at the start of the line, but having pills in the message at all doesn't always trigger this behaviour. Multiple pills or not don't seem to change the weirdness.

Version information

  • Platform: web

For the web app:

  • Browser: Firefox 66.0.4 (64-bit)
  • OS: Ubuntu
  • URL: riot.im/develop
@michaelkaye
Copy link
Contributor Author

(this is related but not the same as #7053 which talks about the composer, this is in the displayed text)

@jryans jryans added defect S-Minor Impairs non-critical functionality or suitable workarounds exist P2 labels May 20, 2019
@jryans jryans added the A-Pills label Jun 23, 2020
@jryans jryans removed the defect label Mar 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Pills A-Timeline P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

3 participants