Skip to content

fix: show full AI model card text - #33369

Open
superhighfives wants to merge 10 commits into
productionfrom
fix/ai-model-card-text-overflow
Open

fix: show full AI model card text#33369
superhighfives wants to merge 10 commits into
productionfrom
fix/ai-model-card-text-overflow

Conversation

@superhighfives

@superhighfives superhighfives commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Defect

AI model cards on /ai/models visually clipped titles and descriptions and showed ellipses after the second line.

Fix

  • Remove forced line-clamp-2 from model-card titles and descriptions.
  • Add min-w-0 so long titles wrap beside the logo and Beta badge.
  • Preserve author truncation and shared card/grid primitives.
  • Add focused Astro regression coverage for the unclamped title and description contract.

Validation

  • Hosted Pre Build: passed Astro check, Worker TypeScript check, ESLint, Prettier, and 125 Node/Astro tests.
  • Local rendered model-page smoke checks: no card line-clamp-2; author truncation retained.
  • The local worker-suite run was blocked by Miniflare’s 25 MiB per-asset limit on a generated 52 MiB build asset. This local build-artifact limitation is unrelated to this change.

@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Review

✅ No issues found in commit d8ec352.

Code Review

This code review is in beta and may not always be helpful — use your judgment.

No code review issues found.

Conventions

No convention issues found.

Style Guide Review

No style-guide issues found.

Commands

Only codeowners can run commands. Post a comment with the command to trigger it.

Command Description
/review Runs a review now. Incremental if a prior review exists, full if not.
/full-review Re-reviews the entire PR diff from scratch, ignoring incremental history. Useful after a rebase, when you want a fresh review, or if the bot gets out of sync and reports issues that no longer exist.
/ignore-review-limit Permanently lifts the 2-review automatic limit for this PR. Future pushes will trigger reviews as normal.
/disable-auto-review Stops automatic reviews from triggering on future pushes to this PR. Codeowners can still run /review or /full-review manually.
/rebase Rebases the PR branch against production. On conflict, attempts to resolve automatically using AI. Stops with an explanation if confidence is not high enough.

@github-actions

Copy link
Copy Markdown
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/components/models/ @ethulia, @kflansburg, @mchenco, @shridhar-cf, @superhighfives, @cloudflare/product-owners, @cloudflare/content-engineering

@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 10, 2026

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://fix-ai-model-card-text-overflow.previews.developers.cloudflare.com (commit d8ec352)

This URL reflects your latest Preview deployment

Preview Deployments by commit

Status Deployment URL Commit Updated (UTC) See this deployment's details
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://52086bb8.previews.developers.cloudflare.com d8ec352 2026-09-10T21:22:19.866Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://b6a33ffe.previews.developers.cloudflare.com e36a6a0 2026-09-10T20:22:37.336Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://221a1d3c.previews.developers.cloudflare.com d685b2f 2026-09-10T19:31:27.756Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://6410e087.previews.developers.cloudflare.com d74db33 2026-09-10T18:48:29.909Z Visit the dashboard ↗
  • Build: In progress 🔵

View logs ↗
b18746b 2026-09-10T18:25:28.752Z View logs ↗
  • Build: In progress 🔵

View logs ↗
f5debfe 2026-09-10T17:55:09.736Z View logs ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://1c765ff5.previews.developers.cloudflare.com e11808c 2026-09-10T17:46:34.067Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://bfc19480.previews.developers.cloudflare.com 9a848d3 2026-09-10T17:07:25.582Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://9030c2dd.previews.developers.cloudflare.com c16b26c 2026-09-10T16:19:10.536Z Visit the dashboard ↗

@superhighfives
superhighfives force-pushed the fix/ai-model-card-text-overflow branch 4 times, most recently from f5debfe to c88ae0b Compare September 10, 2026 18:15

Copy link
Copy Markdown
Contributor Author

/full-review

2 similar comments

Copy link
Copy Markdown
Contributor Author

/full-review

Copy link
Copy Markdown
Contributor Author

/full-review

@superhighfives
superhighfives force-pushed the fix/ai-model-card-text-overflow branch 2 times, most recently from d685b2f to e36a6a0 Compare September 10, 2026 20:03

Copy link
Copy Markdown
Contributor Author

/full-review

@superhighfives
superhighfives force-pushed the fix/ai-model-card-text-overflow branch from e36a6a0 to d8ec352 Compare September 10, 2026 21:14
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.

5 participants