diff options
author | Clement Ho <clemmakesapps@gmail.com> | 2017-08-03 13:32:58 +0000 |
---|---|---|
committer | Clement Ho <clemmakesapps@gmail.com> | 2017-08-03 13:32:58 +0000 |
commit | b0464fa4e19e187adc46ec054ccd68832faec08c (patch) | |
tree | 15f5d6a9b3df259e18482a4d249664879ffabce6 | |
parent | 2f05a6a44cc1f2605982b6a0f562fe0b2cb2c1e8 (diff) | |
parent | 9b9c86787a009c420536c65aecfa5823af88f665 (diff) | |
download | gitlab-ce-b0464fa4e19e187adc46ec054ccd68832faec08c.tar.gz |
Merge branch '35232-next-unresolved' into 'master'
fix Jump to next discussion
Closes #35232
See merge request !13076
-rw-r--r-- | app/assets/javascripts/diff_notes/components/jump_to_discussion.js | 10 | ||||
-rw-r--r-- | changelogs/unreleased/35232-next-unresolved.yml | 4 |
2 files changed, 9 insertions, 5 deletions
diff --git a/app/assets/javascripts/diff_notes/components/jump_to_discussion.js b/app/assets/javascripts/diff_notes/components/jump_to_discussion.js index 37ddca29e71..298f737a2bc 100644 --- a/app/assets/javascripts/diff_notes/components/jump_to_discussion.js +++ b/app/assets/javascripts/diff_notes/components/jump_to_discussion.js @@ -94,7 +94,7 @@ const JumpToDiscussion = Vue.extend({ hasDiscussionsToJumpTo = false; } } - } else if (activeTab !== 'notes') { + } else if (activeTab !== 'show') { // If we are on the commits or builds tabs, // there are no discussions to jump to. hasDiscussionsToJumpTo = false; @@ -103,12 +103,12 @@ const JumpToDiscussion = Vue.extend({ if (!hasDiscussionsToJumpTo) { // If there are no discussions to jump to on the current page, // switch to the notes tab and jump to the first disucssion there. - window.mrTabs.activateTab('notes'); - activeTab = 'notes'; + window.mrTabs.activateTab('show'); + activeTab = 'show'; jumpToFirstDiscussion = true; } - if (activeTab === 'notes') { + if (activeTab === 'show') { discussionsSelector = '.discussion[data-discussion-id]'; discussionIdsInScope = discussionIdsForElements($(discussionsSelector)); } @@ -156,7 +156,7 @@ const JumpToDiscussion = Vue.extend({ let $target = $(`${discussionsSelector}[data-discussion-id="${nextUnresolvedDiscussionId}"]`); - if (activeTab === 'notes') { + if (activeTab === 'show') { $target = $target.closest('.note-discussion'); // If the next discussion is closed, toggle it open. diff --git a/changelogs/unreleased/35232-next-unresolved.yml b/changelogs/unreleased/35232-next-unresolved.yml new file mode 100644 index 00000000000..45f3fb429a8 --- /dev/null +++ b/changelogs/unreleased/35232-next-unresolved.yml @@ -0,0 +1,4 @@ +--- +title: fix jump to next discussion button +merge_request: +author: |