-
Notifications
You must be signed in to change notification settings - Fork 94
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
Add APIs for outline state to RichTextReader and WrapperComponent #3462
Conversation
Test summaryRun details
View run in Cypress Dashboard ➡️ Flakiness
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
2bf1b53
to
76c4614
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.
So far looks good to me. I guess the handing down props and propagating up events could be avoided by using provide and inject - but from what i see it's only two levels. So I'm not sure if that's worth it.
Export `OUTLINE_STATE` and `OUTLINE_ACTIONS` in NPM package. Also add `EditorOutline` to `BaseReader` dom structure, so it can be toggled by providing `OUTLINE_STATE` and `OUTLINE_ACTIONS` from a parent vue component. Signed-off-by: Jonas <jonas@freesources.org>
* Emit `outline-toggled` event when outline got toggled from Text * Allow to pass property `showOutlineOutside` for changing outline state from parent component. This is required to support toggeling the outline from inside Collectives. Related: #2919 Signed-off-by: Jonas <jonas@freesources.org>
Signed-off-by: Jonas <jonas@freesources.org>
76c4614
to
ac3518e
Compare
/compile |
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
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.
Had a quick look at the code again.
/backport 2ed516d,a9f4db5ad7bae90d30e7960da68b02657165ff10,ac3518e4c5e7c064f5bd15489c32d3125b3b4fda to stable25 |
📝 Summary
Adds APIs to interfere with outline state to
RichTextReader
andWrapperComponent
components.Required for https://gitlab.com/collectivecloud/collectives/-/merge_requests/649
🚧 TODO
🏁 Checklist
npm run lint
/npm run stylelint
/composer run cs:check
)