Ticket #2395 (closed defect: fixed)

Opened 9 years ago

Last modified 9 years ago

Cannot scroll by dragging mouse on panel

Reported by: egmont Owned by: andrew_b
Priority: minor Milestone: 4.7
Component: mc-core Version: 4.7.0.9
Keywords: Cc:
Blocked By: Blocking:
Branch state: Votes for changeset: committed-master committed-stable

Description (last modified by andrew_b) (diff)

Start mc, and make sure that at least one of the two displayed directiories contains so many files that they cannot all be displayed at once.
Click in this panel and hold the mouse button pressed, and then move your mouse downwards. As you pass beyond the bottom of this list, it should start scrolling.
Scrolling in the other direction (if you move your mouse upwards) works correctly.
This one used to work too, it's only broken since 4.7.0.6.

Attachments

mc-4.7.0.9-panel-drag-scroll.patch (451 bytes) - added by egmont 9 years ago.
seems to fix it

Change History

comment:1 Changed 9 years ago by egmont

Patch of ticket #2219 broke it.

Changed 9 years ago by egmont

seems to fix it

comment:2 Changed 9 years ago by egmont

This simple patch seems to fix it for me - I'm not sure whether it has any side effects, I haven't studied the code at all.

comment:3 Changed 9 years ago by andrew_b

  • Status changed from new to accepted
  • Owner set to andrew_b
  • Votes for changeset set to andrew_b
  • severity changed from no branch to on review
  • Keywords stable-candidate added

Seems there are no side effects. Thanks!

Created 2395_panel_scroll_drag branch. Parent branch is master.
changeset:1256af2269cd1b8d27ec40a65778fa0a1ec2b6e5

comment:4 Changed 9 years ago by angel_il

  • Votes for changeset changed from andrew_b to andrew_b angel_il
  • severity changed from on review to approved

comment:5 Changed 9 years ago by andrew_b

  • Status changed from accepted to testing
  • Keywords stable-candidate removed
  • Votes for changeset changed from andrew_b angel_il to committed-master committed-stable
  • Resolution set to fixed
  • severity changed from approved to merged

comment:6 Changed 9 years ago by andrew_b

  • Status changed from testing to closed
  • Description modified (diff)
Note: See TracTickets for help on using tickets.