Skip to content

Comment and Selection Plugins#99

Open
rowanc1 wants to merge 19 commits intomainfrom
feat/selection-plugin
Open

Comment and Selection Plugins#99
rowanc1 wants to merge 19 commits intomainfrom
feat/selection-plugin

Conversation

@rowanc1
Copy link
Member

@rowanc1 rowanc1 commented Nov 19, 2021

  • Revisit demo, see if we can simplify.
  • Review selection / comment.

@rowanc1 rowanc1 force-pushed the feat/selection-plugin branch 5 times, most recently from f5ef8a5 to b0022a7 Compare November 23, 2021 22:32
@rowanc1 rowanc1 force-pushed the feat/selection-plugin branch from db17a4a to f3f3050 Compare December 21, 2021 22:05
Copy link
Member Author

@rowanc1 rowanc1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Taking a look through.

Comment on lines +121 to +125
// TODO: This has to be based on the actual from/to
// NodeSelection.create(tr.doc, from);
// if (isNodeSelection(tr.selection)) {
// deco = Decoration.node(from, to, params, spec);
// }
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Address?

@rowanc1 rowanc1 force-pushed the feat/selection-plugin branch from 2c55644 to 2396f4f Compare December 21, 2021 23:34
@rowanc1 rowanc1 force-pushed the feat/selection-plugin branch from 2396f4f to c747270 Compare December 22, 2021 21:01
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.

2 participants