Skip to content

fix: prevent wheel scroll on class table from changing images - #111

Merged
developer0hye merged 1 commit into
masterfrom
fix/wheel-scroll-propagation
Feb 16, 2026
Merged

developer0hye merged 1 commit into
masterfrom
fix/wheel-scroll-propagation

Conversation

@developer0hye

Copy link
Copy Markdown
Owner

Summary

Scrolling the mouse wheel on the class list table rapidly navigates images when reaching the top or bottom of the table. This happens because wheel events propagate from QTableWidget to MainWindow::wheelEvent, which unconditionally triggers prev_img()/next_img().

Key Changes

  • Restrict wheel-based image navigation and zoom to only trigger when the cursor is over the image widget (label_image)
  • Scrolling on the class table, slider, or other UI elements no longer causes unintended image changes
  • Update README shortcuts table to clarify wheel actions require cursor over image

🤖 Generated with Claude Code

Wheel events that propagated from the class list table to
MainWindow caused rapid image navigation when scrolling past
the table's top or bottom. Restrict wheel-based image navigation
and zoom to only trigger when the cursor is over the image widget.

Signed-off-by: developer0hye <developer.0hye@gmail.com>

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: developer0hye <developer.0hye@gmail.com>
@developer0hye
developer0hye force-pushed the fix/wheel-scroll-propagation branch from 15adf9e to c24b1c9 Compare February 16, 2026 11:30
@developer0hye
developer0hye merged commit c4b0f68 into master Feb 16, 2026
5 checks passed
@developer0hye
developer0hye deleted the fix/wheel-scroll-propagation branch February 16, 2026 11:59
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.

1 participant