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
Enhance context menu #8930
Enhance context menu #8930
Conversation
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.
@megayu Impressive work! I appreciate the additional fixes and refactoring ❤️.
There is an issue with the way set-heading-aux!
behaves. When we set a fixed heading in markdown mode, we just need to modify the content of the block, to add the corresponding number of #
. If we already have a heading set, the new heading is now set as a block property (e.g. heading:: 4
). The only case that we need to do that, is when the heading is set to auto
, because there is no auto heading in markdown mode.
@sprocketc yes, it's a silly mistake, fixed in 8e081f5 |
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.
Thanks! Confirmed that the issue is fixed.
Add some functions to custom context menu, including
only select current block during open "bullet" context menu, that fix #7689
fix undo heading needs multiple steps