<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
On 13-11-2014 08:30, Yu Xin Huo wrote:<br>
<blockquote cite="mid:54648839.2010203@linux.vnet.ibm.com"
type="cite">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
<br>
<img src="cid:part1.03040600.07000402@linux.vnet.ibm.com" alt=""><br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
</blockquote>
Good mockup! I have a few comments though:<br>
<ul>
<li>The revert button (like all other buttons, I believe) should
have a tooltip displayed when the user hovers over it. I don't
think that button states very clearly what it does, even though
I don't have a better suggestion for another icon right now.</li>
<li>When the user clicks on the button "+", a snapshot should be
created right away. I don't think we should let them choose a
name when creating a snapshot. Just to make things simpler for
them. Later the user should be able to rename the snapshots, so
that the text field and the floppy disk button won't exist for
now. Currently, the backend doesn't support snapshot renaming.</li>
<li>There could be another column displaying the creation time
(i.e. the field 'created' in the snapshot JSON structure).
That's a very important information when talking about
snapshots. In the mockup example, we can see the time the
snapshot was created because the time is in its name, but that
won't be the case forever, the user will be able to rename the
snapshot.</li>
<li>I think there should be a visual indication of the current
snapshot. I thought about adding a different icon (like the
"loading" one) to indicate that.</li>
</ul>
<blockquote cite="mid:54648839.2010203@linux.vnet.ibm.com"
type="cite">
<pre wrap="">_______________________________________________
Kimchi-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Kimchi-devel@ovirt.org">Kimchi-devel@ovirt.org</a>
<a class="moz-txt-link-freetext" href="http://lists.ovirt.org/mailman/listinfo/kimchi-devel">http://lists.ovirt.org/mailman/listinfo/kimchi-devel</a>
</pre>
</blockquote>
<br>
</body>
</html>