-
Notifications
You must be signed in to change notification settings - Fork 26
UI Fixes: Consistent styling and layouts #3416
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?
Changes from all commits
16b51e7
8365b92
a738a5f
40ddb1c
f8cbe23
ea9bbfa
054f1a5
a7052a6
7204d2e
2a82cc5
dc4ace9
5d16d53
3d63b12
e9c238c
576ceee
2984e12
0c72c02
fc03acc
d24a1e0
0f9344b
f2c1634
a349328
a96d3d3
59649b5
245decf
94adf5e
a795f32
40c7a21
bb17f18
390f79d
631907b
012e1e6
f5d98a2
d2fd5b2
2d949a4
ca687a6
9e795b5
76ccbfb
b7e1671
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ah, I see, this is the tag badges added to badge the figma |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -108,19 +108,30 @@ export default async function DocPage({ params }: PageProps) { | |
| // Copy button is always enabled - it's a client component that works with static pages | ||
| const showCopyButton = true; | ||
|
|
||
| // Hide breadcrumbs for top-level section index pages | ||
| const topLevelIndexPages = [ | ||
| "moosestack", | ||
| "moosestack/index", | ||
| "hosting/overview", | ||
| "hosting/index", | ||
| "ai/overview", | ||
| "ai/index", | ||
| ]; | ||
| const isTopLevelIndex = topLevelIndexPages.includes(slug); | ||
|
Comment on lines
+111
to
+120
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Include root
✅ Proposed fix const topLevelIndexPages = [
"moosestack",
"moosestack/index",
+ "hosting",
"hosting/overview",
"hosting/index",
+ "ai",
"ai/overview",
"ai/index",
];🤖 Prompt for AI Agents |
||
|
|
||
| return ( | ||
| <> | ||
| <div className="flex w-full flex-col gap-6 pt-4"> | ||
| <div className="flex w-full min-w-0 flex-col gap-6 pt-4"> | ||
| <div className="flex items-center justify-between"> | ||
| <DocBreadcrumbs items={breadcrumbs} /> | ||
| {!isTopLevelIndex && <DocBreadcrumbs items={breadcrumbs} />} | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Menu shifts left when breadcrumbs are hiddenLow Severity When |
||
| {showCopyButton && ( | ||
| <MarkdownMenu | ||
| content={content.content} | ||
| isMDX={content.isMDX ?? false} | ||
| /> | ||
| )} | ||
| </div> | ||
| <article className="prose prose-slate dark:prose-invert max-w-none w-full min-w-0"> | ||
| <article className="prose dark:prose-invert max-w-none w-full min-w-0 overflow-x-auto"> | ||
| {content.isMDX ? | ||
| <MDXRenderer source={content.content} /> | ||
| : <div dangerouslySetInnerHTML={{ __html: content.content }} />} | ||
|
|
||


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 this is to make sure the tag badges should up like in the figma