Skip to content

Fix/trending heading z index#104

Merged
ketankauntia merged 4 commits into
ketankauntia:masterfrom
khushal-winner:fix/trending-heading-z-index
Feb 26, 2026
Merged

Fix/trending heading z index#104
ketankauntia merged 4 commits into
ketankauntia:masterfrom
khushal-winner:fix/trending-heading-z-index

Conversation

@khushal-winner
Copy link
Copy Markdown
Contributor

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

Summary

  • "Trending GSoC Organizations" was appearing on the back of the carousel on mobile phone

Testing

  • before
image - after image

Checklist

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

Summary by CodeRabbit

  • Style
    • Enhanced layout alignment and responsiveness across value blocks for improved display on smaller screens.
    • Improved visual layering in the trending section header.

- 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
- Add relative z-20 to Trending GSoC Organizations heading
- Ensures heading appears above carousel gradient overlay (z-10)
- Maintains existing styling while fixing visual hierarchy
@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

.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

Two UI components receive styling refinements: trending-orgs-client.tsx adds a z-index adjustment to its header, while value-blocks.tsx implements responsive text centering and alignment classes for improved mobile layout presentation.

Changes

Cohort / File(s) Summary
Styling & Z-index
components/trending-orgs-client.tsx
Minor comment formatting adjustment and addition of "relative z-20" classes to Heading component for z-index stacking control.
Responsive Layout
components/value-blocks.tsx
Added responsive centering classes (text-center, items-center) for smaller viewports while preserving lg:text-right and lg:text-left alignment on larger screens.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A hop through the styles, a z-index gain,
Centering content on small screens, champagne!
Layout responsive, left and right aligned,
No logic to break, just views refined. ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Fix/trending heading z index' directly addresses the main change—adding z-index to fix the heading visibility issue on mobile, matching the PR's primary objective.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Description check ✅ Passed The PR description includes a clear summary, testing evidence with before/after screenshots, and a completed checklist matching the template requirements.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 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.

@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:25pm

@ketankauntia ketankauntia merged commit ce16b4e into ketankauntia:master Feb 26, 2026
4 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