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

refactor: Clean up component code #637

Merged
merged 10 commits into from
Mar 14, 2024
Merged

Conversation

matthewlipski
Copy link
Collaborator

This PR cleans up code for children of the Toolbar component, as well as some other small/misc cleanups for components.

@matthewlipski matthewlipski changed the title Cleaned up component code Refactor: Clean up component code Mar 12, 2024
Copy link

vercel bot commented Mar 12, 2024

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

Name Status Preview Updated (UTC)
blocknote ✅ Ready (Inspect) Visit Preview Mar 14, 2024 3:15pm
blocknote-website ✅ Ready (Inspect) Visit Preview Mar 14, 2024 3:15pm

@matthewlipski matthewlipski merged commit 3346ee0 into ui-refactor Mar 14, 2024
4 checks passed
matthewlipski added a commit that referenced this pull request Mar 14, 2024
* wip

* Refactored image toolbar

* Refactored image toolbar, hyperlink toolbar, and table handles

* Implemented PR feedback

* Implemented PR feedback

* Implemented PR feedback

* refactor: Clean up component code (#637)

* Cleaned up component code

* Renamed "Link" in components to "Hyperlink"

* Implemented PR feedback

* alternative solution for toolbar fadeout (#645)

* chore: Playwright update (#640)

* Updated playwright

* Small fix

* dont upgrade tiptap etc

* fix pw install?

---------

Co-authored-by: yousefed <yousefdardiry@gmail.com>

* fix `setEditable` called when not necessary (#635)

* Added temp fix for shortcuts and input rules in tables (#561)

---------

Co-authored-by: Yousef <yousefdardiry@gmail.com>

* Renamed hyperlink toolbar -> link toolbar and small changes

---------

Co-authored-by: Yousef <yousefdardiry@gmail.com>
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