-
-
Notifications
You must be signed in to change notification settings - Fork 202
Fix Grid Scrollbar Position and DragScroll ID Mismatch #8983
Copy link
Copy link
Closed
Labels
bugSomething isn't workingSomething isn't working
Description
Problem:
- Vertical Scrollbar moves horizontally: The vertical scrollbar scrolls with the content instead of staying pinned.
- GridDragScroll Regression: Horizontal drag scrolling is broken because
ScrollManagerpasses the Component ID (id) instead of the rendered DOM ID (id__wrapper) to theGridDragScrolladdon.
Fix:
- Revert to
overflow-y: hidden: We changedresources/scss/src/grid/Container.scssfromoverflow-y: clipback tooverflow-y: hiddenon the inner.neo-grid-container.- Rationale:
clipforces the parent wrapper (which hasoverflow-x: auto) to handle the scrolling. Since the vertical scrollbar is an absolute child of that wrapper, it scrolls away with the content.hiddencreates a new Block Formatting Context (BFC) / Scroll Container on the inner element, trapping the wide content inside. This means the inner element scrolls, while the wrapper (and the scrollbar pinned to it) stays static.
- Rationale:
- Focus Scroll Prevention: The reason we originally moved to
clipwas to prevent the browser from scrolling the grid body into view when focused (which sometimes pushed headers out). We solved this properly insrc/grid/Body.mjsby passing{preventScroll: true}to thefocus()method inonRowClick.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working