Hi Shubendu, I saw your invite for your walkthrough on the feature - Gluster volume snapshots and also followed the feature link on the wiki and reviewed it. From the UX/ UI perspective, it looks good and I had only a couple of comments.
The remove snapshot section seems to indicate an ability to delete multiple snapshots at once and I am not sure we support any batch actions as of now elsewhere in the tool. Ctrl+click is not supported on lists today I think. So, I wanted to find out if you were going to introduce this for your feature. If yes, then we should ensure all other grids are enabled with this multi-select capability. If not, we should refrain from introducing something just for this feature for the sake of consistency and across feature product cohesiveness. The other feedback I have is on the snapshot configuration dialog. It will be great if any units can be added to the right of the field because one is a number and the other is a percentage but the field labels are so similar sounding. The field labels with the dashes in between the words also make it look very homegrown and seems like we are exposing some internal labels to our users. Last but not least, since the terminology of 'snapshots' overlaps with VM snapshots, we should just ensure that no similar action does wildly different things although it seems like snapshotting here is much more simple. Hope this is helpful, Thanks Malini _______________________________________________ Users mailing list Users@ovirt.org http://lists.ovirt.org/mailman/listinfo/users
