Socket Sentry applet

Rob Hasselbaum rob at hasselbaum.net
Tue Mar 30 04:40:01 CEST 2010


Hi Aaron. Thanks for the feedback. My replies below.

On Mon, Mar 29, 2010 at 9:10 PM, Aaron J. Seigo <aseigo at kde.org> wrote:
> * is there any particular reason why the global settings are global instead of
> local?

Efficiency, mainly. All widget instances that watch the same network
interface are getting shared updates from a single data engine source.
So actually, the "global" settings could be broken down by source. But
just to keep things simple, I made them apply across sources for now.
I might revisit that decision if there's demand for more granularity.

> * "show subdomain levels" could be done with a single spinbox and a special
> value (which is what shows when the value of a spinbox is the minimum) of
> "All"

OK, thanks. I will remember that if I need to claim more real estate.
The radio buttons do take up a lot of space.

> * "Show one per row" -> is that essentially grouping?

("Show one row per...")Yes, exactly.

> * any particular reason the column sorting couldn't happen by just right
> clicking on the table headers as usual?

Yup, it does. Sorting and column resize happen as you'd expect if you
click on the column headers, and the widget's context menu lets you
pick the columns you want to see. The column selector in the config
dialog is just another way to get at that functionality and--more
importantly--increases the visibility of the column choices in case
the user doesn't think to try the context menu.

> * Search boxes are usually[1] in the upper right corner, and with
> setClickMessage you can put the string "Search" right inside the searchbox
> itself. (a little less clutter :)

I hadn't run across setClickMessage. I like that. Thanks, I'll incorporate that.

> i suppose you've published this on kde-look.org?

Yup.

> * instead of "Freeze sort" perhaps just "Sort" and reverse the logic of the
> checkbox

I'm not quite sold on this one. What I'm hoping is that "freeze sort"
gives the user the impression that this is a temporary override of the
default behavior, which is to sort. So the sort indicator on the
header doesn't disappear when you check "freeze sort", and when you
uncheck it again, the same sort order you had before will apply again.
Clicking a column header also turns off the freeze.


More information about the Plasma-devel mailing list