Review Request 127587: [Kate view] Correct override cursor

[Kåre Särs]

> On April 8, 2016, 7:29 a.m., Kåre Särs wrote:
> > What is it that needs fixing?
> > 
> > I don't have a problem in either case listed.
> 
> Emmanuel Pescosta wrote:
>     > 3. Context menu is closed
>     
>     I can reproduce this problem
> 
> Anthony Fieroni wrote:
>     Drag file from Dolphin to Kate. When drag ends (drop or cancel) cursor stays pointer. Plays Kate on bottom right corner, maximize with double click on title => cursor stays pointer. Context menu has a stupid Qt bug, if cursor is over him and hit ESC cursor became caret, but if you cancel context menu and curson is not over him stays pointer. I try to fix it but setCursor not works in that way. https://bugreports.qt.io/browse/QTBUG-4514
> 
> Kåre Särs wrote:
>     Sorry I can not reproduce the problems even the more detailed description. This is on Kubuntu 15.10 with neon repos and on Windows.
>     
>     What version of Qt and distro are you using and is it Wayland/X11/...?
> 
> David Rosca wrote:
>     > 3. Context menu is closed
>     
>     This is regression in Qt 5.6 https://bugreports.qt.io/browse/QTBUG-49222
> 
> Anthony Fieroni wrote:
>     KaOS here, Qt 5.6, kf 5.21 Drag, maximize happens every time.
> 
> Anthony Fieroni wrote:
>     I can't image you can't reproduce since it has many ways to do it.
>     https://youtu.be/WYwhVDJKpMo

It seems quite clear to me that it is Qt 5.6 that has a regression in the xcb platform plugin. My kubuntu uses Qt 5.5.1 and does not seem to be effected. The patch makes it work, but it is not the "proper" fix. I hope Qt 5.6.1 will fix it, but if not we might have to make a temporary workaround... I feel a bit uncomfortable with adding workarounds that might not be needed in a couple of weeks...


- Kåre


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127587/#review94421
-----------------------------------------------------------


On April 8, 2016, 4:36 a.m., Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127587/
> -----------------------------------------------------------
> 
> (Updated April 8, 2016, 4:36 a.m.)
> 
> 
> Review request for Kate, KDE Frameworks, Dominik Haumann, and Kåre Särs.
> 
> 
> Bugs: 361075
>     https://bugs.kde.org/show_bug.cgi?id=361075
> 
> 
> Repository: ktexteditor
> 
> 
> Description
> -------
> 
> Currsor must be override after these conditions:
> 1. Drag move ends
> 2. Minimize / restore / maximize
> 3. Context menu is closed
> 
> 
> Diffs
> -----
> 
>   src/view/kateviewinternal.cpp ca5acfc 
> 
> Diff: https://git.reviewboard.kde.org/r/127587/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160408/9ff94528/attachment.html>