Bug 136355

Summary: add "Search in Google for this lyric" message when finding no lyrics
Product: [Applications] amarok Reporter: Sven Krohlas <sven>
Component: Context ViewAssignee: Amarok Bugs <amarok-bugs-null>
Status: RESOLVED FIXED    
Severity: wishlist    
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Sven Krohlas 2006-10-26 20:35:08 UTC 
Version:           1.4.4 SVN (using KDE KDE 3.5.5)
Installed from:    SuSE RPMs
Compiler:          gcc version 4.0.2 20050901 (prerelease) (SUSE Linux) 
OS:                Linux

Currently if the lyrics browser can't find lyrics or there is a timeout you simply get an error message. These messages could be enhanced with a link to a Google search for that lyric to make the search feature easier to discover.
Comment 1 Alexandre Oliveira 2006-10-27 22:14:22 UTC 
SVN commit 599604 by aoliveira:

When the script can't find lyrics, provide a link for a web search.
BUG: 136355


 M  +2 -0      contextbrowser.cpp  


--- trunk/extragear/multimedia/amarok/src/contextbrowser.cpp #599603:599604
@@ -3049,6 +3049,8 @@
                 lyrics += "</a><br/>\n";
             }
         }
+        lyrics += i18n( "<p>You can <a href=\"%1\">search for the lyrics</a> on the Web.</p>" )
+                  .arg( QString( m_lyricSearchUrl ).replace( QRegExp( "^http:" ), "externalurl:" ) );
     }
     else {
         lyrics = el.text();