Revert "Merge branch '37179-dashboard-project-dropdown' into 'master'"

app/assets/javascripts/project_select_combo_button.js

@@ -14,14 +14,7 @@ export default class ProjectSelectComboButton {
 
   bindEvents() {
     this.projectSelectInput.siblings('.new-project-item-select-button')
-      .on('click', e => this.openDropdown(e));
-
-    this.newItemBtn.on('click', (e) => {
-      if (!this.getProjectFromLocalStorage()) {
-        e.preventDefault();
-        this.openDropdown(e);
-      }
-    });
+      .on('click', this.openDropdown);
 
     this.projectSelectInput.on('change', () => this.selectProject());
   }
@@ -37,9 +30,8 @@ export default class ProjectSelectComboButton {
     }
   }
 
-  // eslint-disable-next-line class-methods-use-this
-  openDropdown(event) {
-    $(event.currentTarget).siblings('.project-item-select').select2('open');
+  openDropdown() {
+    $(this).siblings('.project-item-select').select2('open');
   }
 
   selectProject() {
@@ -66,8 +58,10 @@ export default class ProjectSelectComboButton {
     if (project) {
       this.newItemBtn.attr('href', project.url);
       this.newItemBtn.text(`${this.formattedText.defaultTextPrefix} in ${project.name}`);
+      this.newItemBtn.enable();
     } else {
       this.newItemBtn.text(`Select project to create ${this.formattedText.presetTextSuffix}`);
+      this.newItemBtn.disable();
     }
   }
 

changelogs/unreleased/37179-dashboard-project-dropdown.yml

----
-title: Removes disabled state from dashboard project button
-merge_request:
-author:
-type: fixed

spec/javascripts/project_select_combo_button_spec.js

@@ -32,6 +32,11 @@ describe('Project Select Combo Button', function () {
       this.comboButton = new ProjectSelectComboButton(this.projectSelectInput);
     });
 
+    it('newItemBtn is disabled', function () {
+      expect(this.newItemBtn.hasAttribute('disabled')).toBe(true);
+      expect(this.newItemBtn.classList.contains('disabled')).toBe(true);
+    });
+
     it('newItemBtn href is null', function () {
       expect(this.newItemBtn.getAttribute('href')).toBe('');
     });
@@ -48,6 +53,11 @@ describe('Project Select Combo Button', function () {
       this.comboButton = new ProjectSelectComboButton(this.projectSelectInput);
     });
 
+    it('newItemBtn is not disabled', function () {
+      expect(this.newItemBtn.hasAttribute('disabled')).toBe(false);
+      expect(this.newItemBtn.classList.contains('disabled')).toBe(false);
+    });
+
     it('newItemBtn href is correctly set', function () {
       expect(this.newItemBtn.getAttribute('href')).toBe(this.defaults.projectMeta.url);
     });
@@ -72,6 +82,11 @@ describe('Project Select Combo Button', function () {
         .trigger('change');
     });
 
+    it('newItemBtn is not disabled', function () {
+      expect(this.newItemBtn.hasAttribute('disabled')).toBe(false);
+      expect(this.newItemBtn.classList.contains('disabled')).toBe(false);
+    });
+
     it('newItemBtn href is correctly set', function () {
       expect(this.newItemBtn.getAttribute('href'))
         .toBe('http://myothercoolproject.com/issues/new');