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

[Aaron Seigo]

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


this is too late for 4.5, but can certainly go into 4.6. it also gives us time to polish this feature up :)

instead of two lists, why not just have one list with checkboxes in it? (Qt's item views support an isCheckable property in the models, making it pretty easy.) also, instead of "Display" for the heading, how about "Sessions"? once this is polished for this widget, it should also be shared with / implemented in the konsole and konqueror session plasmoids as well for consistency.

- Aaron


On 2010-06-09 14:03:49, Yuen Hoe Lim wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4264/
> -----------------------------------------------------------
> 
> (Updated 2010-06-09 14:03:49)
> 
> 
> Review request for Kate and Plasma.
> 
> 
> 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
> -----
> 
>   /trunk/KDE/kdesdk/kate/plasma/session/katesessionConfig.ui PRE-CREATION 
>   /trunk/KDE/kdesdk/kate/plasma/session/katesessionapplet.h 1125007 
>   /trunk/KDE/kdesdk/kate/plasma/session/katesessionapplet.cpp 1125007 
> 
> Diff: http://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://reviewboard.kde.org/r/4264/s/428/
> 
> 
> Thanks,
> 
> Yuen Hoe
> 
>