[virt-tools-list] [PATCH 2/2] Add support to view the password entry content

Christophe Fergeau cfergeau at redhat.com
Mon Sep 15 11:45:47 UTC 2014


On Mon, Sep 15, 2014 at 01:27:57PM +0200, Fabiano Fidêncio wrote:
> On Mon, 2014-09-15 at 13:25 +0200, Christophe Fergeau wrote:
> > Hey,
> > 
> > Why not if we want to let the user unobscure the password. The icon is a
> > bit weird though, I would not know what it's about without clicking on
> > it.
> > Code looks good.
> 
> I loved the icon :-) That's the *only* reason I ended up in file-roller
> code, looking for the icon name.
> Do you have any suggestion about another icon that could be used?

Nope, obviously not, bike shedding is easy, making constructive
suggestions is harder ;)

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20140915/f8ca5d52/attachment.sig>


More information about the virt-tools-list mailing list