-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[code-infra] Move ComponentLinkHeader to @mui/docs #42061
Conversation
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.
Nice, Base UI seems to work with this: mui/base-ui#378.
The only remaining component we need to have locally is ComponentPageTabs
.
@@ -12,7 +12,15 @@ export function createRender(context: { | |||
ignoreLanguagePages: (path: string) => boolean; | |||
}): (markdown: string) => string; | |||
|
|||
export function getHeaders(markdown: string): Record<string, string | string[]>; | |||
export interface MarkdownHeaders { |
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.
These headers also include components
and hooks
.
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.
Added those and removed some of the typescript casts
@mui/docs/...
@mui/docs
ComponentLinkHeader
. We could think about supporting named exports with something like: