Skip to content

Conversation

@dantovska
Copy link
Contributor

@dantovska dantovska commented Dec 5, 2025

What

Fix scroll issue where Y-axis scroll does not appear when tutorials exceed container height on database list page.

  • Add height: 100% to ExplorePanelWrapper to ensure proper flex container chain
  • Ensures height constraint chain is complete from PageWrapperExplorePanelWrapperExplorePanelTemplate → side panels
  • Fixes scroll functionality on database list page while maintaining existing behavior on browser page

Testing

  1. Open database list page
  2. Click lightbulb icon to open tutorials panel
  3. Expand tutorials or add custom tutorials to make content exceed container height
  4. Verify Y-axis scroll bar appears and works correctly
  5. Verify all tutorial content is accessible via scrolling
  6. Regression test: Open browser page, verify scroll still works (no regression)
Before After
image image

Technical Details

The issue was caused by an incomplete flex container height chain. The ExplorePanelWrapper component needed height: 100% to properly participate in the flex layout and maintain height constraints down to the scroll container in the Navigation component.

Files Changed:

  • redisinsight/ui/src/templates/home-page-template/HomePageTemplate.styles.ts: Added height: 100% to ExplorePanelWrapper

Closes #RI-7853


Note

Adds height: 100% to ExplorePanelWrapper to complete the flex height chain and enable Y-axis scrolling in the tutorials panel on the database list page.

Written by Cursor Bugbot for commit a1d5367. This will update automatically on new commits. Configure here.

- Add height: 100% to ExplorePanelWrapper to ensure proper flex container chain
- Fixes scroll issue on database list page when tutorials exceed container height
- Maintains height constraint chain from PageWrapper to side panels

Fixes #RI-7853
@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2025

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 82.83% 21078/25448
🟡 Branches 67.99% 8869/13044
🟡 Functions 78% 5777/7406
🟢 Lines 83.23% 20644/24803

Test suite run success

5481 tests passing in 703 suites.

Report generated by 🧪jest coverage report action from a1d5367

@dantovska dantovska merged commit 7001541 into main Dec 5, 2025
28 checks passed
@dantovska dantovska deleted the fe/bugfix/RI-7853/tutorials-scroll-db-list-page branch December 5, 2025 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants