Lior Vernia has posted comments on this change.
Change subject: webadmin: fix QoS dialog indentation
......................................................................
Patch Set 4:
(1 comment)
....................................................
File
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/networkQoS/NetworkQoSPopupView.java
Line 112: setStyle();
Line 113: }
Line 114:
Line 115: private void setStyle() {
Line 116:
inboundAverageEditor.setContentWidgetStyleName(style.valueWidth());
That's disappointing, if the only widget that's displayed in
EntityModelTextBoxOnlyEditor is the text box, I would have expected to be able
to set its style easily. If you feel extra-responsible today you might override
its setStyle() method to set the style of its content widget :)
Line 117:
inboundAverageEditor.setLabelStyleName(style.hideLabelStyle());
Line 118:
inboundPeakEditor.setContentWidgetStyleName(style.valueWidth());
Line 119: inboundPeakEditor.setLabelStyleName(style.hideLabelStyle());
Line 120:
inboundBurstEditor.setContentWidgetStyleName(style.valueWidth());
--
To view, visit http://gerrit.ovirt.org/19356
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifbcfa6849284d2e087c6884aa79cd77d9f4c3096
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Lior Vernia <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
