Review Request 110082: fixes Bug 275821 - JJ: Proper tooltips for Saved Playlists; remove Playlist::description() method

[Matěj Laitl]

> On April 21, 2013, 12:05 a.m., Matěj Laitl wrote:
> > src/core/playlists/Playlist.h, line 143
> > <http://git.reviewboard.kde.org/r/110082/diff/1/?file=139744#file139744line143>
> >
> >     I think you've also forgot to remove some description() implementations under podcasts.
> >     
> >     Or perhaps for podcasts the description is actually useful & used?
> 
> Vedant Agarwala wrote:
>     description() is part of PodcastMetaCommon class too (and is virtual). PodcastChannel, and thereby its child classes, inherit from both PodcastMetaCommon and Playlist classes. Hence I left it.

Okay, don't touch podcasts for now.


- Matěj


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


On April 21, 2013, 12:05 a.m., Vedant Agarwala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110082/
> -----------------------------------------------------------
> 
> (Updated April 21, 2013, 12:05 a.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Description
> -------
> 
> As agreed on the review for https://git.reviewboard.kde.org/r/104048/ , Qt::TooltipRole has been updated so that now the tooltip displays full name of the playlist. Occurrences of "description" have been removed (from the Playlist base class as well as the subclasses).
> 
> 
> This addresses bug 275821.
>     https://bugs.kde.org/show_bug.cgi?id=275821
> 
> 
> Diffs
> -----
> 
>   src/playlistmanager/sql/SqlPlaylist.cpp 2d6ef61 
>   src/playlistmanager/sql/SqlPlaylist.h d28d161 
>   src/playlistmanager/SyncedPlaylist.cpp 56be7e8 
>   src/playlistmanager/SyncedPlaylist.h 214bb5c 
>   src/core/playlists/Playlist.h cf9a4c0 
>   src/core-impl/playlists/types/file/PlaylistFile.cpp 073e140 
>   src/core-impl/playlists/types/file/PlaylistFile.h 0358196 
>   src/core-impl/collections/mediadevicecollection/playlist/MediaDevicePlaylist.cpp 3510481 
>   src/core-impl/collections/mediadevicecollection/playlist/MediaDevicePlaylist.h 9b94872 
>   src/browsers/playlistbrowser/PlaylistBrowserModel.cpp 6ee3db3 
> 
> Diff: http://git.reviewboard.kde.org/r/110082/diff/
> 
> 
> Testing
> -------
> 
> Testing done. Works. Builds successfully and passes the tests.
> 
> 
> File Attachments
> ----------------
> 
> displays the new tooltip
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/04/19/amarok_screenshot1.png
> displays the new tooltip
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/04/19/amarok_screenshot.png
> 
> 
> Thanks,
> 
> Vedant Agarwala
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130421/347574a9/attachment.html>