Review Request 109470: A patch which allows us to distinguish between cached lyrics and lyrics newly downloaded from the internet.

[Matěj Laitl]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109470/#review30013
-----------------------------------------------------------


Is there a wish request or any other motivation to implement this feature? I don't see many reasons why user should care that the lyrics have been just downloaded or already cached. On the other hand, user may want to know whether lyrics have been previously edited by her. Additionally, some file formats allow storing lyrics in tags, it would be nice if Amarok supported it. (then it would make sense to show her what is the source of lyrics being shown)

- Matěj Laitl


On March 27, 2013, 7:22 p.m., mayank jha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109470/
> -----------------------------------------------------------
> 
> (Updated March 27, 2013, 7:22 p.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Description
> -------
> 
> It required modifications, when there is no change in the lyrics downloaded and lyrics retrieved from cache the title display of the lyrics browser changes to "Cached Lyrics" from "Lyrics" so we can tell the difference between old and new. 
> 
> 
> Diffs
> -----
> 
>   src/context/engines/lyrics/LyricsEngine.cpp 2befa91 
>   src/context/applets/lyrics/LyricsApplet.cpp 2394964 
>   src/context/engines/lyrics/LyricsEngine.h b187b73 
> 
> Diff: http://git.reviewboard.kde.org/r/109470/diff/
> 
> 
> Testing
> -------
> 
> Its working fine!
> 
> 
> Thanks,
> 
> mayank jha
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130328/4c34e91f/attachment.html>