Hi!

> I've tried it, and it looks great, congrats!
>
Thanks!
 
> A few UI remarks:
> 
> * I like it very much :)
> 
> * I'm not sure about the subtitle in the title bar, that reads
>   "Display Tor circuits and streams": maybe drop "Display"? Anyway,
>   I'll let sajolida nitpick on such things :)
> 
> * Displaying circuits without any separator but a space between nodes
>   is hard to read for me. Vidalia uses comma+space as a separator, and
>   it's easier for me to read it this way.
> 
Ok.

> * The "Name" header for the circuits list feels wrong. How about
>   "Path" instead?
> 
Ok.

> * I suspect the circuit status strings should be capitalized.
> 
You mean: "Capitalized"? If yes: ok.

> * When I disconnect from the network and then reconnect, Tor Monitor
>   tells me it's going to try to reconnect to Tor, but clicking on the
>   OK button doesn't trigger any visible action and the notification
>   stays in place. Buggy thread/parallelism handling?
> 
The info bar is just a notification. The only action triggered by "Ok"
is to hide the notification.

> * When I disconnect from the network and then reconnect, Tor Monitor
>   tells me it's going to try to reconnect to Tor, but it still lists
>   a few circuits from the previous Tor session. Maybe the list should
>   be emptied in such a case?
> 
You mean: emptied as soon as the connection to the Tor deamon breaks?
That makes sense.

> * Tor Monitor needs an icon to display in the task bar instead of the
>   default one :)
> 
That should be solved if installed with --prefix=/usr, but I still need
to work on the setup script.

> Great job.
> 
Thanks!
_______________________________________________
Tails-dev mailing list
[email protected]
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
[email protected].

Reply via email to