Ramesh N has uploaded a new change for review. Change subject: webadmin: add dash - before items in confirmation pop up ......................................................................
webadmin: add dash - before items in confirmation pop up Chagning the the DefaultConfirmationPopupView to add dash (-) before every item in the pop for consistency purpose. Change-Id: Ib09a5ca23e81e2c11e64e08084691ebc03e2f602 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1024734 Signed-off-by: Ramesh Nachimuthu <[email protected]> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/DefaultConfirmationPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/uicommon/VolumeModule.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/RemoveBrickPopupView.java 3 files changed, 12 insertions(+), 4 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/30/20730/1 diff --git a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/DefaultConfirmationPopupView.java b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/DefaultConfirmationPopupView.java index 9260e42..212d351 100644 --- a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/DefaultConfirmationPopupView.java +++ b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/DefaultConfirmationPopupView.java @@ -57,12 +57,16 @@ ArrayList<String> items = (ArrayList<String>) object.getItems(); for (String item : items) { - descriptionPanel.add(new Label(item)); + descriptionPanel.add(new Label(getItemTextFormatted(item))); } } }); } + private String getItemTextFormatted(String itemText) { + return "- " + itemText; //$NON-NLS-1$ + } + @Override public ConfirmationModel flush() { return driver.flush(); diff --git a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/uicommon/VolumeModule.java b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/uicommon/VolumeModule.java index 9e60c9e..743ead5 100644 --- a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/uicommon/VolumeModule.java +++ b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/uicommon/VolumeModule.java @@ -92,7 +92,7 @@ @Provides @Singleton public SearchableDetailModelProvider<GlusterBrickEntity, VolumeListModel, VolumeBrickListModel> getVolumeBrickListProvider(EventBus eventBus, - Provider<DefaultConfirmationPopupPresenterWidget> defaultConfirmPopupProvider, + final Provider<DefaultConfirmationPopupPresenterWidget> defaultConfirmPopupProvider, final Provider<AddBrickPopupPresenterWidget> addBrickPopupProvider, final Provider<RemoveBrickPopupPresenterWidget> removeConfirmPopupProvider, final Provider<RemoveBrickPopupPresenterWidget> removeBrickPopupProvider, @@ -130,7 +130,7 @@ if (lastExecutedCommand == getModel().getRemoveBricksCommand()) { return removeConfirmPopupProvider.get(); } else { - return super.getConfirmModelPopup(source, lastExecutedCommand); + return defaultConfirmPopupProvider.get(); } } diff --git a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/RemoveBrickPopupView.java b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/RemoveBrickPopupView.java index 9eb189f..ba04508 100644 --- a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/RemoveBrickPopupView.java +++ b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/RemoveBrickPopupView.java @@ -108,7 +108,7 @@ ArrayList<String> items = (ArrayList<String>) object.getItems(); for (String item : items) { - itemsPanel.add(new Label(item)); + itemsPanel.add(new Label(getItemTextFormatted(item))); } } }); @@ -132,6 +132,10 @@ }); } + private String getItemTextFormatted(String itemText) { + return "- " + itemText; //$NON-NLS-1$ + } + @Override public void setMessage(String message) { super.setMessage(message); -- To view, visit http://gerrit.ovirt.org/20730 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ib09a5ca23e81e2c11e64e08084691ebc03e2f602 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
