Skip to content

feat(tables): add the ability to enable/disable table features #1470

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

Merged
merged 3 commits into from
Feb 28, 2025

Conversation

nperez0111
Copy link
Contributor

This adds a new option to the editor instance settings which is for editor specific configuration that we want our UI to be reactive to.
For now, it only implements enabling and disabling specific table functionality that may be too much for certain applications. Now, you can use this settings to enable the ability to use those features

Copy link

vercel bot commented Feb 28, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
blocknote ✅ Ready (Inspect) Visit Preview Feb 28, 2025 2:27pm
blocknote-website ✅ Ready (Inspect) Visit Preview Feb 28, 2025 2:27pm

Copy link
Collaborator

@matthewlipski matthewlipski left a comment

Choose a reason for hiding this comment

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

Looks good! Just 1 question

/**
* Allows enabling / disabling features of tables.
*/
tables?: {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does it make sense putting tables under a settings option? Having both seems a bit redundant, or are you also thinking of adding/moving other stuff to the settings option in the future?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was thinking that we'd have more settings in the future, so it made sense to namespace it. But, I'm open to the suggestion of a flat-list. I just feel like it often gets namespaced anyway

@nperez0111 nperez0111 merged commit 52cd035 into tables Feb 28, 2025
2 checks passed
@nperez0111 nperez0111 deleted the feature/tables-disable branch March 4, 2025 12:38
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