⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@YousefED
Copy link
Collaborator

@YousefED YousefED commented Jan 20, 2026

Summary

  • upgrade main dependencies (nextjs, fumadocs, better-auth, etc)
  • Fumadocs DocsLayout is no longer ejected, HomeLayout is because we're making changes to the navbar
  • clean up routes (no more routes needed for every "content" page
  • fix lint + tests
  • move og images from API to statically rendered

(This was done by first creating a new fumadocs project and step-by-step moving things over)

TODO:

[ ] PR review
[ ] Testing across screen sizes and light / darkmode
[ ] move react-email changes to separate PR
[ ] check if db migration is required

@vercel
Copy link

vercel bot commented Jan 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
blocknote Ready Ready Preview Jan 21, 2026 7:40am
blocknote-website Ready Ready Preview Jan 21, 2026 7:40am

Request Review

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 20, 2026

Open in StackBlitz

@blocknote/ariakit

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/ariakit@2371

@blocknote/code-block

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/code-block@2371

@blocknote/core

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/core@2371

@blocknote/mantine

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/mantine@2371

@blocknote/react

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/react@2371

@blocknote/server-util

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/server-util@2371

@blocknote/shadcn

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/shadcn@2371

@blocknote/xl-ai

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/xl-ai@2371

@blocknote/xl-docx-exporter

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/xl-docx-exporter@2371

@blocknote/xl-email-exporter

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/xl-email-exporter@2371

@blocknote/xl-multi-column

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/xl-multi-column@2371

@blocknote/xl-odt-exporter

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/xl-odt-exporter@2371

@blocknote/xl-pdf-exporter

npm i https://pkg.pr.new/TypeCellOS/BlockNote/@blocknote/xl-pdf-exporter@2371

commit: 8d5e60c

@YousefED YousefED marked this pull request as ready for review January 20, 2026 18:54
color: var(--color-fd-primary);
}

#nd-nav {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@matthewlipski I removed this as it was causing z-index issues with the search bar.

Can you see if removing it causes any issues that it originally solved when you added it? (e.g. via git blame)?


// suggested here: https://github.com/fuma-nama/fumadocs/issues/1095#issuecomment-2495855920
// before highlight call
await getSingletonHighlighter({
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@nperez0111 I couldn't find exactly why this was necessary, seems to work fine without?

Copy link
Contributor

Choose a reason for hiding this comment

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

Was around twoslash support, looks like it may have been resolved though. Just would want to check that you can still hover over a code-block's type & see the type

light: "github-light",
dark: "github-dark",
},
// suggested here: https://github.com/fuma-nama/fumadocs/issues/1095#issuecomment-2495855920
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@nperez0111 similar to above, do you remember why / where this was needed?

Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above

@nperez0111
Copy link
Contributor

Too much code changed to meaningfully review, maybe instead you can share what your goals are with this?
I ran the dev server for this, and didn't find it meaningfully different, so I'm unsure the goal to review against

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.

3 participants