opendev/gerrit
Code Issues Proposed changes

Include author attributes in CommentPanel.

Browse Source 
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
This commit is contained in:
Johan Dahlin
2013-07-10 17:33:26 -03:00
parent 09636fc57b
commit a833eca50c
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gerrit-gwtui/src/main/java/com/google/gerrit/client/ui/CommentPanel.java
View File

@@ -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) {