Review Request: Allow hiding of items in the Kate Sessions Plasmoid

[Yuen Hoe Lim]

-----------------------------------------------------------
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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100905/63ebcb6b/attachment.htm