Merge branch 'ph-even-more-es-modules' into 'master'

Exported JS classes as modules

See merge request gitlab-org/gitlab-ce!15960

app/assets/javascripts/boards/components/board_sidebar.js

 /* eslint-disable comma-dangle, space-before-function-paren, no-new */
 /* global MilestoneSelect */
-/* global Sidebar */
 
 import Vue from 'vue';
 import Flash from '../../flash';
+import Sidebar from '../../right_sidebar';
 import eventHub from '../../sidebar/event_hub';
 import assigneeTitle from '../../sidebar/components/assignees/assignee_title';
 import assignees from '../../sidebar/components/assignees/assignees';

app/assets/javascripts/dispatcher.js

@@ -11,7 +11,7 @@ import NotificationsForm from './notifications_form';
 import notificationsDropdown from './notifications_dropdown';
 import groupAvatar from './group_avatar';
 import GroupLabelSubscription from './group_label_subscription';
-/* global LineHighlighter */
+import LineHighlighter from './line_highlighter';
 import BuildArtifacts from './build_artifacts';
 import CILintEditor from './ci_lint_editor';
 import groupsSelect from './groups_select';
@@ -21,7 +21,7 @@ import NamespaceSelect from './namespace_select';
 import NewCommitForm from './new_commit_form';
 import Project from './project';
 import projectAvatar from './project_avatar';
-/* global MergeRequest */
+import MergeRequest from './merge_request';
 import Compare from './compare';
 import initCompareAutocomplete from './compare_autocomplete';
 import ProjectFindFile from './project_find_file';
@@ -29,7 +29,7 @@ import ProjectNew from './project_new';
 import projectImport from './project_import';
 import Labels from './labels';
 import LabelManager from './label_manager';
-/* global Sidebar */
+import Sidebar from './right_sidebar';
 import IssuableTemplateSelectors from './templates/issuable_template_selectors';
 import Flash from './flash';
 import CommitsList from './commits';

app/assets/javascripts/init_issuable_sidebar.js

@@ -2,7 +2,7 @@
 /* global MilestoneSelect */
 import LabelsSelect from './labels_select';
 import IssuableContext from './issuable_context';
-/* global Sidebar */
+import Sidebar from './right_sidebar';
 
 import DueDateSelectors from './due_date_select';
 
@@ -15,5 +15,5 @@ export default () => {
   new LabelsSelect();
   new IssuableContext(sidebarOptions.currentUser);
   new DueDateSelectors();
-  window.sidebar = new Sidebar();
+  Sidebar.initialize();
 };

app/assets/javascripts/line_highlighter.js

@@ -175,4 +175,4 @@ LineHighlighter.prototype.__setLocationHash__ = function(value) {
   }, document.title, value);
 };
 
-window.LineHighlighter = LineHighlighter;
+export default LineHighlighter;

app/assets/javascripts/main.js

@@ -45,14 +45,12 @@ import './layout_nav';
 import LazyLoader from './lazy_loader';
 import './line_highlighter';
 import initLogoAnimation from './logo';
-import './merge_request';
 import './merge_request_tabs';
 import './milestone_select';
 import './notes';
 import './preview_markdown';
 import './projects_dropdown';
 import './render_gfm';
-import './right_sidebar';
 import initBreadcrumbs from './breadcrumb';
 
 import './dispatcher';

app/assets/javascripts/merge_request.js

 /* eslint-disable func-names, space-before-function-paren, no-var, prefer-rest-params, wrap-iife, quotes, no-underscore-dangle, one-var, one-var-declaration-per-line, consistent-return, dot-notation, quote-props, comma-dangle, object-shorthand, max-len, prefer-arrow-callback */
-/* global MergeRequestTabs */
 
 import 'vendor/jquery.waitforimages';
 import TaskList from './task_list';
@@ -7,9 +6,7 @@ import MergeRequestTabs from './merge_request_tabs';
 import IssuablesHelper from './helpers/issuables_helper';
 import { addDelimiter } from './lib/utils/text_utility';
 
-(function() {
-  this.MergeRequest = (function() {
-    function MergeRequest(opts) {
+function MergeRequest(opts) {
   // Initialize MergeRequest behavior
   //
   // Options:
@@ -40,26 +37,26 @@ import { addDelimiter } from './lib/utils/text_utility';
       }
     });
   }
-    }
+}
 
