fix(code-preview): update overflow to auto IX-4305#2599
Conversation
|
✅ Deploy Preview for ix-storybook canceled.
|
|
Caution Review failedAn error occurred during the review process. Please try again later. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Code Review
This pull request updates the global CSS across multiple test applications (Angular Standalone, Angular, HTML, React, and Vue) by changing the html element's overflow property from hidden to auto. The reviewer correctly pointed out that combining width: 100vw with overflow: auto triggers an unwanted horizontal scrollbar when vertical scrolling is active, as 100vw includes the scrollbar width. To fix this layout issue, it is recommended to change width: 100vw to width: 100%.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|



💡 What is the current behavior?
Preview example pages in all five test apps set
overflow: hiddenon thehtmlelement inglobal.css. When example content exceeds the viewport (common in embedded code previews), overflow is clipped and the page cannot be scrolled.GitHub Issue Number: IX-4305 (branch
feature/IX-4305)🆕 What is the new behavior?
html { overflow: hidden; }is changed tooverflow: autoinsrc/preview-examples/global.cssacross all test apps.packages/html-test-apppackages/react-test-apppackages/vue-test-apppackages/angular-test-apppackages/angular-standalone-test-app🏁 Checklist
A pull request can only be merged if all of these conditions are met (where applicable):
pnpm test) — N/Apnpm lint)pnpm build, changes pushed)