opendev/gerrit
Code Issues Proposed changes

Add slash between options in message-list

Browse Source 
Bug: Issue 4965
Change-Id: I13862a3ecf992a8a1c382124fb349a9641ea7f5e
This commit is contained in:
Kasper Nilsson
2016-11-18 16:27:09 -08:00
parent 214726d5ce
commit 2b86c8803e
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() {