Review Request: Adding history to pastebin plasmoid.

Danilo Cesar Lemes de Paula danilo.eu at gmail.com
Tue Mar 31 13:43:52 CEST 2009


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



trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.cpp
<http://reviewboard.kde.org/r/483/#comment450>

    Yes.
    
    topSeparator was used to separate "Paste" action from "Pastebin Settings". As I'm adding new items under that action, it would be nice to have another separator on the bottom at that list.


- Danilo Cesar


On 2009-03-31 04:38:08, Danilo Cesar Lemes de Paula wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/483/
> -----------------------------------------------------------
> 
> (Updated 2009-03-31 04:38:08)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Adding history to pastebin plasmoid.
> 
> Plasmoid should show the last 3 links shown by the plasmoid (when you click with the right button), and it should be copied to clipboard when user click on it.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.h 945384 
>   trunk/KDE/kdeplasma-addons/applets/pastebin/pastebin.cpp 945384 
> 
> Diff: http://reviewboard.kde.org/r/483/diff
> 
> 
> Testing
> -------
> 
>  
> 
> 
> Thanks,
> 
> Danilo Cesar
> 
>



More information about the Plasma-devel mailing list