Skip to content

fix: mobile alignment for homepage feature blocks#103

Closed
khushal-winner wants to merge 3 commits into
ketankauntia:masterfrom
khushal-winner:feature/fix-mobile-alignment
Closed

fix: mobile alignment for homepage feature blocks#103
khushal-winner wants to merge 3 commits into
ketankauntia:masterfrom
khushal-winner:feature/fix-mobile-alignment

Conversation

@khushal-winner
Copy link
Copy Markdown
Contributor

@khushal-winner khushal-winner commented Feb 22, 2026

Summary

fix: mobile alignment for homepage feature blocks

Testing

  • before
image
  • after#
image

closes: #98

Checklist

  • One logical change per PR
  • PR description is complete
  • No refactors without prior discussion
  • Follows existing project structure

Summary by CodeRabbit

  • Style
    • Centered text and content across value blocks (Organizations, Previous Editions, Tech Stack, Analytics), replacing prior right-aligned layouts for improved visual consistency and readability.
    • Adjusted responsive alignment and text wrapping so headings and paragraphs present neatly at different screen sizes while preserving content order and meaning.

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 22, 2026

@khushal-winner is attempting to deploy a commit to the Ketan's Personal Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 22, 2026

Warning

Rate limit exceeded

@khushal-winner has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 13 minutes and 14 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

Warning

.coderabbit.yaml has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)
Validation error: Expected 'de' | 'de-DE' | 'de-AT' | 'de-CH' | 'en' | 'en-US' | 'en-AU' | 'en-GB' | 'en-CA' | 'en-NZ' | 'en-ZA' | 'es' | 'es-AR' | 'fr' | 'fr-CA' | 'fr-CH' | 'fr-BE' | 'nl' | 'nl-BE' | 'pt-AO' | 'pt' | 'pt-BR' | 'pt-MZ' | 'pt-PT' | 'ar' | 'ast-ES' | 'ast' | 'be-BY' | 'be' | 'br-FR' | 'br' | 'ca-ES' | 'ca' | 'ca-ES-valencia' | 'ca-ES-balear' | 'da-DK' | 'da' | 'de-DE-x-simple-language' | 'el-GR' | 'el' | 'eo' | 'fa' | 'ga-IE' | 'ga' | 'gl-ES' | 'gl' | 'it' | 'ja-JP' | 'ja' | 'km-KH' | 'km' | 'ko-KR' | 'ko' | 'pl-PL' | 'pl' | 'ro-RO' | 'ro' | 'ru-RU' | 'ru' | 'sk-SK' | 'sk' | 'sl-SI' | 'sl' | 'sv' | 'ta-IN' | 'ta' | 'tl-PH' | 'tl' | 'tr' | 'uk-UA' | 'uk' | 'zh-CN' | 'zh' | 'crh-UA' | 'crh' | 'cs-CZ' | 'cs' | 'nb' | 'no' | 'nl-NL' | 'de-DE-x-simple-language-DE' | 'es-ES' | 'it-IT' | 'fa-IR' | 'sv-SE' | 'de-LU' | 'fr-FR' | 'bg-BG' | 'bg' | 'he-IL' | 'he' | 'hi-IN' | 'hi' | 'vi-VN' | 'vi' | 'th-TH' | 'th' | 'bn-BD' | 'bn', received object at "language"
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Walkthrough

The value-blocks component layout was changed from right-aligned/off-center to centered alignment. Container and nested text elements now use centered alignment classes; paragraph line-wrapping was adjusted. No data-fetching, exports, or event handling were modified.

Changes

Cohort / File(s) Summary
Layout alignment
components/value-blocks.tsx
Replaced right-aligned/off-center utility classes with centered alignment (items-center, text-center, etc.) on primary and nested containers; adjusted heading/paragraph alignment and line-wrapping. No API, export, or logic changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I hopped along the layout trees,
Centered text now sways with ease,
Mobile hums a balanced tune,
Blocks align beneath the moon,
Joyful hops — a neat UX breeze ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: fixing mobile alignment for homepage feature blocks, which aligns with the changeset's focus on layout adjustments.
Description check ✅ Passed The description includes all required sections (Summary, Testing, Checklist) with before/after screenshots demonstrating the fix and addresses the linked issue #98.
Linked Issues check ✅ Passed The PR successfully addresses issue #98 by fixing mobile alignment of the four feature text blocks through layout adjustments (centering text and content), meeting the stated requirement.
Out of Scope Changes check ✅ Passed All changes are narrowly scoped to fixing mobile alignment through layout modifications (adding items-center, text-center classes) with no unrelated refactors or structural changes introduced.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@components/value-blocks.tsx`:
- Line 62: The left-image blocks aren't applying large-screen left alignment;
update the className on the container divs in PreviousEditionsBlock and
AnalyticsBlock (the divs currently using "flex gap-4 flex-col flex-1
items-center text-center") to include the large-screen overrides "lg:items-start
lg:text-left" so they mirror the right-image blocks' desktop behavior; ensure
both occurrences in PreviousEditionsBlock and AnalyticsBlock are changed.

Comment thread components/value-blocks.tsx Outdated
- Add lg:items-start lg:text-left to PreviousEditionsBlock container
- Add lg:items-start lg:text-left to AnalyticsBlock container
- Ensures left-image blocks mirror right-image blocks' desktop behavior
- Maintains mobile centering while adding proper desktop left alignment
- Add lg:text-right to OrganizationsBlock and TechStackBlock inner divs
- Add lg:text-left to PreviousEditionsBlock and AnalyticsBlock inner divs
- Ensures proper desktop alignment: right-image blocks right-align, left-image blocks left-align
- All blocks maintain mobile centering with consistent responsive behavior
@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 26, 2026

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

Project Deployment Actions Updated (UTC)
gsoc-orgs Ready Ready Preview, Comment Feb 26, 2026 1:23pm

Copy link
Copy Markdown
Owner

@ketankauntia ketankauntia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work @khushal-winner

@ketankauntia
Copy link
Copy Markdown
Owner

i feel this has already been resolved by you in #104 and hence i am closing this.

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.

Homepage Mobile view Ui fix required

2 participants