Fix integration icons in production - #532
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
There was a problem hiding this comment.
Risk assessment: Low
Approved. This is a scoped docs-site presentation change with no application logic, auth, data, or infrastructure impact.
What the diff actually changes
docs.json: addsiconpaths on four existing Integrations groups (Claude, Computer Use Models, Stripe Projects, Vercel). Navigation structure is unchanged.- 8 integration MDX files:
iconfrontmatter only; page content is unchanged. - 12 SVGs under
images/integration-icons/: path-only logos (<svg>/<path>/<title>). No<script>, event handlers,foreignObject, or externalhrefs. style.css: rules scoped to#navigation-itemsandimg[src*="/images/integration-icons/"]for size, opacity, flexorder, and dark-modeinvert(1). No@importorurl()fetches..github/tests/integration-icon-css.html: local computed-style fixture, not served as product code.
Why Low (not Very Low)
The CSS is user-facing (docs sidebar icons) and uses attribute-contains selectors plus :has() flex-order rules. That is more than a typo or copy-only edit, but the match surface is limited to integration-icon URLs and the sidebar container.
Why not Medium+
No shared services, auth/billing/permissions, APIs, schema, or infra. Blast radius is visual treatment of integration icons in the Mintlify sidebar.
No CODEOWNERS file is present, and the main ruleset does not require code-owner review. No prior approval on this PR.
Sent by Cursor Automation: Assign PR reviewers
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit c528f9b. Configure here.
| "integrations/valtown", | ||
| { | ||
| "group": "Vercel", | ||
| "icon": "/images/integration-icons/vercel.svg", |
There was a problem hiding this comment.
Custom integration sidebar icons reintroduced
Medium Severity
This change restores custom icon values on Integrations groups and pages, new assets under images/integration-icons/, and CSS that sizes, scales, reorders, and dark-mode-inverts those sidebar icons. Integration nav items stay text-only after those icons were reverted.
Additional Locations (2)
Triggered by learned rule: Do not add custom icons to Integrations nav or pages
Reviewed by Cursor Bugbot for commit c528f9b. Configure here.




Summary
#navigation-itemsand use semantic group orderingMintlify rewrites local image paths to absolute CDN URLs with query strings in production. The previous prefix/suffix selectors only matched local preview URLs, so production lost all sizing, opacity, ordering, and dark-mode styles.
Verification
npx --yes mint@latest validatexmllint --nooutfor all integration SVGsNotes
This PR does not use
!importantfor integration icon styling.Note
Low Risk
Documentation and presentation CSS only; no runtime API, auth, or data-handling changes.
Overview
Restores integration sidebar icons in production by wiring monochrome SVG assets into navigation and fixing CSS so it still applies after Mintlify rewrites image URLs to Mintlify CDN hosts with query strings.
Adds
iconpaths indocs.jsonfor nested Integrations groups (Claude, Computer Use Models, Stripe Projects, Vercel) andiconfrontmatter on individual integration MDX pages, backed by new assets underimages/integration-icons/.Updates
style.csswith#navigation-items-scoped rules that match icons viaimg[src*="/images/integration-icons/"](works for both local and CDNsrc), including 14px sizing, opacity, dark-modeinvert(1), flexorderso icons sit correctly beside labels, and small per-logo scale/opacity tweaks.Adds
.github/tests/integration-icon-css.htmlas a browser regression check that local and CDN-shaped URLs get the same computed styles.Reviewed by Cursor Bugbot for commit c528f9b. Bugbot is set up for automated code reviews on this repo. Configure here.