[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: Review Request 113124: MouseEventListener: Don't process event from children multiple times.
From: "Kevin Funk" <krf () gmx ! de>
Date: 2013-11-18 9:49:44
Message-ID: 20131118094944.24899.88919 () vidsolbach ! de
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
> On Oct. 31, 2013, 3:27 p.m., Kevin Funk wrote:
> > Bump. Can we have this fixed soonish? Nicolas, can you push?
Bump again -- Without this fix, controlling volume in KMix via the mouse wheel is \
close to impossible. Can someone push?
- Kevin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113124/#review42747
-----------------------------------------------------------
On Oct. 6, 2013, 7:36 p.m., Nicolás Alvarez wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113124/
> -----------------------------------------------------------
>
> (Updated Oct. 6, 2013, 7:36 p.m.)
>
>
> Review request for Plasma.
>
>
> Bugs: 316546
> http://bugs.kde.org/show_bug.cgi?id=316546
>
>
> Repository: kde-runtime
>
>
> Description
> -------
>
> MouseEventListener: Don't process event from children multiple times.
>
> If an event passed through multiple visually-overlapping children of
> MouseEventListener, the event filter was processing the event every time.
> For example, this caused problems for the KMix icon in the tray, which
> would trigger volume changes three times for every mouse wheel event.
>
> BUG:316546
>
>
> Diffs
> -----
>
> plasma/declarativeimports/qtextracomponents/mouseeventlistener.cpp \
> 36530e075dc52348f32032c32457a090b904c3b9
> Diff: http://git.reviewboard.kde.org/r/113124/diff/
>
>
> Testing
> -------
>
> The KMix problem is gone, each mouse wheel step now changes the volume by the \
> appropriate amount.
>
> Thanks,
>
> Nicolás Alvarez
>
>
[Attachment #5 (text/html)]
<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 \
solid;"> <tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/113124/">http://git.reviewboard.kde.org/r/113124/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: \
10px;"> <p style="margin-top: 0;">On October 31st, 2013, 3:27 p.m. UTC, <b>Kevin \
Funk</b> wrote:</p> <blockquote style="margin-left: 1em; border-left: 2px solid \
#d0d0d0; padding-left: 10px;"> <pre style="white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">Bump. Can we have this fixed soonish? Nicolas, can you push?</pre> \
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Bump again -- Without \
this fix, controlling volume in KMix via the mouse wheel is close to impossible. Can \
someone push?</pre> <br />
<p>- Kevin</p>
<br />
<p>On October 6th, 2013, 7:36 p.m. UTC, Nicolás Alvarez wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); \
background-position: left top; background-repeat: repeat-x; border: 1px black \
solid;"> <tr>
<td>
<div>Review request for Plasma.</div>
<div>By Nicolás Alvarez.</div>
<p style="color: grey;"><i>Updated Oct. 6, 2013, 7:36 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=316546">316546</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kde-runtime
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" \
style="border: 1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">MouseEventListener: Don't process event from children multiple \
times.
If an event passed through multiple visually-overlapping children of
MouseEventListener, the event filter was processing the event every time.
For example, this caused problems for the KMix icon in the tray, which
would trigger volume changes three times for every mouse wheel event.
BUG:316546</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: \
1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">The KMix problem is gone, each mouse wheel step now changes the volume \
by the appropriate amount.</pre> </td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>plasma/declarativeimports/qtextracomponents/mouseeventlistener.cpp <span \
style="color: grey">(36530e075dc52348f32032c32457a090b904c3b9)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/113124/diff/" style="margin-left: \
3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic