Review Request 113420: port activitybar applet to Plasma2

Bhushan Shah bhush94 at gmail.com
Sun Oct 27 04:22:01 UTC 2013



> On Oct. 27, 2013, 2:12 a.m., David Edmundson wrote:
> > Looks good to me.
> > 
> > FWIW, when you move files it's best if you make diffs with "git diff --find-copies-harder" that way reviewboard doesn't show that you deleted and created a file. Otherwise I end up commenting on things you didn't change. (

Thanks for the tip! :)


- Bhushan


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


On Oct. 24, 2013, 8:39 p.m., Bhushan Shah wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113420/
> -----------------------------------------------------------
> 
> (Updated Oct. 24, 2013, 8:39 p.m.)
> 
> 
> Review request for Plasma and Sebastian Kügler.
> 
> 
> Repository: kde-workspace
> 
> 
> Description
> -------
> 
> port activitybar applet to plasma2
> 
> 
> Diffs
> -----
> 
>   plasma/generic/applets/CMakeLists.txt f03f941 
>   plasma/generic/applets/activitybar/CMakeLists.txt 297f1a9 
>   plasma/generic/applets/activitybar/contents/ui/main.qml PRE-CREATION 
>   plasma/generic/applets/activitybar/metadata.desktop PRE-CREATION 
>   plasma/generic/applets/activitybar/package/contents/ui/main.qml a6d5026 
>   plasma/generic/applets/activitybar/package/metadata.desktop b1f2ece 
> 
> Diff: http://git.reviewboard.kde.org/r/113420/diff/
> 
> 
> Testing
> -------
> 
> I don't have any stuffs that can help me to test if it switch activities in Plasma2 or not. It lists my kde4 activities..
> 
> 
> Thanks,
> 
> Bhushan Shah
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20131027/420a4b50/attachment-0001.html>


More information about the Plasma-devel mailing list