D8822: Fix scrolling during inline renaming causes rename of wrong file

[Andreas Krutzler]

akrutzler added a comment.


  In https://phabricator.kde.org/D8822#168505, @rkflx wrote:
  
  > @akrutzler Thanks for the analysis, sounds like you are onto another fix :) If you are interested (if not, I can also recommend a few Gwenview bugs): https://bugs.kde.org/show_bug.cgi?id=334533#c7
  
  
  334533 seems to be a tricky one, but definitely related to this Model/View bug, lets see what i can do. I am not very familiar with gwenview as a user, but yes i can have a look at it :)

INLINE COMMENTS

> ngraham wrote in kitemlistroleeditor.cpp:59
> Not all users will be using a mouse with a wheel. Can we use `QEvent::Scroll` instead, so this continues to work with a laptop touchpad or touchscreen?

I tested this with my touchpad too and it works pretty well. But since i have no touchscreen available right now i cannot test this case. But then i have to catch QEvent::Wheel and QEvent::Scroll right? Because i don't get any scroll events when using mouse/touchpad.

REPOSITORY
  R318 Dolphin

BRANCH
  arcpatch-D8822

REVISION DETAIL
  https://phabricator.kde.org/D8822

To: akrutzler, ngraham, rkflx, #dolphin
Cc: elvisangelaccio, #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20171117/e2300020/attachment.htm>