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

[Henrik Fehlauer]

rkflx added a comment.


  Let's not get too sidetracked on the touchscreen topic. I'm not even sure Dolphin supports scrolling the view on a touchscreen without resorting to the scrollbar.
  
  Also not sure if fiddling with the painting code/visibility (i.e. what is related to expose events) helps us here.
  
  As I said, there are two approaches:
  
  - React to the scrolling after it happened (original patch, which is not pretty but works).
  - Catch the scrolling before it propagates (current patch, but maybe needs to be caught in a different place, i.e. you could try to investigate where `scrollOffsetChanged` is triggered from).

REPOSITORY
  R318 Dolphin

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

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