summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKasper Nilsson <kaspern@google.com>2018-10-18 14:20:08 -0700
committerPaladox none <thomasmulhall410@yahoo.com>2018-10-18 22:47:04 +0000
commit67f158c73422c3298aee77fc9b28728551b6467f (patch)
tree92f78c78a31a9eb5a4ec7a11f0dc795ebce73cef
parent1eead16a66840b02688b5501dcfc2b3c5c9332d3 (diff)
Handle dashboard edge case
If a user has no outgoing changes, but is not eligible to view the create change help panel, the "Outgoing reviews" section appears blank. This change adds back in the "No changes" placeholder text. Change-Id: I55915ef9999e5356dc13824352d087419f7ce5f4 (cherry picked from commit 9a5d102115f60e5890f8c53fd6b16e3cb00d7c43)
-rw-r--r--polygerrit-ui/app/elements/change-list/gr-dashboard-view/gr-dashboard-view.html3
-rw-r--r--polygerrit-ui/app/elements/change-list/gr-dashboard-view/gr-dashboard-view_test.html14
2 files changed, 17 insertions, 0 deletions
diff --git a/polygerrit-ui/app/elements/change-list/gr-dashboard-view/gr-dashboard-view.html b/polygerrit-ui/app/elements/change-list/gr-dashboard-view/gr-dashboard-view.html
index 2bc12c6c1c..99aa265008 100644
--- a/polygerrit-ui/app/elements/change-list/gr-dashboard-view/gr-dashboard-view.html
+++ b/polygerrit-ui/app/elements/change-list/gr-dashboard-view/gr-dashboard-view.html
@@ -101,6 +101,9 @@ limitations under the License.
<template is="dom-if" if="[[_showNewUserHelp]]">
<gr-create-change-help on-create-tap="createChangeTap"></gr-create-change-help>
</template>
+ <template is="dom-if" if="[[!_showNewUserHelp]]">
+ No changes
+ </template>
</div>
</gr-change-list>
</div>
diff --git a/polygerrit-ui/app/elements/change-list/gr-dashboard-view/gr-dashboard-view_test.html b/polygerrit-ui/app/elements/change-list/gr-dashboard-view/gr-dashboard-view_test.html
index 17484b0188..de74218078 100644
--- a/polygerrit-ui/app/elements/change-list/gr-dashboard-view/gr-dashboard-view_test.html
+++ b/polygerrit-ui/app/elements/change-list/gr-dashboard-view/gr-dashboard-view_test.html
@@ -299,6 +299,20 @@ limitations under the License.
});
});
+ test('_showNewUserHelp', () => {
+ element._loading = false;
+ element._showNewUserHelp = false;
+ flushAsynchronousOperations();
+
+ assert.equal(element.$.emptyOutgoing.textContent.trim(), 'No changes');
+ assert.isNotOk(element.$$('gr-create-change-help'));
+ element._showNewUserHelp = true;
+ flushAsynchronousOperations();
+
+ assert.notEqual(element.$.emptyOutgoing.textContent.trim(), 'No changes');
+ assert.isOk(element.$$('gr-create-change-help'));
+ });
+
test('_computeUserHeaderClass', () => {
assert.equal(element._computeUserHeaderClass(undefined), '');
assert.equal(element._computeUserHeaderClass(''), '');