modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/util/message/MessageCenterView.java | 37 ++++------ 1 file changed, 17 insertions(+), 20 deletions(-)
New commits: commit 7f51c8b8b62835bb0c7eaa68726de75bbbff9866 Author: John Mazzitelli mazz@redhat.com Date: Wed Mar 23 21:36:09 2011 -0400
BZ 689951 - message center window's list grid needed to be refreshed in order to have its rows be selectable
diff --git a/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/util/message/MessageCenterView.java b/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/util/message/MessageCenterView.java index f861a37..b5907fd 100644 --- a/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/util/message/MessageCenterView.java +++ b/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/util/message/MessageCenterView.java @@ -90,31 +90,28 @@ public class MessageCenterView extends Table implements MessageCenter.MessageLis */ public void showMessageCenterWindow() { try { - createWindow().show(); + if (window == null) { + window = new MessageCenterWindow("MessageCenterViewWindow"); + window.addItem(this); + window.addCloseClickHandler(new CloseClickHandler() { + @Override + public void onCloseClick(CloseClientEvent event) { + try { + window.hide(); + } catch (Throwable e) { + Log.warn("Cannot destroy message center", e); + } + } + }); + } + + window.show(); + markForRedraw(); // need this to ensure the list grid rows are selectable } catch (Throwable e) { Log.error("Cannot show message center window", e); } }
- private Window createWindow() { - if (window == null) { - window = new MessageCenterWindow("MessageCenterViewWindow"); - window.addItem(this); - window.addCloseClickHandler(new CloseClickHandler() { - @Override - public void onCloseClick(CloseClientEvent event) { - try { - window.hide(); - } catch (Throwable e) { - Log.warn("Cannot destroy message center", e); - } - } - }); - } - - return window; - } - @Override public void onMessage(final Message message) { try {
rhq-commits@lists.fedorahosted.org