Skip to content
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

Adds modifyBlocks option to make structure read-only #570

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hermzz
Copy link
Contributor

@hermzz hermzz commented Mar 7, 2019

Setting the option to false won't allow users to add/remove or re-order
blocks.

Setting the option to false won't allow users to add/remove or re-order
blocks.
@raffij
Copy link
Contributor

raffij commented Mar 8, 2019

Thanks for this PR @hermzz. It's a good idea and something we'll happily merge in.

So you know. One of the things that hasn't been fully resolved since we moved from a block to a document based approach is the block limitations like this.

Something you might not have considered is pasting content into a block. Try pasting a couple of paragraphs of text into a text block and see how it behaves and it's as you expect.

I'm currently working on improving block selection which includes some paste improvements. So it might be best to wait till those are done before making sure the above PR does as you expect too.

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