-    // Local jQuery finder
-    MergeRequest.prototype.$ = function(selector) {
+// Local jQuery finder
+MergeRequest.prototype.$ = function(selector) {
   return this.$el.find(selector);
-    };
+};
 
-    MergeRequest.prototype.initTabs = function() {
+MergeRequest.prototype.initTabs = function() {
   if (window.mrTabs) {
     window.mrTabs.unbindEvents();
   }
   window.mrTabs = new MergeRequestTabs(this.opts);
-    };
+};
 
-    MergeRequest.prototype.showAllCommits = function() {
+MergeRequest.prototype.showAllCommits = function() {
   this.$('.first-commits').remove();
   return this.$('.all-commits').removeClass('hide');
-    };
+};
 
-    MergeRequest.prototype.initMRBtnListeners = function() {
+MergeRequest.prototype.initMRBtnListeners = function() {
   var _this;
   _this = this;
   return $('a.btn-close, a.btn-reopen').on('click', function(e) {
@@ -81,9 +78,9 @@ import { addDelimiter } from './lib/utils/text_utility';
       }
     }
   });
-    };
+};
 
-    MergeRequest.prototype.submitNoteForm = function(form, $button) {
+MergeRequest.prototype.submitNoteForm = function(form, $button) {
   var noteText;
   noteText = form.find("textarea.js-note-text").val();
   if (noteText.trim().length > 0) {
@@ -91,9 +88,9 @@ import { addDelimiter } from './lib/utils/text_utility';
     $button.data('submitted', true);
     return $button.trigger('click');
   }
-    };
+};
 
-    MergeRequest.prototype.initCommitMessageListeners = function() {
+MergeRequest.prototype.initCommitMessageListeners = function() {
   $(document).on('click', 'a.js-with-description-link', function(e) {
     var textarea = $('textarea.js-commit-message');
     e.preventDefault();
@@ -111,24 +108,24 @@ import { addDelimiter } from './lib/utils/text_utility';
     $('.js-with-description-hint').show();
     $('.js-without-description-hint').hide();
   });
-    };
+};
 
-    MergeRequest.prototype.updateStatusText = function(classToRemove, classToAdd, newStatusText) {
+MergeRequest.prototype.updateStatusText = function(classToRemove, classToAdd, newStatusText) {
   $('.detail-page-header .status-box')
     .removeClass(classToRemove)
     .addClass(classToAdd)
     .find('span')
     .text(newStatusText);
-    };
+};
 
-    MergeRequest.prototype.decreaseCounter = function(by = 1) {
+MergeRequest.prototype.decreaseCounter = function(by = 1) {
   const $el = $('.nav-links .js-merge-counter');
   const count = Math.max((parseInt($el.text().replace(/[^\d]/, ''), 10) - by), 0);
 
   $el.text(addDelimiter(count));
-    };
+};
 
-    MergeRequest.prototype.hideCloseButton = function() {
+MergeRequest.prototype.hideCloseButton = function() {
   const el = document.querySelector('.merge-request .js-issuable-actions');
   const closeDropdownItem = el.querySelector('li.close-item');
   if (closeDropdownItem) {
@@ -141,8 +138,6 @@ import { addDelimiter } from './lib/utils/text_utility';
   }
   // Dropdown for mobile screen
   el.querySelector('li.js-close-item').classList.add('hidden');
-    };
+};
 
-    return MergeRequest;
-  })();
-}).call(window);
+export default MergeRequest;

app/assets/javascripts/repo/components/repo_preview.vue

 <script>
-/* global LineHighlighter */
 import { mapGetters } from 'vuex';
