Review Request: Add playlist export action to Playlist Dock save action.

Matěj Laitl matej at laitl.cz
Wed Aug 8 14:42:53 UTC 2012



> On Aug. 8, 2012, 11:36 a.m., Bart Cerneels wrote:
> > I wouldn't do this. Toolbars become less usable the more actions are in it.
> > What is wrong with using the menu with a function that is not supposed to be used to often?

Did you say this with knowing that this doesn't add another toolbar enty, just one more item to the menu of the Save toolbar button? Ralf, this is really worth a screenshot.


- Matěj


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


On Aug. 2, 2012, 12:14 p.m., Ralf Engels wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105824/
> -----------------------------------------------------------
> 
> (Updated Aug. 2, 2012, 12:14 p.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Description
> -------
> 
> Add playlist export action to Playlist Dock save action.
> 
> 
> Diffs
> -----
> 
>   src/playlist/PlaylistDock.cpp ae1644f 
> 
> Diff: http://git.reviewboard.kde.org/r/105824/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ralf Engels
> 
>

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


More information about the Amarok-devel mailing list