Alona Kaplan has posted comments on this change.

Change subject: webadmin: adding ListModelRadioGroupEditor
......................................................................


Patch Set 4:

1. There is no limit. All the radio buttons will be displayed vertically.
I"m not sure if there will be a scroll bar if needed.
2. Each radio button resides in a separate panel. RadioGroup has method- 
getPanel(K key) which returns the panel of the radioButton.
So the user can control the layout of the panels.

It was enough for me. IMO, if anyone needs more (scroll bar, horizontal 
alignment) he can extract the class.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7a2754a1157b1be725871a57a4618404ad4b3223
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <[email protected]>
Gerrit-Reviewer: Alexander Wels <[email protected]>
Gerrit-Reviewer: Alona Kaplan <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Lior Vernia <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to