Conversation
Co-authored-by: OrangeX4 <34951714+OrangeX4@users.noreply.github.com>
|
Co-authored-by: OrangeX4 <34951714+OrangeX4@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR fixes two cosmetic issues on the homepage: suppressing an unwanted external-link icon on the "Universe" navbar item, and adding proper spacing between "Browse themes →" and "Browse universe →" feature card links.
Changes:
- Add CSS class hook to the Universe navbar item and suppress its auto-injected SVG icon.
- Wrap feature card links in a flex container with gap styling.
- Add a secondary link ("Browse universe") to the themes feature card.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
docusaurus.config.js |
Adds className to Universe navbar item; updates a footer link URL; adds Universe to footer nav. |
src/css/custom.css |
Hides the external-link SVG for the Universe navbar item. |
src/components/HomepageFeatures/index.js |
Adds secondaryLink/secondaryLinkLabel props and wraps links in a container div. |
src/components/HomepageFeatures/styles.module.css |
Adds .featureLinks flex container style. |
touying |
Bumps the touying git submodule to a newer commit. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| </Link> | ||
| )} | ||
| {secondaryLink && ( | ||
| <Link className={styles.featureLink} to={secondaryLink}> |
There was a problem hiding this comment.
The Docusaurus <Link> component's to prop is intended for internal (relative) paths. For external URLs like https://typst.app/universe/search/?q=touying, the href prop should be used instead. Using to with an absolute URL may result in incorrect routing behavior (e.g., the URL being treated as a relative path).
Two cosmetic issues on the homepage: the "Universe" navbar link showed an unwanted external-link arrow icon, and "Browse themes →" / "Browse universe →" were rendered flush against each other with no gap.
Changes
docusaurus.config.js— AddclassName: 'navbar--universe-link'to the Universe navbar item as a CSS hook.src/css/custom.css— Suppress the auto-injected Docusaurus external-link SVG for that item:src/components/HomepageFeatures/index.js— Wrap the primary and secondary feature card links in a container<div>instead of emitting them as adjacent inline elements.src/components/HomepageFeatures/styles.module.css— Add.featureLinks { display: flex; flex-wrap: wrap; gap: 0.75rem; }to space the links apart.Before / After
Universewith external arrow iconUniverse— no iconBrowse themes →Browse universe →flush togetherBrowse themes →·Browse universe →with gap💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.