Bug 136589 - shadows of cover-pop-up in taskbar
Summary: shadows of cover-pop-up in taskbar
Status: RESOLVED FIXED
Alias: None
Product: amarok
Classification: Applications
Component: general (show other bugs)
Version: 1.4.4
Platform: Slackware Linux
: NOR wishlist
Target Milestone: ---
Assignee: Amarok Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-31 21:11 UTC by Maik Wachsmuth
Modified: 2006-11-05 11:27 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Maik Wachsmuth 2006-10-31 21:11:44 UTC
Version:           1.4.4 (using KDE KDE 3.5.4)
Installed from:    Slackware Packages

Hey there.

I just created a music-queue and noticed that the albumcover in the popup, appearing when I focus the queueicon with the cursor, is shadowed. That looks great!

Now my question: In my opinion the statuspopup I get when focusing the amaroKicon in the taskbar is kind of boring. What do you guys think of shadowing the albumcovers?!

Please do something with it - It does not fit with the power of amaroK ;)

That would be great ;)

Regards,
Maik =)
Comment 1 Seb Ruiz 2006-10-31 22:00:31 UTC
SVN commit 600789 by seb:

Show shadowed cover images for the tooltip
BUG: 136589


 M  +2 -2      tracktooltip.cpp  


--- trunk/extragear/multimedia/amarok/src/tracktooltip.cpp #600788:600789
@@ -224,10 +224,10 @@
         m_tooltip += "<center><b>Amarok</b></center><table cellpadding='2' cellspacing='2' align='center'><tr>";
 
         m_tooltip += "%1"; //the cover gets substituted in, in tooltip()
-        m_cover = CollectionDB::instance()->podcastImage( tags );
+        m_cover = CollectionDB::instance()->podcastImage( tags, true );
         if( m_cover.isEmpty() || m_cover.contains( "nocover" ) != -1 )
         {
-            m_cover = CollectionDB::instance()->albumImage( tags );
+            m_cover = CollectionDB::instance()->albumImage( tags, true );
             if ( m_cover == CollectionDB::instance()->notAvailCover() )
                 m_cover = QString::null;
         }
Comment 2 Maik Wachsmuth 2006-11-05 11:27:43 UTC
thank you =)