[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    D5490: Only follow mouse when moved (Fixes Bug #372635)
From:       Leszek Lesner <noreply () phabricator ! kde ! org>
Date:       2017-04-18 20:59:23
Message-ID: 20170418205922.22997.40913.3976E9CA () phabricator ! kde ! org
[Download RAW message or body]

leszeklesner edited the summary of this revision.

REPOSITORY
  R112 Milou

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

To: leszeklesner, broulik
Cc: ltoscano, qi437103, lfurmetz, anthonyfieroni, davidedmundson, plasma-devel, \
spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol


[Attachment #3 (text/html)]

<table><tr><td style="">leszeklesner edited the summary of this revision. <a \
href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-jl2cdoislqotmnm/" \
rel="noreferrer">(Show Details)</a> </td><a style="text-decoration: none; padding: \
4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; \
border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to \
bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" \
href="https://phabricator.kde.org/D5490" rel="noreferrer">View \
Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION \
SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">Use a new \
variable moved to detect if mouse moved and only change index if the mouse moved. \
This helps preventing index changes when only using keyboard to search something in \
milou and to not accidently start/open something you did not want (see bug report \
https://bugs.kde.org/show_bug.cgi?id=372635 )<br /> <br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, \
.6);">BUG: 372635<br /> <br />
</span>In general the onEntered signal seems to be broken in Qt somehow as I could \
not make it work reliably with Qt 5.7.1. Sometimes it worked but mostly the code \
using onEntered failed to work with onPositionChanged (I guess this also does not \
always set it to true).<div style="padding: 8px 0;">...</div></div></div></div><br \
/><div><strong>REPOSITORY</strong><div><div>R112 Milou</div></div></div><br \
/><div><strong>REVISION DETAIL</strong><div><a \
href="https://phabricator.kde.org/D5490" \
rel="noreferrer">https://phabricator.kde.org/D5490</a></div></div><br \
/><div><strong>To: </strong>leszeklesner, broulik<br /><strong>Cc: </strong>ltoscano, \
qi437103, lfurmetz, anthonyfieroni, davidedmundson, plasma-devel, spstarr, progwolff, \
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>



[prev in list] [next in list] [prev in thread] [next in thread]