-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Prevent duplicating ToDos in documents content #10063
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
base: develop
Are you sure you want to change the base?
Conversation
Signed-off-by: Nikolay Marchuk <[email protected]>
Signed-off-by: Nikolay Marchuk <[email protected]>
Connected to Huly®: UBERF-13907 |
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.
Pull Request Overview
This PR prevents duplicating ToDo items when copying and pasting content within text editor documents by implementing deduplication logic.
- Adds an EditorContextExtension to track the current editor's object context (ID, class, and space)
- Implements a TodoItemPastePlugin that cleans duplicate ToDo attributes during paste operations
- Extends ToDo items with parent context attributes to identify their origin
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
File | Description |
---|---|
plugins/text-editor-resources/src/kits/editor-kit.ts | Integrates the new EditorContextExtension into the editor kit |
plugins/text-editor-resources/src/components/extension/todo/todo.ts | Implements deduplication logic and adds parent context tracking to ToDo items |
plugins/text-editor-resources/src/components/extension/editorContext.ts | Creates new extension to provide editor context for ToDo deduplication |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
plugins/text-editor-resources/src/components/extension/todo/todo.ts
Outdated
Show resolved
Hide resolved
plugins/text-editor-resources/src/components/extension/todo/todo.ts
Outdated
Show resolved
Hide resolved
Signed-off-by: Nikolay Marchuk <[email protected]>
Signed-off-by: Nikolay Marchuk <[email protected]>
9b1391b
to
05dfd9f
Compare
No description provided.