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