Change in ovirt-engine[master]: webadmin: Irregular behavior of checkbox in QuotaPopupView -...
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(a)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(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server