summaryrefslogtreecommitdiff
authorsimon <simon>2002-12-09 20:02:52 (UTC)
committer simon <simon>2002-12-09 20:02:52 (UTC)
commit2d2d57e3b7988f4246319cdfd0dc7fa90110db8e (patch) (side-by-side diff)
treefb4dfb832e2630e7da1cc57e92ba6c62ead22dee
parent046f040efcb0f313818d1dbb1dc1db9021678175 (diff)
downloadopie-2d2d57e3b7988f4246319cdfd0dc7fa90110db8e.zip
opie-2d2d57e3b7988f4246319cdfd0dc7fa90110db8e.tar.gz
opie-2d2d57e3b7988f4246319cdfd0dc7fa90110db8e.tar.bz2
- reworked the mouse event handling to be much simpler
- fixed missing return in Play case in handleCommand, which accidentially broke pausing - accept the key events we receive in the audioWidget - commented out suspicious connect to pauseCheck in mediaplayer, which broke pausing via space bar
Diffstat (more/less context) (show whitespace changes)
-rw-r--r--noncore/multimedia/opieplayer2/audiowidget.cpp8
-rw-r--r--noncore/multimedia/opieplayer2/mediaplayer.cpp3
-rw-r--r--noncore/multimedia/opieplayer2/mediawidget.cpp78
-rw-r--r--noncore/multimedia/opieplayer2/mediawidget.h3
4 files changed, 48 insertions, 44 deletions
diff --git a/noncore/multimedia/opieplayer2/audiowidget.cpp b/noncore/multimedia/opieplayer2/audiowidget.cpp
index 5986a72..8e9bbf1 100644
--- a/noncore/multimedia/opieplayer2/audiowidget.cpp
+++ b/noncore/multimedia/opieplayer2/audiowidget.cpp
@@ -346,18 +346,22 @@ void AudioWidget::keyReleaseEvent( QKeyEvent *e) {
case Key_F9: //activity
hide();
// qDebug("Audio F9");
+ e->accept();
break;
case Key_F10: //contacts
break;
case Key_F11: //menu
mediaPlayerState.toggleBlank();
+ e->accept();
break;
case Key_F12: //home
break;
case Key_F13: //mail
mediaPlayerState.toggleBlank();
+ e->accept();
break;
case Key_Space: {
+ e->accept();
mediaPlayerState.togglePaused();
}
break;
@@ -366,22 +370,26 @@ void AudioWidget::keyReleaseEvent( QKeyEvent *e) {
emit lessClicked();
emit lessReleased();
// toggleButton(6);
+ e->accept();
break;
case Key_Up:
// toggleButton(5);
emit moreClicked();
emit moreReleased();
// toggleButton(5);
+ e->accept();
break;
case Key_Right:
// toggleButton(3);
mediaPlayerState.setNext();
// toggleButton(3);
+ e->accept();
break;
case Key_Left:
// toggleButton(4);
mediaPlayerState.setPrev();
// toggleButton(4);
+ e->accept();
break;
case Key_Escape: {
}
diff --git a/noncore/multimedia/opieplayer2/mediaplayer.cpp b/noncore/multimedia/opieplayer2/mediaplayer.cpp
index ed7e37f..e1bfc2d 100644
--- a/noncore/multimedia/opieplayer2/mediaplayer.cpp
+++ b/noncore/multimedia/opieplayer2/mediaplayer.cpp
@@ -45,7 +45,8 @@ MediaPlayer::MediaPlayer( PlayListWidget &_playList, MediaPlayerState &_mediaPla
connect( &mediaPlayerState, SIGNAL( playingToggled( bool ) ), this, SLOT( setPlaying( bool ) ) );
- connect( &mediaPlayerState, SIGNAL( pausedToggled( bool ) ), this, SLOT( pauseCheck( bool ) ) );
+// What is pauseCheck good for? (Simon)
+// connect( &mediaPlayerState, SIGNAL( pausedToggled( bool ) ), this, SLOT( pauseCheck( bool ) ) );
connect( &mediaPlayerState, SIGNAL( next() ), this, SLOT( next() ) );
connect( &mediaPlayerState, SIGNAL( prev() ), this, SLOT( prev() ) );
diff --git a/noncore/multimedia/opieplayer2/mediawidget.cpp b/noncore/multimedia/opieplayer2/mediawidget.cpp
index 439ba2e..4c4cead 100644
--- a/noncore/multimedia/opieplayer2/mediawidget.cpp
+++ b/noncore/multimedia/opieplayer2/mediawidget.cpp
@@ -69,58 +69,52 @@ void MediaWidget::paintEvent( QPaintEvent *pe )
}
}
-void MediaWidget::mouseMoveEvent( QMouseEvent *event )
+MediaWidget::Button *MediaWidget::buttonAt( const QPoint &position )
{
- for ( ButtonVector::iterator it = buttons.begin(); it != buttons.end(); ++it ) {
- Button &button = *it;
- Command command = button.command;
+ if ( position.x() <= 0 || position.y() <= 0 ||
+ position.x() >= buttonMask.width() ||
+ position.y() >= buttonMask.height() )
+ return 0;
- if ( event->state() == QMouseEvent::LeftButton ) {
- // The test to see if the mouse click is inside the button or not
- bool isOnButton = isOverButton( event->pos() - upperLeftOfButtonMask, command );
+ int pixelIdx = buttonMask.pixelIndex( position.x(), position.y() );
+ for ( ButtonVector::iterator it = buttons.begin(); it != buttons.end(); ++it )
+ if ( it->command + 1 == pixelIdx )
+ return &( *it );
- if ( isOnButton && !button.isHeld ) {
- button.isHeld = TRUE;
- toggleButton( button );
- switch ( command ) {
- case VolumeUp:
- emit moreClicked();
- return;
- case VolumeDown:
- emit lessClicked();
- return;
- case Forward:
- emit forwardClicked();
- return;
- case Back:
- emit backClicked();
- return;
- default: break;
- }
- } else if ( !isOnButton && button.isHeld ) {
- button.isHeld = FALSE;
- toggleButton( button );
- }
- } else {
- if ( button.isHeld ) {
- button.isHeld = FALSE;
- if ( button.type != ToggleButton ) {
- setToggleButton( button, FALSE );
- }
- handleCommand( command, button.isDown );
- }
- }
- }
+ return 0;
}
void MediaWidget::mousePressEvent( QMouseEvent *event )
{
- mouseMoveEvent( event );
+ Button *button = buttonAt( event->pos() - upperLeftOfButtonMask );
+
+ if ( !button ) {
+ QWidget::mousePressEvent( event );
+ return;
+ }
+
+ switch ( button->command ) {
+ case VolumeUp: emit moreClicked(); return;
+ case VolumeDown: emit lessClicked(); return;
+ case Back: emit backClicked(); return;
+ case Forward: emit forwardClicked(); return;
+ default: break;
+ }
}
void MediaWidget::mouseReleaseEvent( QMouseEvent *event )
{
- mouseMoveEvent( event );
+ Button *button = buttonAt( event->pos() - upperLeftOfButtonMask );
+
+ if ( !button ) {
+ QWidget::mouseReleaseEvent( event );
+ return;
+ }
+
+ if ( button->type == ToggleButton )
+ toggleButton( *button );
+
+ handleCommand( button->command, button->isDown );
}
void MediaWidget::makeVisible()
@@ -130,7 +124,7 @@ void MediaWidget::makeVisible()
void MediaWidget::handleCommand( Command command, bool buttonDown )
{
switch ( command ) {
- case Play: mediaPlayerState.togglePaused();
+ case Play: mediaPlayerState.togglePaused(); return;
case Stop: mediaPlayerState.setPlaying(FALSE); return;
case Next: if( playList.currentTab() == PlayListWidget::CurrentPlayList ) mediaPlayerState.setNext(); return;
case Previous: if( playList.currentTab() == PlayListWidget::CurrentPlayList ) mediaPlayerState.setPrev(); return;
diff --git a/noncore/multimedia/opieplayer2/mediawidget.h b/noncore/multimedia/opieplayer2/mediawidget.h
index 9042d5b..8031371 100644
--- a/noncore/multimedia/opieplayer2/mediawidget.h
+++ b/noncore/multimedia/opieplayer2/mediawidget.h
@@ -87,7 +87,8 @@ protected:
virtual void paintEvent( QPaintEvent *pe );
- virtual void mouseMoveEvent( QMouseEvent *event );
+ Button *buttonAt( const QPoint &position );
+
virtual void mousePressEvent( QMouseEvent *event );
virtual void mouseReleaseEvent( QMouseEvent *event );