Skip to content

fix(sidebar): collapse external-link cue with the item label - #2286

Merged
markdumay merged 1 commit into
mainfrom
fix/sidebar-collapsed-external-cue
Oct 28, 2026
Merged

markdumay merged 1 commit into
mainfrom
fix/sidebar-collapsed-external-cue

Conversation

@markdumay

Copy link
Copy Markdown
Collaborator

In a collapsible sidebar, assets/link.html appended the external-link cue after the label span as a bare " ", so it stayed visible beside the item icon in icon-only mode. Render the cue inside .sidebar-item-label instead and pass cue=false to the link partial, so the cue collapses and fades with the label. Expanded rendering is unchanged and the site's externalLinks.cue setting is still honoured.

In a collapsible sidebar, assets/link.html appended the external-link
cue after the label span as a bare "&nbsp;<svg>", so it stayed visible
beside the item icon in icon-only mode. Render the cue inside
.sidebar-item-label instead and pass cue=false to the link partial, so
the cue collapses and fades with the label. Expanded rendering is
unchanged and the site's externalLinks.cue setting is still honoured.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@netlify

netlify Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for gethinode-demo ready!

Name Link
🔨 Latest commit 271f021
🔍 Latest deploy log https://app.netlify.com/projects/gethinode-demo/deploys/6ae2039ea08dd7cca2259fce
😎 Deploy Preview https://deploy-preview-2286--gethinode-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@markdumay
markdumay merged commit dc88977 into main Oct 28, 2026
17 checks passed
@markdumay
markdumay deleted the fix/sidebar-collapsed-external-cue branch October 28, 2026 14:22
@markdumay

Copy link
Copy Markdown
Collaborator Author

🎉 This PR is included in version 3.46.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant