From a833eca50c77033e51a04315766bbc5530cf548d Mon Sep 17 00:00:00 2001 From: Johan Dahlin Date: Wed, 10 Jul 2013 17:33:26 -0300 Subject: [PATCH] Include author attributes in CommentPanel. Include author's email and name as separate attributes in a CommentPanel. That makes it possible to filter out CI based comment using user scripts. Change-Id: I9c81c687d705f3c4077fe5f77ced6a35a8ff4161 --- .../main/java/com/google/gerrit/client/ui/CommentPanel.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/ui/CommentPanel.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/ui/CommentPanel.java index 05c6b5a09d..01e71fe6a9 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/ui/CommentPanel.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/ui/CommentPanel.java @@ -50,6 +50,7 @@ public class CommentPanel extends Composite implements HasDoubleClickHandlers, HasFocusHandlers, FocusHandler, HasBlurHandlers, BlurHandler { private static final int SUMMARY_LENGTH = 75; private final HandlerManager handlerManager = new HandlerManager(this); + private final FlowPanel body; private final FlexTable header; private final InlineLabel messageSummary; private final FlowPanel content; @@ -73,7 +74,7 @@ public class CommentPanel extends Composite implements HasDoubleClickHandlers, protected CommentPanel(CommentLinkProcessor commentLinkProcessor) { this.commentLinkProcessor = commentLinkProcessor; - final FlowPanel body = new FlowPanel(); + body = new FlowPanel(); initWidget(body); setStyleName(Gerrit.RESOURCES.css().commentPanel()); @@ -129,6 +130,8 @@ public class CommentPanel extends Composite implements HasDoubleClickHandlers, public void setAuthorNameText(final AccountInfo author, final String nameText) { header.setWidget(0, 0, new AvatarImage(author, 26)); header.setText(0, 1, nameText); + body.getElement().setAttribute("email", author.email()); + body.getElement().setAttribute("name", author.name()); } protected void setDateText(final String dateText) {