+import LineHighlighter from '../../line_highlighter';
 import syntaxHighlight from '../../syntax_highlight';
 
 export default {

app/assets/javascripts/right_sidebar.js

@@ -3,25 +3,29 @@
 import _ from 'underscore';
 import Cookies from 'js-cookie';
 
-(function() {
-  this.Sidebar = (function() {
-    function Sidebar(currentUser) {
+function Sidebar(currentUser) {
   this.toggleTodo = this.toggleTodo.bind(this);
   this.sidebar = $('aside');
 
   this.removeListeners();
   this.addEventListeners();
+}
+
+Sidebar.initialize = function(currentUser) {
+  if (!this.instance) {
+    this.instance = new Sidebar(currentUser);
   }
+};
 
-    Sidebar.prototype.removeListeners = function () {
+Sidebar.prototype.removeListeners = function () {
   this.sidebar.off('click', '.sidebar-collapsed-icon');
   this.sidebar.off('hidden.gl.dropdown');
   $('.dropdown').off('loading.gl.dropdown');
   $('.dropdown').off('loaded.gl.dropdown');
   $(document).off('click', '.js-sidebar-toggle');
-    };
+};
 
-    Sidebar.prototype.addEventListeners = function() {
+Sidebar.prototype.addEventListeners = function() {
   const $document = $(document);
 
   this.sidebar.on('click', '.sidebar-collapsed-icon', this, this.sidebarCollapseClicked);
@@ -31,9 +35,9 @@ import Cookies from 'js-cookie';
 
   $document.on('click', '.js-sidebar-toggle', this.sidebarToggleClicked);
   return $(document).off('click', '.js-issuable-todo').on('click', '.js-issuable-todo', this.toggleTodo);
-    };
+};
 
-    Sidebar.prototype.sidebarToggleClicked = function (e, triggered) {
+Sidebar.prototype.sidebarToggleClicked = function (e, triggered) {
   var $allGutterToggleIcons, $this, $thisIcon;
   e.preventDefault();
   $this = $(this);
@@ -53,9 +57,9 @@ import Cookies from 'js-cookie';
   if (!triggered) {
     Cookies.set("collapsed_gutter", $('.right-sidebar').hasClass('right-sidebar-collapsed'));
   }
-    };
+};
 
-    Sidebar.prototype.toggleTodo = function(e) {
+Sidebar.prototype.toggleTodo = function(e) {
   var $btnText, $this, $todoLoading, ajaxType, url;
   $this = $(e.currentTarget);
   ajaxType = $this.attr('data-delete-path') ? 'DELETE' : 'POST';
@@ -86,9 +90,9 @@ import Cookies from 'js-cookie';
       return _this.todoUpdateDone(data);
     };
   })(this));
-    };
+};
 
-    Sidebar.prototype.todoUpdateDone = function(data) {
+Sidebar.prototype.todoUpdateDone = function(data) {
   const deletePath = data.delete_path ? data.delete_path : null;
   const attrPrefix = deletePath ? 'mark' : 'todo';
   const $todoBtns = $('.js-issuable-todo');
@@ -115,9 +119,9 @@ import Cookies from 'js-cookie';
       $elText.text($el.data(`${attrPrefix}-text`));
     }
   });
-    };
+};
 
-    Sidebar.prototype.sidebarDropdownLoading = function(e) {
+Sidebar.prototype.sidebarDropdownLoading = function(e) {
   var $loading, $sidebarCollapsedIcon, i, img;
   $sidebarCollapsedIcon = $(this).closest('.block').find('.sidebar-collapsed-icon');
   img = $sidebarCollapsedIcon.find('img');
@@ -130,9 +134,9 @@ import Cookies from 'js-cookie';
     i.before($loading);
     return i.hide();
   }
-    };
+};
 
-    Sidebar.prototype.sidebarDropdownLoaded = function(e) {
+Sidebar.prototype.sidebarDropdownLoaded = function(e) {
   var $sidebarCollapsedIcon, i, img;
   $sidebarCollapsedIcon = $(this).closest('.block').find('.sidebar-collapsed-icon');
   img = $sidebarCollapsedIcon.find('img');
@@ -143,9 +147,9 @@ import Cookies from 'js-cookie';
   } else {
     return i.show();
   }
-    };
+};
 
