diff --git a/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog_html.ts b/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog_html.ts index ad5b6a32d1..3cc32b244b 100644 --- a/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog_html.ts +++ b/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog_html.ts @@ -321,13 +321,13 @@ export const htmlTemplate = html`
Bring to attention of ... - (select users) + (click chips to select users)
Owner
{ this._config = config; }); } - get isAttentionSetEnabled() { - return !!this._config && !!this._config.change - && !!this._config.change.enable_attention_set - && !!this.highlightAttention && !!this.change && !!this.account; + _isAttentionSetEnabled(config, highlight, account, change) { + return !!config && !!config.change + && !!config.change.enable_attention_set + && !!highlight && !!change && !!account; } - get hasAttention() { - if (!this.isAttentionSetEnabled || !this.change.attention_set) return false; - return this.change.attention_set.hasOwnProperty(this.account._account_id); - } - - _computeShowAttentionIcon(config, highlightAttention, account, change) { - return this.isAttentionSetEnabled && this.hasAttention; + _hasAttention(config, highlight, account, change, force) { + if (force) return true; + return this._isAttentionSetEnabled(config, highlight, account, change) + && change.attention_set + && change.attention_set.hasOwnProperty(account._account_id); } _computeName(account, config) { diff --git a/polygerrit-ui/app/elements/shared/gr-account-label/gr-account-label_html.ts b/polygerrit-ui/app/elements/shared/gr-account-label/gr-account-label_html.ts index c26e304521..b338c4c512 100644 --- a/polygerrit-ui/app/elements/shared/gr-account-label/gr-account-label_html.ts +++ b/polygerrit-ui/app/elements/shared/gr-account-label/gr-account-label_html.ts @@ -78,7 +78,7 @@ export const htmlTemplate = html`