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

Use vanilla React for link-editing form, and remove react-hook-form dependency #6

Merged
merged 3 commits into from
Jun 10, 2023

Conversation

sjdemartini
Copy link
Owner

@sjdemartini sjdemartini commented Jun 10, 2023

The form behavior should essentially be identical, but we can reduce the bundle impact and simplify things by reducing external dependencies.

This adds back the bubble menu as a dependency, since otherwise because
it's a dependency of the tiptap/react plugin, it would get installed
with a higher version that had incompatibilities with our other
packages.

To get `react-hook-form` to be removed upon install, I had to delete the
pnpm-lock file for the example app in order to force it to re-install
the package based on its latest package.json definition. Otherwise, in
this case, the `react-hook-form` package was not getting removed, since
it seems the dependencies of the `file:../` for mui-tiptap were somehow
cached.
@sjdemartini sjdemartini merged commit 94defc9 into main Jun 10, 2023
@sjdemartini sjdemartini deleted the remove-react-hook-form branch June 10, 2023 03:48
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.

1 participant