-    Sidebar.prototype.sidebarCollapseClicked = function(e) {
+Sidebar.prototype.sidebarCollapseClicked = function(e) {
   var $block, sidebar;
   if ($(e.currentTarget).hasClass('dont-change-state')) {
     return;
@@ -154,9 +158,9 @@ import Cookies from 'js-cookie';
   e.preventDefault();
   $block = $(this).closest('.block');
   return sidebar.openDropdown($block);
-    };
+};
 
-    Sidebar.prototype.openDropdown = function(blockOrName) {
+Sidebar.prototype.openDropdown = function(blockOrName) {
   var $block;
   $block = _.isString(blockOrName) ? this.getBlock(blockOrName) : blockOrName;
   if (!this.isOpen()) {
@@ -169,34 +173,34 @@ import Cookies from 'js-cookie';
   setTimeout(() => {
     $block.find('.js-sidebar-dropdown-toggle').trigger('click');
   });
-    };
+};
 
-    Sidebar.prototype.setCollapseAfterUpdate = function($block) {
+Sidebar.prototype.setCollapseAfterUpdate = function($block) {
   $block.addClass('collapse-after-update');
   return $('.layout-page').addClass('with-overlay');
-    };
+};
 
-    Sidebar.prototype.onSidebarDropdownHidden = function(e) {
+Sidebar.prototype.onSidebarDropdownHidden = function(e) {
   var $block, sidebar;
   sidebar = e.data;
   e.preventDefault();
   $block = $(e.target).closest('.block');
   return sidebar.sidebarDropdownHidden($block);
-    };
+};
 
-    Sidebar.prototype.sidebarDropdownHidden = function($block) {
+Sidebar.prototype.sidebarDropdownHidden = function($block) {
   if ($block.hasClass('collapse-after-update')) {
     $block.removeClass('collapse-after-update');
     $('.layout-page').removeClass('with-overlay');
     return this.toggleSidebar('hide');
   }
-    };
+};
 
-    Sidebar.prototype.triggerOpenSidebar = function() {
+Sidebar.prototype.triggerOpenSidebar = function() {
   return this.sidebar.find('.js-sidebar-toggle').trigger('click');
-    };
+};
 
-    Sidebar.prototype.toggleSidebar = function(action) {
+Sidebar.prototype.toggleSidebar = function(action) {
   if (action == null) {
     action = 'toggle';
   }
@@ -213,16 +217,14 @@ import Cookies from 'js-cookie';
       return this.triggerOpenSidebar();
     }
   }
-    };
+};
 
-    Sidebar.prototype.isOpen = function() {
+Sidebar.prototype.isOpen = function() {
   return this.sidebar.is('.right-sidebar-expanded');
-    };
+};
 
-    Sidebar.prototype.getBlock = function(name) {
+Sidebar.prototype.getBlock = function(name) {
   return this.sidebar.find(".block." + name);
-    };
+};
 
-    return Sidebar;
-  })();
-}).call(window);
+export default Sidebar;

app/assets/javascripts/shortcuts_issuable.js

 /* global Mousetrap */
-/* global sidebar */
 
 import _ from 'underscore';
 import 'mousetrap';
+import Sidebar from './right_sidebar';
 import ShortcutsNavigation from './shortcuts_navigation';
 import { CopyAsGFM } from './behaviors/copy_as_gfm';
 
@@ -69,7 +69,7 @@ export default class ShortcutsIssuable extends ShortcutsNavigation {
   }
 
   static openSidebarDropdown(name) {
-    sidebar.openDropdown(name);
+    Sidebar.instance.openDropdown(name);
     return false;
   }
 }

spec/javascripts/collapsed_sidebar_todo_spec.js

-/* global Sidebar */
 /* eslint-disable no-new */
 import _ from 'underscore';
-import '~/right_sidebar';
+import Sidebar from '~/right_sidebar';
 
 describe('Issuable right sidebar collapsed todo toggle', () => {
   const fixtureName = 'issues/open-issue.html.raw';

spec/javascripts/line_highlighter_spec.js

 /* eslint-disable space-before-function-paren, no-var, no-param-reassign, quotes, prefer-template, no-else-return, new-cap, dot-notation, no-return-assign, comma-dangle, no-new, one-var, one-var-declaration-per-line, jasmine/no-spec-dupes, no-underscore-dangle, max-len */
-/* global LineHighlighter */
 
-import '~/line_highlighter';
+import LineHighlighter from '~/line_highlighter';
 
 (function() {
   describe('LineHighlighter', function() {

spec/javascripts/merge_request_spec.js

 /* eslint-disable space-before-function-paren, no-return-assign */
-/* global MergeRequest */
 
-import '~/merge_request';
+import MergeRequest from '~/merge_request';
 import CloseReopenReportToggle from '~/close_reopen_report_toggle';
 import IssuablesHelper from '~/helpers/issuables_helper';
 

spec/javascripts/right_sidebar_spec.js

 /* eslint-disable space-before-function-paren, no-var, one-var, one-var-declaration-per-line, new-parens, no-return-assign, new-cap, vars-on-top, max-len */
-/* global Sidebar */
 
 import '~/commons/bootstrap';
-import '~/right_sidebar';
+import Sidebar from '~/right_sidebar';
 
 (function() {
   var $aside, $icon, $labelsIcon, $page, $toggle, assertSidebarState;