Skip to content

fix(docs): give the SCA sidebar group a lock icon - #813

Merged
pengying merged 1 commit into
mainfrom
08-10-fix_docs_give_the_sca_sidebar_group_a_lock_icon
Aug 10, 2026
Merged

fix(docs): give the SCA sidebar group a lock icon#813
pengying merged 1 commit into
mainfrom
08-10-fix_docs_give_the_sca_sidebar_group_a_lock_icon

Conversation

@pengying

@pengying pengying commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

The API reference sidebar sets tag-group icons through CSS ::before rules keyed on li[data-title="..."] in mintlify/style.css. Every OpenAPI tag in the spec had a rule except Strong Customer Authentication, which rendered with no icon at all.

Mintlify offers no supported way to set an icon on an auto-generated tag group from the spec itself: icon on a tag object fails OpenAPI validation, and x-mint: { icon } is silently ignored. Declaring the group explicitly in docs.json does render an icon, but an explicit pages array turns off endpoint auto-generation for the whole group — so the CSS rule is the right fix here.

Verified against mint dev: the group's ::before now resolves to lock.svg at 20x20, and all 23 sidebar groups have an icon (none missing).

Note: lock.svg is also used by the Embedded Wallet Auth group. Happy to switch SCA to shield.svg if we'd rather they read distinctly.

🤖 Generated with Claude Code

The API reference sidebar sets tag-group icons through CSS ::before rules
keyed on li[data-title="..."] in mintlify/style.css. Every OpenAPI tag in
the spec had a rule except Strong Customer Authentication, which rendered
with no icon at all.

Mintlify offers no supported way to set an icon on an auto-generated tag
group from the spec itself: `icon` on a tag object fails OpenAPI
validation, and `x-mint: { icon }` is silently ignored. Declaring the
group explicitly in docs.json does render an icon, but an explicit
`pages` array turns off endpoint auto-generation for the whole group, so
the CSS rule is the right fix here.

Verified against `mint dev`: the group's ::before now resolves to
lock.svg at 20x20, and all 23 sidebar groups have an icon.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
grid-flow-builder Ignored Ignored Preview Aug 10, 2026 6:20pm
grid-wallet-demo Ignored Ignored Preview Aug 10, 2026 6:20pm

Request Review

@mintlify

mintlify Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Grid 🟢 Ready View Preview Aug 10, 2026, 6:23 PM

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@greptile-apps

greptile-apps Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds the missing lock icon for the Strong Customer Authentication API-reference sidebar group.

  • Uses the same title-based selector pattern as existing sidebar icon rules.
  • References the existing lock SVG asset already used by another authentication group.

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The selector exactly matches the SCA OpenAPI tag, follows the established sidebar CSS pattern, and references an existing SVG asset.

Important Files Changed

Filename Overview
mintlify/style.css Adds a correctly targeted SCA sidebar icon rule consistent with neighboring rules and the existing asset layout.

Reviews (1): Last reviewed commit: "fix(docs): give the SCA sidebar group a ..." | Re-trigger Greptile

pengying commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

Merge activity

  • Aug 10, 6:38 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Aug 10, 6:39 PM UTC: @pengying merged this pull request with Graphite.

@pengying
pengying merged commit 2639f67 into main Aug 10, 2026
10 checks passed
@pengying
pengying deleted the 08-10-fix_docs_give_the_sca_sidebar_group_a_lock_icon branch August 10, 2026 18:39
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