Bug 179396 - Clicking on the links on the lyrics applet does not work
Summary: Clicking on the links on the lyrics applet does not work
Status: RESOLVED FIXED
Alias: None
Product: amarok
Classification: Applications
Component: Context View (show other bugs)
Version: 2.0
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Amarok Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-01-02 18:03 UTC by Roman S.
Modified: 2009-12-09 11:29 UTC (History)
4 users (show)

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 Roman S. 2009-01-02 18:03:49 UTC
Version:           2.0 (using KDE 4.1.3)
OS:                Linux
Installed from:    SuSE RPMs

The lyrics applets shows some links (author, title, link to edit the lyrics on the lyrcwiki.org), but they are not clickable - they behave just like ordinary, noneditable text. I am not sure if this should be marked as bug or a wish, but if some text is underlined and has the same color as URL links, it should be clickable and bring the user to appropriate location, possibly using external web browser...
Comment 1 Lucio Menzel 2009-02-02 16:53:51 UTC
Within the Context tab, when I click on Author sub-tab it shows a page coming from Wikipedia asking to select the language instead of showing the Author info from Wikipedia.

The lirycs does not work by default.  I noticed that the lyrc script needs to be set to run, then it works.  But it does not remember to run the script on the next time amarok starts.

When starting Amarok restores the window size and position and the playlist to some state that is not the same as it was on the previous session.  It seems that Amarok is not saving these params when the system shutsdown.
Comment 2 Valentin Rouet 2009-03-25 12:12:27 UTC
SVN commit 944255 by vrouet:

Turned the lyrics QTextEdit into a QTextBrowser, and set the links to be opened in the default external web browser.
CCBUG: 179396

 M  +2 -0      ChangeLog  
 M  +4 -2      src/context/applets/lyrics/LyricsApplet.cpp  
 M  +2 -2      src/context/applets/lyrics/LyricsApplet.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=944255
Comment 3 Valentin Rouet 2009-03-25 14:42:50 UTC
My changes seems to be ok, marking as resolved