Bug 147199 - moodbar layered on top of volume control
Summary: moodbar layered on top of volume control
Status: RESOLVED FIXED
Alias: None
Product: amarok
Classification: Applications
Component: Notifications (show other bugs)
Version: unspecified
Platform: Slackware Linux
: NOR normal
Target Milestone: ---
Assignee: Amarok Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-25 14:26 UTC by richlv
Modified: 2007-06-27 17:05 UTC (History)
0 users

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


Attachments
screenshot (36.26 KB, image/png)
2007-06-25 14:27 UTC, richlv
Details

Note You need to log in before you can comment on or make changes to this bug.
Description richlv 2007-06-25 14:26:13 UTC
Version:           svn rev 680033, stable branch (using KDE KDE 3.5.7)
Installed from:    Slackware Packages
OS:                Linux

see the screenshot. this bug has been there before, but it was very rare to encounter. it seems that latest changes to osd (rating related) have made it reproducible all the time.

in an essence, volume control bar is too high, and has moodbar overlayed at the bottom.
Comment 1 richlv 2007-06-25 14:27:19 UTC
Created attachment 20951 [details]
screenshot
Comment 2 Mark Kretschmann 2007-06-25 15:02:36 UTC
15:01 < CIA-5> markey * r680113 /branches/stable/extragear/multimedia/amarok/src/osd.cpp:
15:01 < CIA-5> Revert part of the patch for BUG 147059 that apparently caused the moodbar to malfunction in the OSD. Rich, does that fix
               it?

Comment 3 richlv 2007-06-26 11:36:22 UTC
yep, now moodbar isn't shown constantly.
it still might creep out now and then, but i will probably have to come up with a way to reproduce that one reliably anyway.
Comment 4 Mark Kretschmann 2007-06-26 11:40:34 UTC
Thanks, should we consider this report fixed, or leave it open?
Comment 5 richlv 2007-06-26 12:00:24 UTC
i'd say, fixed.
if i or anybody else comes up with a way to reproduce the sporadic behaviour, it can be reopened.

i've seen this behaviour since moodbar was introduced, but have never gotten more information regarding reproducability.
Comment 6 Mark Kretschmann 2007-06-26 12:13:42 UTC
*** Bug has been marked as fixed ***.
Comment 7 Mark Kretschmann 2007-06-27 17:05:45 UTC
SVN commit 680951 by markey:

Patch by Tuomas that keeps the moodbar shown when cycling through ratings, _without
breaking anything(?)_ this time. 

CCBUG: 146918
CCBUG: 147199



 M  +1 -0      app.cpp  
 M  +1 -0      osd.cpp  


--- branches/stable/extragear/multimedia/amarok/src/app.cpp #680950:680951
@@ -1254,6 +1254,7 @@
         CollectionDB::instance()->setSongRating( path, n, true );
         const int rating = CollectionDB::instance()->getSongRating( path );
         EngineController::instance()->updateBundleRating( rating );
+        Amarok::OSD::instance()->OSDWidget::setMoodbar( EngineController::instance()->bundle() );
         Amarok::OSD::instance()->OSDWidget::ratingChanged( rating );
         if( !Amarok::OSD::instance()->isShown() && !PlaylistWindow::self()->isReallyShown() )
             Amarok::OSD::instance()->forceToggleOSD();
--- branches/stable/extragear/multimedia/amarok/src/osd.cpp #680950:680951
@@ -94,6 +94,7 @@
     //m_text = '\n' + i18n( "Rating changed" );
     setRating( rating ); //Checks isEnabled() before doing anything
 
+    OSDWidget::setMoodbar( EngineController::instance()->bundle() );
     if( isShown() )
         show();
 }