-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
[docs] Setup a new next.js application for documentation #299
Conversation
f042409
to
d899dbd
Compare
5780e3e
to
0265ee2
Compare
This app only uses Pigment and some CSS module for styling and will serve as an internal dogfood for us.
0265ee2
to
66e8d13
Compare
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.
I think we can merge this and iterate. Few things:
- can we remove the unused svgs from docs/public?
- the logo would look better if it is black
- the versions dropdown is not opening (do we even need it to be a dropdown at this point?
|
||
## Installation | ||
|
||
Pigment CSS has two category of packages. First one is to be imported and used in your source code. This includes the public API of the package. Second category is to be imported in your bundler config file to configure and actually be able to use Pigment CSS. We currently support [`Next.js`](https://nextjs.org/) (no Turbopack yet), [`Vite`](https://vite.dev/) and [`Webpack`](https://webpack.js.org/). |
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.
Pigment CSS has two category of packages. First one is to be imported and used in your source code. This includes the public API of the package. Second category is to be imported in your bundler config file to configure and actually be able to use Pigment CSS. We currently support [`Next.js`](https://nextjs.org/) (no Turbopack yet), [`Vite`](https://vite.dev/) and [`Webpack`](https://webpack.js.org/). | |
Pigment CSS has two category of packages. | |
First one is to be imported and used in your source code. | |
This includes the public API of the package. | |
Second category is to be imported in your bundler config file to configure and actually be able to use Pigment CSS. | |
We currently support [`Next.js`](https://nextjs.org/) (no Turbopack yet), [`Vite`](https://vite.dev/) and [`Webpack`](https://webpack.js.org/). |
Not important for this PR, but putting each sentence in a new line, makes the review so much easier. This is the convention we use in the Material UI's docs
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.
Will do in separate PRs. This wasn't even real content.
docs/next.config.ts
Outdated
const DATA_DIR = path.join(process.cwd(), 'data'); | ||
|
||
const nextConfig: NextConfig = { | ||
/* config options here */ |
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.
/* config options here */ |
805e5ea
to
7be2a78
Compare
I'll merge this and do more refinements (point 2 and 3) in separate PRs. |
Sounds good 👍 |
'react/react-in-jsx-scope': 'off', | ||
'react/no-unknown-property': ['error', { ignore: ['sx'] }], | ||
'import/extensions': [ | ||
'error', |
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.
Config will need to move to the root folder eslint config file to prepare Eslint v9 upgrade.
I wired it to https://pigment-css.com/ with |
I think we could have waited with this. Maybe we can make the Landing page with something like "Under construction" and don't like the docs yet. cc @brijeshb42 |
This app only uses Pigment and some CSS module for styling and will serve as an internal dogfood for us.
The styling and most of the mdx related logic has been taken from the Base UI docs.
Similarly, it uses the app router with static export to build the whole docs statically that we can then deploy on Netlify.
Preview - https://deploy-preview-299--pigment-css.netlify.app/