diff options
author | Kasper Nilsson <kaspern@google.com> | 2018-10-18 14:20:08 -0700 |
---|---|---|
committer | Paladox none <thomasmulhall410@yahoo.com> | 2018-10-18 22:47:04 +0000 |
commit | 67f158c73422c3298aee77fc9b28728551b6467f (patch) | |
tree | 92f78c78a31a9eb5a4ec7a11f0dc795ebce73cef | |
parent | 1eead16a66840b02688b5501dcfc2b3c5c9332d3 (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.html | 3 | ||||
-rw-r--r-- | polygerrit-ui/app/elements/change-list/gr-dashboard-view/gr-dashboard-view_test.html | 14 |
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(''), ''); |