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

feat(supersearch): Improved handling of double-click and triple-click for text selection #1171

Conversation

johanbissemattsson
Copy link
Contributor

@johanbissemattsson johanbissemattsson commented Nov 19, 2024

Description

Tickets involved

LWS-267

Solves

Improves the handling of text-selection when double-clicking and triple-clicking on text in the collapsed editor.

Double-clicks results in the selection of the word at the given position while triple-clicks selects the entire line.

Summary of changes

  • Improve handling of double-click and triple-clicks for text selection

… text selection

Removes the need of a delay before showing the dialog element
johanbissemattsson added a commit that referenced this pull request Nov 19, 2024
If the expanded search text box is placed at the same place as the collapsed, text-selection by double-clicking and triple-clicking will work mostly natively out-of-the-box.

If we decide that we need to support cases where the expanded search text box is placed somewhere else then #1171 can be merged.
@johanbissemattsson
Copy link
Contributor Author

johanbissemattsson commented Nov 19, 2024

I think we should keep this PR unmerged for now to keep the code as simple as possible (If the expanded search text field is placed at the same place as the collapsed, text-selection by double-clicking and triple-clicking will work mostly natively out-of-the-box).

We can merge it if we decide that we need to support cases where the expanded search text box is placed somewhere else.

@johanbissemattsson
Copy link
Contributor Author

Closing for now, we can re-open it later if we decide that we need it.

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