opendev/gerrit
Code Issues Proposed changes

Merge "Add slash between options in message-list"

Browse Source
This commit is contained in:
Wyatt Allen
2016-11-19 00:54:56 +00:00
committed by Gerrit Code Review
parent 67d66ec598 2b86c8803e
commit 32c2ef35c4
2 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
polygerrit-ui/app/elements/change/gr-messages-list/gr-messages-list.html
View File

@@ -48,11 +48,15 @@ limitations under the License.
on-tap="_handleExpandCollapseTap">
[[_computeExpandCollapseMessage(_expanded)]]
</gr-button>
<gr-button id="automatedMessageToggle" link
on-tap="_handleAutomatedMessageToggleTap"
<div
id="automatedMessageToggleContainer"
hidden$="[[!_hasAutomatedMessages(messages)]]">
[[_computeAutomatedToggleText(_hideAutomated)]]
</gr-button>
/
<gr-button id="automatedMessageToggle" link
on-tap="_handleAutomatedMessageToggleTap">
[[_computeAutomatedToggleText(_hideAutomated)]]
</gr-button>
</div>
</div>
</div>
<template

2
polygerrit-ui/app/elements/change/gr-messages-list/gr-messages-list_test.html
View File

@@ -88,7 +88,7 @@ limitations under the License.
test('hide messages does not appear when no automated messages',
function() {
assert.isOk(element.$$('#automatedMessageToggle[hidden]'));
assert.isOk(element.$$('#automatedMessageToggleContainer[hidden]'));
});
test('scroll to message', function() {