Add Fumadocs #111
Open
Add Fumadocs #111
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@pomber is attempting to deploy a commit to the Solana Foundation Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
It looks like on mobile we're missing the hamburger that opens the sidebar nav. Will need an alternative. On the POC, the issue is that when on mobile, there's no way to open sidebar. The icon on the solana-com nav is for the entire website. On the current solana-com its floating on the bottom left: On the default fumadocs layout the icon to open the sidebar is part of the nav, but that's overridden by the default solana-com navbar. 
|
|
Minor styling things: The font type and font size may need to be updated to better match solana-com. The breadcrumbs should stick below the navbar when scrolling.
There's extra spacing above the TOC: 
|
|
There is also currently a bunch of "webpack-internal" warnings when running locally |
catalinred
reviewed
Nov 19, 2024
There was a problem hiding this comment.
@pomber Thanks for working on this!
Overall, this is looking promising, and I’m looking forward to seeing locale implementation for the app router. Having an alternative app router layout is something I’m particularly excited about, as it will help us migrate pages step-by-step from the pages router to the app router.
I second what John wrote above: first and foremost, we don’t want to lose any of the existing functionality. The UI should also match the current level of implementation, including hovers, active states, sizing, and overall typography.
Additionally, tables, alerts, and code snippets from within the /docs* are critical elements, and we need to ensure they are migrated properly.
P.S. Please fetch the latest changes from the main branch and apply them to the newly created next.config.mjs and rewrites-redirects.mjs.
| "@types/node": "^22.7.4", | ||
| "@types/styled-components": "^5.1.34", | ||
| "autoprefixer": "^10.4.20", |
There was a problem hiding this comment.
Is this really needed? Trying to understand how autoprefixer helps here since most modern browsers implemented standardized CSS properties with minimal need for prefixes.
There was a problem hiding this comment.
it's probably safe to remove, I need to check if Tailwind is using any newish feature that depends on prefixes though, because they recommend it in their docs
| @@ -73,17 +78,19 @@ | |||
| "postcss-preset-env": "^10.0.5", | |||
| "prettier": "^3.3.3", | |||
| "svg-react-loader": "^0.4.6", | |||
| "tailwindcss": "^3.4.14", | |||
There was a problem hiding this comment.
I read your comment about Tailwind and yeah am not a big fan of including tailwindcss once again and w/o the tw prefix this time. But we can accept that given the situation - looking forward though to have this polished as much as possible to reduce the amount of unused CSS here.
Curious if you have any other insights overall here, considering the solana-lib repo et all.
There was a problem hiding this comment.
I have a few questions about solana-lib, I'll ask later on slack
There was a problem hiding this comment.
I read the postcss notes, and yeah these changes need to be reverted back, otherwise as you expect lots of stuff will break.
There was a problem hiding this comment.
yep I'll add the purge plugin back, it's a matter of writing the right regexp so it includes the missing tailwind classes, but I want to profile and compare the before and after output to make sure it's done right
I think the idea for most of those components is to use the ones built-in in fumadocs. For example, the steps component. We shouldn't lose any functionality, the current issues with the layout and styles are mainly because of the "minimum viable fumadocs integration" state of the PR. |
What are the errors? these? I think those were there from before, I can take a look if you want. I'm guessing they come from |
You're right it seems the warnings already existed. I just cloned the repo again, ran locally, and got the same warnings. Please leave them for now and prioritize the migration, thanks. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a draft PR showing how to use fumadocs as the docs framework.
/docspath (with placeholder content)TODO
@ts-ignoreannotations that can be removed after the old docs framework is replaced (because of some conflicts between different versions ofmdx/types)RE Tailwind duplication:
fumadocs components ship with unprefixed classes, while solana-lib use the
twprefix, this means that we need to include two tailwind stylesheets (only for the routes that use fumadocs), we can still use something likecssnanoto remove any duplicated CSS (like tailwind's preflight styles)