-
Notifications
You must be signed in to change notification settings - Fork 619
[TOOL-3095] Portal: Enable Table of contents in Nebula pages, Adjust TOC component #5965
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
[TOOL-3095] Portal: Enable Table of contents in Nebula pages, Adjust TOC component #5965
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
34dd160 to
a9a6840
Compare
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5965 +/- ##
=======================================
Coverage 54.83% 54.83%
=======================================
Files 1150 1150
Lines 61195 61195
Branches 5155 5155
=======================================
Hits 33554 33554
Misses 26914 26914
Partials 727 727
*This pull request uses carry forward flags. Click here to find out more. |
Merge activity
|
…TOC component (#5965) <!-- start pr-codex --> ## PR-Codex overview This PR focuses on improving the `Layout` component in `layout.tsx` and updating the styling of the `TOCLink` component in `TableOfContents.tsx`. ### Detailed summary - In `layout.tsx`, the `DocLayout` component's closing tag has been moved to wrap `{props.children}`. - In `TableOfContents.tsx`, the `data-active` class for the `Link` component has been changed from `text-accent-500` to `text-f-100`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
a9a6840 to
2ccbc14
Compare
…TOC component (#5965) <!-- start pr-codex --> ## PR-Codex overview This PR focuses on refining the `Layout` component's structure and updating the styling for the `TOCLink` component to enhance the user interface. ### Detailed summary - In `Layout`, the `DocLayout` component now wraps `props.children` directly without unnecessary line breaks. - In `TOCLink`, the `data-active` class styling was changed from `text-accent-500` to `text-f-100` for active links. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
2ccbc14 to
8d9854d
Compare

PR-Codex overview
This PR focuses on improving the layout rendering in the
Layoutcomponent and refining the styling of theTOCLinkcomponent.Detailed summary
Layout, theDocLayoutcomponent now directly wrapsprops.children, removing unnecessary line breaks.TOCLink, thedata-activeclass styling is updated fromtext-accent-500totext-f-100.