ggarra13 / mrv2

Professional player and review tool for vfx, animation and computer graphics.
https://mrv2.sourceforge.io
BSD 3-Clause "New" or "Revised" License
190 stars 13 forks source link

Mouse-wheel zoom depends on side panel scroll bar #249

Closed Thane5 closed 2 months ago

Thane5 commented 2 months ago

Small UX annoyance:

I docked my "Media Information" panel to the right side of the screen. When i hover my cursor over the panel and scroll, two things can happen:

This feels pretty inconsistent, so i suggest disabling the image zoom when the cursor is outside of the image display area or over a panel.

ggarra13 commented 2 months ago

Here you go for you to try (also check I spelled your name properly in the Help->About->Thanks).

.exe: https://mega.nz/file/bPhU0BSL#Pk36ID_Hj65QN3nuSHwDfUkMs5s1ROqZl4n8VwLtnDc

.zip: https://mega.nz/file/PKZVlQAQ#XRneZ0dqIFgBjEjuGP0OZmo11ue1Zdnm8CyPMM_rejQ

Thane5 commented 2 months ago

There seems to be some bug where scrolling a panel wont work, even if the scroll bar is displayed.

Steps to reproduce:

  1. collapse all categories in media info panel
  2. un-collapse them until the scrollbar appears
  3. the mouse wheel will now do nothing

It seems that hovering or clicking on the scrollbar fixes it sometimes, but i'm not really sure yet what the requirements are for that to happen.

ggarra13 commented 2 months ago

I can confirm the bug in v1.1.9, but it was always there in v1.1.8 too. This will need some more time to fix.

ggarra13 commented 2 months ago

Ok. This should fix the scroll issue as well as resizing problems when the Media Info Panel was in a window. There are still some minor redraw issues when in a window, which I can't seem to get rid of.

.exe: https://mega.nz/file/DTAmSCoA#ScatFqUdmdzmpjkwKxGI4ng2SL0EkFACwGdfkJj2HOg

.zip: https://mega.nz/file/2bwkDB5Y#zg0X1pgxmOrS8LiMaYlK7fruVfSJkzdxmpVtkOWUZTA

Thane5 commented 2 months ago

Alright that version looks good now.