Skip to content

Fix PageCard description text overflow#176

Merged
NaysKutzu merged 1 commit into
MythicalLTD:mainfrom
Magma-Host:patch-4
May 17, 2026
Merged

Fix PageCard description text overflow#176
NaysKutzu merged 1 commit into
MythicalLTD:mainfrom
Magma-Host:patch-4

Conversation

@Magma-Host
Copy link
Copy Markdown
Contributor

@Magma-Host Magma-Host commented May 17, 2026

Changed the CSS utility class from truncate to line-clamp-2 for the card description. This ensures long text wraps onto a maximum of two lines with an ellipsis instead of clipping horizontally and breaking the card layout.

Summary by CodeRabbit

  • Style
    • Updated description text rendering in page cards to use line-clamping instead of truncation for improved text display.

Review Change Stack

Changed the CSS utility class from `truncate` to `line-clamp-2` for the card description. This ensures long text wraps onto a maximum of two lines with an ellipsis instead of clipping horizontally and breaking the card layout.
@Magma-Host Magma-Host requested a review from NaysKutzu as a code owner May 17, 2026 13:19
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5426078a-07a3-4e91-ae8f-8b13ebbe0a3b

📥 Commits

Reviewing files that changed from the base of the PR and between 32c8ce1 and b970c0d.

📒 Files selected for processing (1)
  • frontendv2/src/components/featherui/PageCard.tsx

Walkthrough

PageCard's description now displays up to two lines of text instead of being truncated to one. The paragraph element swapped from truncate to line-clamp-2, keeping all existing text styling intact while respecting vertical layout constraints.

Changes

PageCard Description Styling

Layer / File(s) Summary
Line-clamped description rendering
frontendv2/src/components/featherui/PageCard.tsx
Description paragraphs now clamp to 2 lines via line-clamp-2 instead of truncating, preserving muted foreground, small text, uppercase tracking, and opacity styling.

Look, someone finally realized that jamming multi-sentence descriptions into a single truncated line was about as user-friendly as a deploy on Friday at 4:59 PM. Now descriptions can breathe for a whole two lines before getting the guillotine. It's not rocket science, but it's the kind of small UX win that doesn't torpedo someone's weekend trying to debug why their carefully-written copy got mangled. Ship it and move on. 🔥

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and clearly describes the main change: replacing truncate with line-clamp-2 to fix text overflow behavior in PageCard description rendering.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@NaysKutzu NaysKutzu merged commit 6114eb7 into MythicalLTD:main May 17, 2026
9 checks passed
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