Change in ovirt-engine[master]: webadmin: Irregular behavior of checkbox in QuotaPopupView -...

[gchaplik at redhat.com]

Gilad Chaplik has submitted this change and it was merged.

Change subject: webadmin: Irregular behavior of checkbox in QuotaPopupView -> QuotaStorageTable (#846267)
......................................................................


webadmin: Irregular behavior of checkbox in QuotaPopupView -> QuotaStorageTable (#846267)

https://bugzilla.redhat.com/846267

Problem:
	In QuotaPopupView -> QuotaStorageTable the checkbox was not behaving as intended
	It would require two "clicks" to select or deselect it, resaulting  in bad states

Soulution:
	Changed the constructor call to CheckboxCell from (false, true) to (true, true)

	this flag is called "dependsOnSelection".
	internally the gwt code use it to indicate that the state of the checkbox should not be saved

	from the gwt code:
	/*
      	 * Save the new value. However, if the cell depends on the selection, then
         * do not save the value because we can get into an inconsistent state.
         */
      	if (value != isChecked && !dependsOnSelection()) {
        	setViewData(context.getKey(), isChecked);
      	} else {
        	clearViewData(context.getKey());
	}

Change-Id: I5094d701be9f2e634eec787f780ad913160dd96e
Signed-off-by: Noam Slomianko <nslomian at redhat.com>
---
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/quota/QuotaPopupView.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Gilad Chaplik: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/7168
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5094d701be9f2e634eec787f780ad913160dd96e
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Noam Slomianko <nslomian at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Noam Slomianko <nslomian at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server