anmolbabu has posted comments on this change.

Change subject: webadmin: fix action availability for create brick
......................................................................


Patch Set 2: Code-Review+1

I understand that the repeated check 

1. first on a set(all items) is for intial stage when nothing  is selected.

2. second on a subset(selected items) is for checking when some selections are 
made.

And I understand that this is required because you might have an option to make 
and or alter the selections after the click of this action also. So, can you 
please add this reason for doing it as a comment because we usually don't have 
the option to alter the selections(at-least I couldn't think of one such 
instance now)

-- 
To view, visit https://gerrit.ovirt.org/39820
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iecaff8f5863bd3107ba109e6f1f9d695d8ee67aa
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ramesh N <[email protected]>
Gerrit-Reviewer: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Ramesh N <[email protected]>
Gerrit-Reviewer: anmolbabu <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to