Skip to content

Make /api public and fix landing page pill icon/text visibility#146

Merged
asim merged 2 commits intomainfrom
copilot/update-api-docs-auth
Feb 25, 2026
Merged

Make /api public and fix landing page pill icon/text visibility#146
asim merged 2 commits intomainfrom
copilot/update-api-docs-auth

Conversation

Copy link
Contributor

Copilot AI commented Feb 25, 2026

/api was gated behind auth, breaking links from the index and docs pages. Landing page pills also had broken icon/text rendering: active state made icons invisible (black icon on black background), and the API & MCP section was missing icons entirely.

Changes

  • main.go: Set "/api": false — API docs are public reference material
  • home/home.go: Add .preview-tab.active img { filter:brightness(0) invert(1); } so icons invert to white on active (black) pill background
  • home/home.go: Add icons to API & MCP section pills, matching the "What's Available" pill style
/* Before: icon invisible when tab active */
.preview-tab.active { background:#000; color:#fff; border-color:#000; }

/* After: icon inverts to white on active tab */
.preview-tab.active { background:#000; color:#fff; border-color:#000; }
.preview-tab.active img { filter:brightness(0) invert(1); }

💡 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.

Co-authored-by: asim <17530+asim@users.noreply.github.com>
Copilot AI changed the title [WIP] Update API documentation to display without authentication Make /api public and fix landing page pill icon/text visibility Feb 25, 2026
@asim asim marked this pull request as ready for review February 25, 2026 08:16
@asim asim merged commit 815b303 into main Feb 25, 2026
@asim asim deleted the copilot/update-api-docs-auth branch March 2, 2026 21:06
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.

2 participants