Tomas Jelinek has submitted this change and it was merged.

Change subject: webadmin: CPU Type and CPU Architecture fields in Cluster 
dialog fix
......................................................................


webadmin: CPU Type and CPU Architecture fields in Cluster dialog fix

* CPU Architecture listbox is placed right above the CPU Type because
  they are related. (ClusterPopupView.ui.xml)

* CPU Architecture field doesn't disappear when CPU Type is selected.
  (Bug fix). (ClusterPopupView.java)

* CPU Architecture filters content of CPU type listbox.
  (ClusterModel.java, FilteredListModel.java)

Bug-Url: https://bugzilla.redhat.com/1159777
Change-Id: I4c0452426f58f09703382495a9527fdbe859d80e
Signed-off-by: Jakub Niedermertl <[email protected]>
---
A 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/FilteredListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml
4 files changed, 123 insertions(+), 17 deletions(-)

Approvals:
  Tomas Jelinek: Looks good to me, approved
  Jakub Niedermertl: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4c0452426f58f09703382495a9527fdbe859d80e
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Jakub Niedermertl <[email protected]>
Gerrit-Reviewer: Jakub Niedermertl <[email protected]>
Gerrit-Reviewer: Tomas Jelinek <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to