Alona Kaplan has uploaded a new change for review. Change subject: webadmin: redundant comma at the end of disabled command button tootip. ......................................................................
webadmin: redundant comma at the end of disabled command button tootip. The disabled command button tootip is a list of comma separeated reason. There should be a "," after the last reason. Change-Id: I5c8f5a87bb75a4c73f7dffc24c259794321adde5 Signed-off-by: Alona Kaplan <[email protected]> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/AbstractUiCommandButton.java 1 file changed, 4 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/03/9003/1 diff --git a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/AbstractUiCommandButton.java b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/AbstractUiCommandButton.java index 1b48f2e..9863e25 100644 --- a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/AbstractUiCommandButton.java +++ b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/AbstractUiCommandButton.java @@ -61,6 +61,10 @@ for (String reason : command.getExecuteProhibitionReasons()) { title += reason + ","; //$NON-NLS-1$ } + if (!title.equals("")) //$NON-NLS-1$ + { + title = title.substring(0, title.length() -1); + } } else { title = command.getTitle(); } -- To view, visit http://gerrit.ovirt.org/9003 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I5c8f5a87bb75a4c73f7dffc24c259794321adde5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
