Skip to content

docs: refresh landing terminal demo and apply NVIDIA fern theme#1615

Merged
pimlock merged 1 commit into
NVIDIA:mainfrom
aschilling-nv:afs/-nvidia-theme
May 28, 2026
Merged

docs: refresh landing terminal demo and apply NVIDIA fern theme#1615
pimlock merged 1 commit into
NVIDIA:mainfrom
aschilling-nv:afs/-nvidia-theme

Conversation

@aschilling-nv
Copy link
Copy Markdown
Contributor

Summary

Refresh the docs landing page so the terminal demo no longer depends on globally-loaded CSS, switch the Fern instance to the NVIDIA theme, and bump the Fern CLI to a current version.

Related Issue

N/A

Changes

  • Extract the landing-page terminal demo into a reusable <CommandTerminal /> component with inline styles, dropping the dependency on nc-* CSS classes previously sourced from fern/main.css.
  • Animate a second command line cycling through claude / opencode / codex using @keyframes scoped inside the component.
  • Inline the BadgeLinks layout styles so the component renders correctly without .badge-links from global CSS.
  • Add docs/_components/jsx.d.ts so editors stop flagging the React global in component TSX files.
  • Switch the Fern instance to global-theme: nvidia with multi-source: true.
  • Bump Fern CLI to 5.40.0 and drop the no-longer-needed basepath-aware experimental flag.
  • Register fern/components/ as a second mdx-components directory.
  • Remove the unused Adobe analytics <script> tag.

Testing

  • mise run license:check passes
  • mise run markdown:lint:md passes
  • mise run pre-committest:rust fails locally due to a missing system z3.h header (pre-existing environment issue, no Rust code changed in this PR). Other relevant lint subsets pass.
  • Fern preview verified visually — please confirm via the docs-preview workflow that the terminal animation and badge layout render as intended.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

- Extract landing-page terminal demo into a reusable <CommandTerminal />
  component with inline styles (no global CSS dependency)
- Animate a second command line cycling through claude/opencode/codex
  via @Keyframes scoped inside the component
- Inline BadgeLinks layout styles so the component renders correctly
  without relying on .badge-links from main.css
- Add jsx.d.ts shim so editors do not flag the React global in component
  TSX files
- Switch fern instance to global-theme: nvidia with multi-source enabled
- Bump fern CLI to 5.40.0 and drop the basepath-aware experimental flag
- Register fern/components/ as a second mdx-components directory
- Remove the unused Adobe analytics script tag
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 28, 2026

All contributors have signed the DCO ✍️ ✅
Posted by the DCO Assistant Lite bot.

@aschilling-nv

This comment was marked as resolved.

@aschilling-nv

This comment was marked as resolved.

@aschilling-nv
Copy link
Copy Markdown
Contributor Author

@miyoungc - This should be good to go. However, please build this locally to ensure it meets your standards.

@miyoungc miyoungc requested review from drew, johntmyers and pimlock May 28, 2026 16:28
@miyoungc
Copy link
Copy Markdown
Collaborator

@pimlock
Copy link
Copy Markdown
Collaborator

pimlock commented May 28, 2026

@aschilling-nv Thank you, this looks good!

One thing I noticed is that License link in the navigation is being partially cut out:
image

I think I remember fixing that in the CSS in the current version by adding some padding.

@aschilling-nv
Copy link
Copy Markdown
Contributor Author

@aschilling-nv Thank you, this looks good!

One thing I noticed is that License link in the navigation is being partially cut out: image

I think I remember fixing that in the CSS in the current version by adding some padding.

This issue has been corrected in the global theme and is viewable here: https://nvidia-preview-ddd91916-5bb0-4c85-bed4-316e3cfdb458.docs.buildwithfern.com/openshell/resources/license

@pimlock pimlock merged commit 63e3a8f into NVIDIA:main May 28, 2026
27 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants