> 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?
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. - Vedant ----------------------------------------------------------- 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 > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
