fix: mobile row layout

[toto04]

No description provided.

[coderabbitai]

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

Walkthrough

The url-record-row.tsx component receives responsive layout adjustments: MobileRow short and original URL links gain flex-1; the short URL anchor now contains two spans to show the full short URL on larger screens and /short_code on small screens.

Changes

Cohort / File(s)	Change Summary
Mobile responsive layout adjustments src/components/url-record-row.tsx	Added flex-1 to short and original URL anchors. Replaced single short-URL text with two spans (max-sm:hidden for full short URL, sm:hidden for /short_code) to control visibility across breakpoints and improve truncation behavior.

Pre-merge checks and finishing touches

❌ 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%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'fix: mobile row layout' directly corresponds to the main change, which is adjusting responsive layout and visibility for mobile breakpoints in the URL record row component.

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 621986d and a0f2c73.

📒 Files selected for processing (1)

• src/components/url-record-row.tsx (2 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• src/components/url-record-row.tsx

---

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

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

src/components/url-record-row.tsx (1)

41-56: Mobile short URL responsiveness looks good; consider consolidating anchors (optional).

The max-sm:hidden / sm:hidden pair correctly ensures only one short-link label is visible per breakpoint, and flex-1 should fix the cramped layout on small screens. If you want slightly cleaner semantics, you could keep a single <a> and swap the inner text with two spans using the same responsive visibility classes, so there’s only one focusable link element. Based on learnings, short codes are 8-character nanoids.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 86bc59e and 621986d.

📒 Files selected for processing (1)

• src/components/url-record-row.tsx (2 hunks)

🧰 Additional context used

📓 Path-based instructions (3)

**/*.{ts,tsx,js,jsx}

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Use 2-space indentation, semicolons as needed, and ES5 trailing commas; configure Biome for auto-formatting on save

Files:

• src/components/url-record-row.tsx

**/*.{ts,tsx}

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Auto-organize imports on save with Biome; exclude shadcn/ui components from Biome formatting

Files:

• src/components/url-record-row.tsx

src/components/**/*.{ts,tsx}

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

src/components/**/*.{ts,tsx}: Use shadcn/ui components with 'new-york' style and slate base color
Mark interactive components as client components; use React hooks for state management in client components
Use Sonner for toast notifications with standard error handling patterns
Call /api/urls endpoints directly from frontend components via standard fetch with JSON; handle errors via toast notifications

Files:

• src/components/url-record-row.tsx

🧠 Learnings (2)

📓 Common learnings

Learnt from: CR
Repo: PoliNetworkOrg/polinet.cc PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-12-05T21:25:30.007Z
Learning: Applies to src/app/[shortCode]/page.tsx : Implement URL redirection via Next.js dynamic routes at `src/app/[shortCode]/page.tsx`

📚 Learning: 2025-12-05T21:25:30.007Z

Learnt from: CR
Repo: PoliNetworkOrg/polinet.cc PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-12-05T21:25:30.007Z
Learning: Applies to src/lib/url-service.ts : Generate short codes as 8-character nanoids in `UrlService.createShortUrl()`

Applied to files:

• src/components/url-record-row.tsx

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Agent

🔇 Additional comments (1)

src/components/url-record-row.tsx (1)

61-66: flex-1 on the original URL anchor aligns layout and truncation behavior.

Adding flex-1 here should let the long URL take remaining space and truncate cleanly without fighting the arrow or copy button; the change is consistent with the updated first row and doesn’t introduce any functional risk.