[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kwrite-devel
Subject:    Re: Review Request: Allow hiding of items in the Kate Sessions
From:       "Aaron Seigo" <aseigo () kde ! org>
Date:       2010-06-09 15:58:12
Message-ID: 20100609155812.8447.63752 () localhost
[Download RAW message or body]


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

_______________________________________________
KWrite-Devel mailing list
KWrite-Devel@kde.org
https://mail.kde.org/mailman/listinfo/kwrite-devel


[prev in list] [next in list] [prev in thread] [next in thread]