Skip to content

[OPIK-5360] [FE] fix: collapse ResizableSidePanel width when closed to prevent scroll overflow#6003

Merged
andriidudar merged 1 commit intomainfrom
andriidudar/OPIK-5360-fix-scrollbar-blank-page
Mar 31, 2026
Merged

[OPIK-5360] [FE] fix: collapse ResizableSidePanel width when closed to prevent scroll overflow#6003
andriidudar merged 1 commit intomainfrom
andriidudar/OPIK-5360-fix-scrollbar-blank-page

Conversation

@andriidudar
Copy link
Copy Markdown
Contributor

@andriidudar andriidudar commented Mar 31, 2026

Details

Fix horizontal scroll showing a blank page when scrolling all the way to the right on any page with a table (Projects, Experiments, Datasets).

The ResizableSidePanel portal wrapper uses absolute inset-0, giving it full viewport width even when closed. While pointer-events-none blocked interactions, the element still contributed to the document's scrollable width. Adding w-0 when the panel is closed collapses the wrapper so it no longer extends the scroll area.

Change checklist

  • User facing
  • Documentation update

Issues

  • OPIK-5360

AI-WATERMARK

AI-WATERMARK: yes

  • Tools: Claude Code
  • Model(s): Claude Opus 4.6
  • Scope: assisted (validation of user's fix)
  • Human verification: code review + manual testing

Testing

  • Go to any page with a scrollable table (Projects, Experiments, Datasets)
  • Scroll all the way to the right — verify no blank page appears
  • Open a side panel (e.g. click a trace row) — verify it opens and renders correctly
  • Resize the side panel — verify drag handle works
  • Close the side panel — verify it closes with smooth animation
  • Verify arrow key navigation still works within the side panel

Documentation

N/A

…o prevent scroll overflow

The panel's portal wrapper used `absolute inset-0` which gave it full viewport width even when closed. While `pointer-events-none` blocked interactions, the element still contributed to the document's scrollable width, causing a blank page when scrolling all the way right. Adding `w-0` when closed collapses the wrapper so it no longer extends the scroll area.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@andriidudar andriidudar requested a review from a team as a code owner March 31, 2026 14:36
@github-actions
Copy link
Copy Markdown
Contributor

📋 PR Linter Failed

Missing Section. The description is missing the ## Details section.


Missing Section. The description is missing the ## Change checklist section.


Missing Section. The description is missing the ## Issues section.


Missing Section. The description is missing the ## Testing section.


Missing Section. The description is missing the ## Documentation section.

@andriidudar andriidudar added the test-environment Deploy Opik adhoc environment label Mar 31, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🔄 Test environment deployment process has started

Phase 1: Deploying base version 1.10.57-4758 (from main branch) if environment doesn't exist
Phase 2: Building new images from PR branch andriidudar/OPIK-5360-fix-scrollbar-blank-page
Phase 3: Will deploy newly built version after build completes

You can monitor the progress here.

@CometActions
Copy link
Copy Markdown
Collaborator

Test environment is now available!

To configure additional Environment variables for your environment, run [Deploy Opik AdHoc Environment workflow] (https://github.com/comet-ml/comet-deployment/actions/workflows/deploy_opik_adhoc_env.yaml)

Access Information

The deployment has completed successfully and the version has been verified.

@andriidudar andriidudar merged commit f99f731 into main Mar 31, 2026
26 of 27 checks passed
@andriidudar andriidudar deleted the andriidudar/OPIK-5360-fix-scrollbar-blank-page branch March 31, 2026 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Frontend test-environment Deploy Opik adhoc environment typescript *.ts *.tsx

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants