Skip to content

[FIX] Restore new landing page design with scrollable right panel - #1823

Merged
Deepak-Kesavan merged 1 commit into
mainfrom
fix/landing-page-default-panel-scroll
Mar 4, 2026
Merged

[FIX] Restore new landing page design with scrollable right panel#1823
Deepak-Kesavan merged 1 commit into
mainfrom
fix/landing-page-default-panel-scroll

Conversation

@vishnuszipstack

@vishnuszipstack vishnuszipstack commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

What

  • Fix the default landing page right panel to allow vertical scrolling for the new design content

Why

  • PR [FIX] Revert ProductContentLayout to original simple component #1820 reverted ProductContentLayout.jsx back to the old "UNLOCK VALUE FROM UNSTRUCTURED DATA" design because the new design used dynamic import() which caused MIME type errors in the OSS Vite build
  • The correct fix keeps ProductContentLayout.jsx as the OSS fallback and renders the new design from the cloud plugin files which use proper static imports
  • The new "Parse, Structure and Automate" design has taller content (compliance badges + two product cards) that requires the right panel to be scrollable

How

  • Changed overflow: hidden to overflow-y: auto on .login-right-section.default-light in Login.css

Can this PR break any existing features. If yes, please list possible items. If no, please explain why. (PS: Admins do not merge the PR without this section filled)

  • No. This only affects the default landing page right panel scroll behavior. The OSS fallback (ProductContentLayout.jsx) still renders fine with the original content, and the cloud plugin content will now scroll properly when it exceeds viewport height.

Database Migrations

  • None

Env Config

  • None

Relevant Docs

  • None

Related Issues or PRs

Dependencies Versions

  • None

Notes on Testing

  • Visit landing page with no product query param — right panel should scroll if content overflows
  • Visit with ?selectedProduct=unstract — Unstract-specific design with region selector
  • Visit with ?selectedProduct=llm-whisperer — LLMWhisperer-specific design with region selector

Screenshots

Checklist

I have read and understood the Contribution Guidelines.

The new landing page design (Parse, Structure and Automate) has taller
content with product cards. Change overflow from hidden to overflow-y
auto so the right panel is scrollable when content exceeds viewport.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Frontend Lint Report (Biome)

All checks passed! No linting or formatting issues found.

@coderabbitai

coderabbitai Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Walkthrough

A CSS styling change to the Login component's right section, modifying the overflow property from hidden to auto on the vertical axis to enable scrolling within the section.

Changes

Cohort / File(s) Summary
Login Component Styling
frontend/src/components/log-in/Login.css
Modified .login-right-section.default-light overflow property to allow vertical scrolling while maintaining other layout styles.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: fixing the landing page design and making the right panel scrollable, which matches the CSS overflow change in the PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The pull request description is comprehensive and well-structured, covering all required template sections with clear, detailed information.

✏️ 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
  • Commit unit tests in branch fix/landing-page-default-panel-scroll

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.

@sonarqubecloud

sonarqubecloud Bot commented Mar 4, 2026

Copy link
Copy Markdown

@Deepak-Kesavan
Deepak-Kesavan merged commit cd51285 into main Mar 4, 2026
7 checks passed
@Deepak-Kesavan
Deepak-Kesavan deleted the fix/landing-page-default-panel-scroll branch March 4, 2026 06:03
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.

3 participants