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

(Updated 2010-09-05 15:48:29.104994)


Review request for Kate and Plasma.


Changes
-------

Implemented Aaron's suggestions. Sorry I left this one behind for so long - 
figured that there was still time before 4.6 so focused on finishing my GSoC 
first :)

New config UI screenshot included.


Summary
-------

Allows the user to specify which items to show or hide in the Kate Sessions 
Plasmoid using a configuration interface. Two possible use-cases for this:

1) Allow users who just want quick access to their projects to remove the 
default three items (Start Kate, New Session etc), which can take up a lot of 
space since the applet is usually quite small.

2) Allow multiple applets displaying different lists, for example having 
applets in different activities each displaying only sessions related to that 
activity.

The Kate Sessions Plasmoid currently does not have a configuration interface so 
I thought this is probably no harm :) I have attached a screenshot of the 
configuration UI. I'm somewhat inexperienced with QtDesigner so feedback on how 
I could improve the patch code or the config UI are most welcome :)


Diffs (updated)
-----

  /trunk/KDE/kdesdk/kate/plasma/session/katesessionConfig.ui PRE-CREATION 
  /trunk/KDE/kdesdk/kate/plasma/session/katesessionapplet.h 1171815 
  /trunk/KDE/kdesdk/kate/plasma/session/katesessionapplet.cpp 1171815 

Diff: http://svn.reviewboard.kde.org/r/4264/diff


Testing
-------

Tested on trunk on my computer. Works as far as I can tell.


Screenshots
-----------

Config UI Screenshot
  http://svn.reviewboard.kde.org/r/4264/s/428/
Updated config UI screenshot
  http://svn.reviewboard.kde.org/r/4264/s/497/


Thanks,

Yuen Hoe

_______________________________________________
Plasma-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to