Skip to content

Company icon alignment#3908

Open
marcusljf wants to merge 1 commit into
mainfrom
partner-company-name
Open

Company icon alignment#3908
marcusljf wants to merge 1 commit into
mainfrom
partner-company-name

Conversation

@marcusljf

@marcusljf marcusljf commented May 14, 2026

Copy link
Copy Markdown
Collaborator

Aligning the company icon to the start instead of center, so if the company name wraps, the icon is aligned with the first line.

After

CleanShot 2026-05-14 at 15 37 53@2x

Before

CleanShot 2026-05-14 at 15 37 17@2x

Summary by CodeRabbit

  • Style
    • Refined partner information card layout with improved element alignment and spacing.

Review Change Stack

Aligning the company icon to the start instead of center, so if the company name wraps, the icon is aligned with the first line.
@vercel

vercel Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
dub Ready Ready Preview May 14, 2026 10:46pm

Request Review

@coderabbitai

coderabbitai Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d03c6c29-1a2c-4861-81c8-f0e96e68cecb

📥 Commits

Reviewing files that changed from the base of the PR and between 263fff9 and 1a07b1b.

📒 Files selected for processing (1)
  • apps/web/ui/partners/partner-info-cards.tsx

📝 Walkthrough

Walkthrough

The partner info card component adjusts its basic field row layout by changing the flex container alignment from center to top alignment and refactoring the icon wrapper markup into a separate span element.

Changes

Field Row Alignment

Layer / File(s) Summary
Basic field row layout adjustment
apps/web/ui/partners/partner-info-cards.tsx
Row container alignment shifts from items-center to items-start, and icon wrapping is refactored into a dedicated span element within the row.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • pepeladeira
  • steven-tey

Poem

A rabbit adjusts the layout with care, 🐰
Icons now start from the top of the air,
No longer centered, but aligned just right,
The info cards dance with improved sight! ✨

🚥 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 pull request title 'Company icon alignment' directly and specifically describes the main change: adjusting the alignment of company icons from center to start to handle text wrapping properly.
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 docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch partner-company-name

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

apps/web/ui/partners/partner-info-cards.tsx

Parsing error: The keyword 'import' is reserved


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.

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