-
-
Notifications
You must be signed in to change notification settings - Fork 675
docs: Website upgrade and cleanup #2371
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
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
@blocknote/ariakit
@blocknote/code-block
@blocknote/core
@blocknote/mantine
@blocknote/react
@blocknote/server-util
@blocknote/shadcn
@blocknote/xl-ai
@blocknote/xl-docx-exporter
@blocknote/xl-email-exporter
@blocknote/xl-multi-column
@blocknote/xl-odt-exporter
@blocknote/xl-pdf-exporter
commit: |
| color: var(--color-fd-primary); | ||
| } | ||
|
|
||
| #nd-nav { |
There was a problem hiding this comment.
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({ |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above
|
Too much code changed to meaningfully review, maybe instead you can share what your goals are with this? |
Summary
(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