Skip to content

dbeaver/pro#9659 add dnd scroll container#4476

Merged
devnaumov merged 2 commits into
develfrom
dbeaver/pro#9659-tree-auto-scroll
Jul 22, 2026
Merged

dbeaver/pro#9659 add dnd scroll container#4476
devnaumov merged 2 commits into
develfrom
dbeaver/pro#9659-tree-auto-scroll

Conversation

@devnaumov

@devnaumov devnaumov commented Jul 20, 2026

Copy link
Copy Markdown
Member

closes 9659

CleanShot.2026-07-21.at.01.01.36.mp4

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 13 complexity

Metric Results
Complexity 13

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

<>
<ElementsTreeTools tree={tree} settingsElements={settingsElements} />
<div ref={treeMergedRef} className={s(styles, { treeBox: true })}>
<DNDScrollContainer ref={treeMergedRef} className={s(styles, { treeBox: true })} isDragging={!!dndBox.state.context}>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The best solution here is to invent something tricky with html/css so it triggers scroll. Cause it already works fine, but this zone is too small
I can see why you want JS to handle that. But feel like there is a native html/css solution, it's just tricky to find

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I couldn't find any native solution. Everyone mentions that the native solution has a very small scrollable area. If you know of any solutions, feel free to suggest them.

@devnaumov
devnaumov merged commit 81a9be1 into devel Jul 22, 2026
5 of 6 checks passed
@devnaumov
devnaumov deleted the dbeaver/pro#9659-tree-auto-scroll branch July 22, 2026 12:24
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.

2 participants