Skip to content

fix: remove rounded corners on doc card links#46

Merged
charlesrhoward merged 1 commit into
mainfrom
feat/remove-card-link-rounding
May 19, 2026
Merged

fix: remove rounded corners on doc card links#46
charlesrhoward merged 1 commit into
mainfrom
feat/remove-card-link-rounding

Conversation

@charlesrhoward
Copy link
Copy Markdown
Contributor

@charlesrhoward charlesrhoward commented May 19, 2026

Summary

  • Set border-radius: 0 on [data-card='true'] so doc card links render as sharp rectangles.

Test plan

  • pnpm lint
  • pnpm types:check
  • pnpm build

View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews

@charlesrhoward charlesrhoward enabled auto-merge (squash) May 19, 2026 22:25
@vercel
Copy link
Copy Markdown

vercel Bot commented May 19, 2026

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

Project Deployment Actions Updated (UTC)
mogplex-docs Ready Ready Preview, Comment May 19, 2026 10:25pm

Request Review

@mogplex
Copy link
Copy Markdown

mogplex Bot commented May 19, 2026

Mogplex PR Review

Status: No material issues found

Summary

A single-line CSS change — replacing border-radius: 0.5rem with border-radius: 0 on #nd-page [data-card='true']. The fix is correct, matches the PR description exactly, and is stylistically consistent with the rest of the theme (the design intentionally uses sharp, flat surfaces throughout global.css). No logic, security, or architectural concerns.

Critical Issues

None.

Warnings

None.

Suggestions

None.

Verdict

APPROVE — Minimal, focused, and correct. Nothing to flag.

Affected files:

  • src/app/global.css

View check run

@charlesrhoward charlesrhoward merged commit 6d21c27 into main May 19, 2026
6 checks passed
@charlesrhoward charlesrhoward deleted the feat/remove-card-link-rounding branch May 19, 2026 22:26
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.

1 participant