Bug 141672 - right-clicking on sound control changes volume
Summary: right-clicking on sound control changes volume
Status: RESOLVED FIXED
Alias: None
Product: amarok
Classification: Applications
Component: general (show other bugs)
Version: 1.4.5
Platform: FreeBSD Ports FreeBSD
: NOR normal
Target Milestone: ---
Assignee: Amarok Developers
URL:
Keywords:
: 141948 (view as bug list)
Depends on:
Blocks:
 
Reported: 2007-02-14 09:20 UTC by Silver Salonen
Modified: 2007-02-19 22:58 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Silver Salonen 2007-02-14 09:20:24 UTC
Version:            (using KDE KDE 3.5.5)
Installed from:    FreeBSD Ports
OS:                FreeBSD

In previous versions volume was changed when clicking with left button. Now, in 1.4.5, the right button both changes volume and shows the volume menu.
Comment 1 Mark Kretschmann 2007-02-14 10:14:01 UTC
SVN commit 633527 by markey:

Fix regression: right-clicking on sound control would change volume

BUG: 141672


 M  +9 -0      sliderwidget.cpp  
 M  +1 -0      sliderwidget.h  


--- branches/stable/extragear/multimedia/amarok/src/sliderwidget.cpp #633526:633527
@@ -414,6 +414,15 @@
 }
 
 void
+Amarok::VolumeSlider::mousePressEvent( QMouseEvent *e )
+{
+    if( e->button() != RightButton ) {
+        Amarok::Slider::mousePressEvent( e );
+        slideEvent( e );
+    }
+}
+                
+void
 Amarok::VolumeSlider::contextMenuEvent( QContextMenuEvent *e )
 {
     KPopupMenu menu;
--- branches/stable/extragear/multimedia/amarok/src/sliderwidget.h #633526:633527
@@ -129,6 +129,7 @@
             virtual void leaveEvent( QEvent* );
             virtual void paletteChange( const QPalette& );
             virtual void slideEvent( QMouseEvent* );
+            virtual void mousePressEvent( QMouseEvent* );
             virtual void contextMenuEvent( QContextMenuEvent* );
             virtual void wheelEvent( QWheelEvent *e );
 
Comment 2 Mark Kretschmann 2007-02-14 10:29:27 UTC
SVN commit 633534 by markey:

Forward-port bugfix from stable branch: Right-click on volume slider would change volume

CCBUG: 141672


 M  +9 -0      sliderwidget.cpp  
 M  +1 -0      sliderwidget.h  


--- trunk/extragear/multimedia/amarok/src/sliderwidget.cpp #633533:633534
@@ -441,6 +441,15 @@
 }
 
 void
+Amarok::VolumeSlider::mousePressEvent( QMouseEvent *e )
+{
+    if( e->button() != Qt::RightButton ) {
+        Amarok::Slider::mousePressEvent( e );
+        slideEvent( e );
+    }
+}
+                
+void
 Amarok::VolumeSlider::contextMenuEvent( QContextMenuEvent *e )
 {
     Q3PopupMenu menu;
--- trunk/extragear/multimedia/amarok/src/sliderwidget.h #633533:633534
@@ -136,6 +136,7 @@
             virtual void leaveEvent( QEvent* );
             virtual void paletteChange( const QPalette& );
             virtual void slideEvent( QMouseEvent* );
+            virtual void mousePressEvent( QMouseEvent* );
             virtual void contextMenuEvent( QContextMenuEvent* );
             virtual void wheelEvent( QWheelEvent *e );
 
Comment 3 Kevin Funk 2007-02-19 22:58:14 UTC
*** Bug 141948 has been marked as a duplicate of this bug. ***