[netsniff-ng] Re: flowtop: Flows visual separating

2016-02-26 Thread Vadim Kochan
On Fri, Feb 26, 2016 at 10:58 AM, Tobias Klauser  wrote:
> On 2016-02-25 at 17:35:56 +0100, Vadim Kochan  wrote:
>> On Sat, Feb 20, 2016 at 8:47 PM, Vadim Kochan  wrote:
>> > On Sat, Feb 20, 2016 at 7:49 PM, Daniel Borkmann  
>> > wrote:
>> >> Hi Vadim,
>> >>
>> >> thanks for looking into this, appreciate it!
>> >>
>> >> On 02/20/2016 03:28 PM, Vadim Kochan wrote:
>> >>>
>> >>> On Sat, Feb 20, 2016 at 1:25 AM, Vadim Kochan  wrote:
>> 
>>  Hi,
>> 
>>  I tried to come up with visual separating of printed flows as currently
>>  its not easy to identify separate flow entry, so I did some changes and
>>  I am not sure if it looks good so I atached the screenshot.
>> 
>>  Regards,
>> >>>
>> >>>
>> >>> I attached another version of odd & even flows entries style, here I
>> >>> used cyan & white colors and it seem looks
>> >>> better as here is no such contrast like in case with black & white
>> >>> background colors (like in previous example),
>> >>> also here 'country' color changed to magenta as it looks better on
>> >>> white & cyan background colors.
>> >>
>> >>
>> >> Not particularly a fan of these background colors, but I understand
>> >> you'd like to improve usability on this. How about making flowtop
>> >> look and navigation more like top or htop? Perhaps some of this info
>> >> can be collapsed?
>
> Fully agree with Daniel, I'm not a big fan of too much background color
> (or even color in general) in TUI interfaces either. I'd certainly
> prefer if you'd go for a top/htop like interface in that case.
>
>> >>
>> >> Thanks,
>> >> Daniel
>> >
>> > Well, if to follow these *top-like tools then we need to print less
>> > info. Curently we print:
>> >
>> > 1) process name
>> > 2) flow state
>> > 3) application proto name
>> > 4) duration time
>> > 5) src/dst hostname
>> > 6) geo info
>> > 7) pkts/bytes stats (counters & rate)
>
> top/htop allow you to select the columns to display. We could define a
> sensible set of default columns (or even add additional ones in case we
> detect a wide enough window) and then let the user add/remove other
> columns.
>
>> >
>> > We can have 2 modes for flows visualization:
>> >
>> > 1) Short mode (1 row per entry) (default):
>> >  a) process name
>> >  b) flow state (but with shortest names)
>> >  c) application proto name
>> >  d) src/dst info hostnames (or only dst with country if it feets)
>> >  e) mixed stats
>> >
>> > 2) Extended mode, like in current implementation
>> > (but maybe be changed to color scheme which I sent in previous 
>> > example).
>
> I don't think it's necessary to have 2 modes if we go for selectable
> columns.
>
>> > Also there might be hot-keys to
>> > 1) expand 1-row entry into 3-row mode
>> > 2) switch between 1- & 3- row mode for all entries.
>
> In case the user's window isn't wide enough to hold all columns, this
> would be a nice option to display additional information.
>
> Cheers
> Tobias

Thanks,

I will consider your comments, I will update you with screen shots (if
you'd like) before sending patches if I will
come up with something useful for the next release.

Regards,

-- 
You received this message because you are subscribed to the Google Groups 
"netsniff-ng" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to netsniff-ng+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[netsniff-ng] Re: flowtop: Flows visual separating

2016-02-26 Thread Tobias Klauser
On 2016-02-25 at 17:35:56 +0100, Vadim Kochan  wrote:
> On Sat, Feb 20, 2016 at 8:47 PM, Vadim Kochan  wrote:
> > On Sat, Feb 20, 2016 at 7:49 PM, Daniel Borkmann  
> > wrote:
> >> Hi Vadim,
> >>
> >> thanks for looking into this, appreciate it!
> >>
> >> On 02/20/2016 03:28 PM, Vadim Kochan wrote:
> >>>
> >>> On Sat, Feb 20, 2016 at 1:25 AM, Vadim Kochan  wrote:
> 
>  Hi,
> 
>  I tried to come up with visual separating of printed flows as currently
>  its not easy to identify separate flow entry, so I did some changes and
>  I am not sure if it looks good so I atached the screenshot.
> 
>  Regards,
> >>>
> >>>
> >>> I attached another version of odd & even flows entries style, here I
> >>> used cyan & white colors and it seem looks
> >>> better as here is no such contrast like in case with black & white
> >>> background colors (like in previous example),
> >>> also here 'country' color changed to magenta as it looks better on
> >>> white & cyan background colors.
> >>
> >>
> >> Not particularly a fan of these background colors, but I understand
> >> you'd like to improve usability on this. How about making flowtop
> >> look and navigation more like top or htop? Perhaps some of this info
> >> can be collapsed?

Fully agree with Daniel, I'm not a big fan of too much background color
(or even color in general) in TUI interfaces either. I'd certainly
prefer if you'd go for a top/htop like interface in that case.

> >>
> >> Thanks,
> >> Daniel
> >
> > Well, if to follow these *top-like tools then we need to print less
> > info. Curently we print:
> >
> > 1) process name
> > 2) flow state
> > 3) application proto name
> > 4) duration time
> > 5) src/dst hostname
> > 6) geo info
> > 7) pkts/bytes stats (counters & rate)

top/htop allow you to select the columns to display. We could define a
sensible set of default columns (or even add additional ones in case we
detect a wide enough window) and then let the user add/remove other
columns.

> >
> > We can have 2 modes for flows visualization:
> >
> > 1) Short mode (1 row per entry) (default):
> >  a) process name
> >  b) flow state (but with shortest names)
> >  c) application proto name
> >  d) src/dst info hostnames (or only dst with country if it feets)
> >  e) mixed stats
> >
> > 2) Extended mode, like in current implementation
> > (but maybe be changed to color scheme which I sent in previous 
> > example).

I don't think it's necessary to have 2 modes if we go for selectable
columns.

> > Also there might be hot-keys to
> > 1) expand 1-row entry into 3-row mode
> > 2) switch between 1- & 3- row mode for all entries.

In case the user's window isn't wide enough to hold all columns, this
would be a nice option to display additional information.

Cheers
Tobias

-- 
You received this message because you are subscribed to the Google Groups 
"netsniff-ng" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to netsniff-ng+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.