D21156: Fix browse mode not disabled after Ctrl is released

Simone Gaiarin noreply at phabricator.kde.org
Wed May 15 14:36:52 BST 2019

simgunz added a comment.

  With the Alt key the bug does not appear, because the existing code disables correctly the browse mode when Alt is released. For some reasons key press and release events are different for Ctrl key but are the same for Alt. Probably the method avoidMenuAltFocus manages this behavior correctly for the Alt key (I have not analyzed in detail what it does) and then we enter correctly in the if at line 230 where source browse mode is disabled. So removing the redundant Ctrl key solves the bug completely (at least I haven't manage to reproduce it).

  R32 KDevelop


To: simgunz, #kdevelop
Cc: rjvbb, kdevelop-devel, christiant, glebaccon, domson, antismap, iodelay, alexeymin, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20190515/7e34c0a6/attachment-0001.html>

More information about the KDevelop-devel mailing list