summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFatih Acet <acetfatih@gmail.com>2017-01-25 19:54:41 +0000
committerFatih Acet <acetfatih@gmail.com>2017-01-25 19:54:41 +0000
commit7ce79f31670f3beaf5b29b06144a34e32b7cc4ec (patch)
treeceacd7cc3b3a66a67d45f9d4f67ad528db808929
parent94715834cef03ab5e7bda0bb558a650f39d41e94 (diff)
parent2c45a73c8e0b07aec0d688a75beb54ebfd08ac07 (diff)
downloadgitlab-ce-7ce79f31670f3beaf5b29b06144a34e32b7cc4ec.tar.gz
Merge branch 'label-select-toggle' into 'master' kamil-test-branch
Fixed label select toggle not updating correctly Closes #26119 See merge request !8601
-rw-r--r--app/assets/javascripts/labels_select.js6
-rw-r--r--changelogs/unreleased/label-select-toggle.yml4
-rw-r--r--spec/features/issues/form_spec.rb16
3 files changed, 25 insertions, 1 deletions
diff --git a/app/assets/javascripts/labels_select.js b/app/assets/javascripts/labels_select.js
index fd1e229e30a..70dc0d06b7b 100644
--- a/app/assets/javascripts/labels_select.js
+++ b/app/assets/javascripts/labels_select.js
@@ -336,7 +336,11 @@
.removeClass('is-active');
}
- if ($dropdown.hasClass('js-filter-bulk-update') || $dropdown.hasClass('js-issuable-form-dropdown')) {
+ if ($dropdown.hasClass('js-issuable-form-dropdown')) {
+ return;
+ }
+
+ if ($dropdown.hasClass('js-filter-bulk-update')) {
_this.enableBulkLabelDropdown();
_this.setDropdownData($dropdown, isMarking, this.id(label));
return;
diff --git a/changelogs/unreleased/label-select-toggle.yml b/changelogs/unreleased/label-select-toggle.yml
new file mode 100644
index 00000000000..af5b4246521
--- /dev/null
+++ b/changelogs/unreleased/label-select-toggle.yml
@@ -0,0 +1,4 @@
+---
+title: Fixed label dropdown toggle text not correctly updating
+merge_request:
+author:
diff --git a/spec/features/issues/form_spec.rb b/spec/features/issues/form_spec.rb
index 8771cc8e157..741ca95f1ca 100644
--- a/spec/features/issues/form_spec.rb
+++ b/spec/features/issues/form_spec.rb
@@ -68,6 +68,22 @@ describe 'New/edit issue', feature: true, js: true do
end
end
end
+
+ it 'correctly updates the dropdown toggle when removing a label' do
+ click_button 'Labels'
+
+ page.within '.dropdown-menu-labels' do
+ click_link label.title
+ end
+
+ expect(find('.js-label-select')).to have_content(label.title)
+
+ page.within '.dropdown-menu-labels' do
+ click_link label.title
+ end
+
+ expect(find('.js-label-select')).to have_content('Labels')
+ end
end
context 'edit issue' do