-
-
Notifications
You must be signed in to change notification settings - Fork 348
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(selection): support triple-click to select all text of block #2037
feat(selection): support triple-click to select all text of block #2037
Conversation
Open in CodeSandbox Web Editor | VS Code | VS Code Insiders |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -309,6 +310,14 @@ export class DefaultSelectionManager { | |||
showFormatQuickBarByDoubleClick(e, this.page, this.container, this.state); | |||
}; | |||
|
|||
private _onContainerTripleClick = (e: SelectionEvent) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lawvs You can add the logic of all the text of the block in this method.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It mainly shows the logic of format-quick-bar and whether the content is correct when copying.
ad1f361
to
fb3b8a5
Compare
fb3b8a5
to
4eeabb9
Compare
4eeabb9
to
d3e2b4e
Compare
d3e2b4e
to
abf4426
Compare
7f33141
to
ab6ed9a
Compare
ab6ed9a
to
83fa11e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is handled pretty well!
Related to: #2036
Screen.Recording.2023-04-14.at.5.19.23.PM.mov