diff options
author | Fatih Acet <acetfatih@gmail.com> | 2018-12-14 11:49:53 +0000 |
---|---|---|
committer | Fatih Acet <acetfatih@gmail.com> | 2018-12-14 11:49:53 +0000 |
commit | 1d0eadc470a2c7324d002e923eb76ae932862354 (patch) | |
tree | c276ce662d842ed8f403ed2121e8c841c28ae8fd | |
parent | e8a7836cee837d71bb0155c01ebb79deaebbbfdb (diff) | |
parent | a2e9ad83f406f2a0ca9e7766a3f41de1f9aadd9a (diff) | |
download | gitlab-ce-1d0eadc470a2c7324d002e923eb76ae932862354.tar.gz |
Merge branch 'winh-resolved-discussions-reply-field' into 'master'
Display reply field if resolved discussion has no replies
Closes #54330
See merge request gitlab-org/gitlab-ce!23801
3 files changed, 16 insertions, 1 deletions
diff --git a/app/assets/javascripts/notes/components/noteable_discussion.vue b/app/assets/javascripts/notes/components/noteable_discussion.vue index 4156fe0d229..07c938a0021 100644 --- a/app/assets/javascripts/notes/components/noteable_discussion.vue +++ b/app/assets/javascripts/notes/components/noteable_discussion.vue @@ -413,7 +413,7 @@ Please check your network connection and try again.`; </template> </ul> <div - v-if="!isRepliesCollapsed" + v-if="!isRepliesCollapsed || !hasReplies" :class="{ 'is-replying': isReplying }" class="discussion-reply-holder" > diff --git a/changelogs/unreleased/winh-resolved-discussions-reply-field.yml b/changelogs/unreleased/winh-resolved-discussions-reply-field.yml new file mode 100644 index 00000000000..01cf35ae8a7 --- /dev/null +++ b/changelogs/unreleased/winh-resolved-discussions-reply-field.yml @@ -0,0 +1,5 @@ +--- +title: Display reply field if resolved discussion has no replies +merge_request: 23801 +author: +type: fixed diff --git a/spec/support/features/discussion_comments_shared_example.rb b/spec/support/features/discussion_comments_shared_example.rb index 922f3df144d..42a086d58d2 100644 --- a/spec/support/features/discussion_comments_shared_example.rb +++ b/spec/support/features/discussion_comments_shared_example.rb @@ -178,6 +178,16 @@ shared_examples 'discussion comments' do |resource_name| let(:note_id) { find("#{comments_selector} .note:first-child", match: :first)['data-note-id'] } let(:reply_id) { find("#{comments_selector} .note:last-child", match: :first)['data-note-id'] } + it 'can be replied to after resolving' do + click_button "Resolve discussion" + wait_for_requests + + refresh + wait_for_requests + + submit_reply('to reply or not reply') + end + it 'shows resolved discussion when toggled' do submit_reply('a') |