use export function instead of export {}, add more comments

Author: wxiaoguang

web_src/js/components/DashboardRepoList.js

@@ -348,7 +348,7 @@ function initVueComponents() {
 }
 
 
-function initDashboardRepoList() {
+export function initDashboardRepoList() {
   const el = document.getElementById('dashboard-repo-list');
   const dashboardRepoListData = pageData.dashboardRepoList || null;
   if (!el || !dashboardRepoListData) return;
@@ -366,5 +366,3 @@ function initDashboardRepoList() {
     },
   });
 }
-
-export {initDashboardRepoList};

web_src/js/components/RepoActivityTopAuthors.vue

@@ -101,10 +101,9 @@ const sfc = {
   }
 };
 
-function initRepoActivityTopAuthorsChart() {
+export function initRepoActivityTopAuthorsChart() {
   initVueApp('#repo-activity-top-authors-chart', sfc);
 }
 
-export default sfc;
-export {initRepoActivityTopAuthorsChart};
+export default sfc; // this line is necessary to activate the IDE's Vue plugin
 </script>

web_src/js/components/RepoBranchTagDropdown.js

@@ -1,6 +1,7 @@
 import Vue from 'vue';
+import {vueDelimiters} from './VueComponentLoader.js';
 
-function initRepoBranchTagDropdown(selector) {
+export function initRepoBranchTagDropdown(selector) {
   $(selector).each(function () {
     const $dropdown = $(this);
     const $data = $dropdown.find('.data');
@@ -26,7 +27,7 @@ function initRepoBranchTagDropdown(selector) {
     $data.remove();
     new Vue({
       el: this,
-      delimiters: ['${', '}'],
+      delimiters: vueDelimiters,
       data,
       computed: {
         filteredItems() {
@@ -157,5 +158,3 @@ function initRepoBranchTagDropdown(selector) {
     });
   });
 }
-
-export {initRepoBranchTagDropdown};

web_src/js/components/VueComponentLoader.js

@@ -1,10 +1,10 @@
 import Vue from 'vue';
 import {svgs} from '../svg.js';
 
-const vueDelimiters = ['${', '}'];
+export const vueDelimiters = ['${', '}'];
 
 let vueEnvInited = false;
-function initVueEnv() {
+export function initVueEnv() {
   if (vueEnvInited) return;
   vueEnvInited = true;
 
@@ -14,7 +14,7 @@ function initVueEnv() {
 }
 
 let vueSvgInited = false;
-function initVueSvg() {
+export function initVueSvg() {
   if (vueSvgInited) return;
   vueSvgInited = true;
 
@@ -36,8 +36,7 @@ function initVueSvg() {
   }
 }
 
-
-function initVueApp(el, opts = {}) {
+export function initVueApp(el, opts = {}) {
   if (typeof el === 'string') {
     el = document.querySelector(el);
   }
@@ -48,5 +47,3 @@ function initVueApp(el, opts = {}) {
     delimiters: vueDelimiters,
   }, opts));
 }
-
-export {vueDelimiters, initVueEnv, initVueSvg, initVueApp};

web_src/js/features/admin-common.js

@@ -1,6 +1,6 @@
 const {csrf} = window.config;
 
-function initAdminCommon() {
+export function initAdminCommon() {
   if ($('.admin').length === 0) {
     return;
   }
@@ -212,5 +212,3 @@ function initAdminCommon() {
     });
   }
 }
-
-export {initAdminCommon};

web_src/js/features/admin-emails.js

@@ -1,4 +1,4 @@
-function initAdminEmails() {
+export function initAdminEmails() {
   function linkEmailAction(e) {
     const $this = $(this);
     $('#form-uid').val($this.data('uid'));
@@ -10,5 +10,3 @@ function initAdminEmails() {
   }
   $('.link-email-action').on('click', linkEmailAction);
 }
-
-export {initAdminEmails};

web_src/js/features/common-global.js

@@ -6,15 +6,15 @@ import 'jquery.are-you-sure';
 
 const {csrf} = window.config;
 
-function initGlobalFormDirtyLeaveConfirm() {
+export function initGlobalFormDirtyLeaveConfirm() {
   // Warn users that try to leave a page after entering data into a form.
   // Except on sign-in pages, and for forms marked as 'ignore-dirty'.
   if ($('.user.signin').length === 0) {
     $('form:not(.ignore-dirty)').areYouSure();
   }
 }
 
-function initHeadNavbarContentToggle() {
+export function initHeadNavbarContentToggle() {
   const content = $('#navbar');
   const toggle = $('#navbar-expand-toggle');
   let isExpanded = false;
@@ -30,7 +30,7 @@ function initHeadNavbarContentToggle() {
   });
 }
 
-function initFootLanguageMenu() {
+export function initFootLanguageMenu() {
   function linkLanguageAction() {
     const $this = $(this);
     $.post($this.data('url')).always(() => {
@@ -42,23 +42,23 @@ function initFootLanguageMenu() {
 }
 
 
-function initGlobalEnterQuickSubmit() {
+export function initGlobalEnterQuickSubmit() {
   $('.js-quick-submit').on('keydown', function (e) {
     if (((e.ctrlKey && !e.altKey) || e.metaKey) && (e.keyCode === 13 || e.keyCode === 10)) {
       $(this).closest('form').trigger('submit');
     }
   });
 }
 
-function initGlobalButtonClickOnEnter() {
+export function initGlobalButtonClickOnEnter() {
   $(document).on('keypress', '.ui.button', (e) => {
     if (e.keyCode === 13 || e.keyCode === 32) { // enter key or space bar
       $(e.target).trigger('click');
     }
   });
 }
 
-function initGlobalCommon() {
+export function initGlobalCommon() {
   // Show exact time
   $('.time-since').each(function () {
     $(this)
@@ -130,7 +130,7 @@ function initGlobalCommon() {
   });
 }
 
-async function initGlobalDropzone() {
+export async function initGlobalDropzone() {
   // Dropzone
   for (const el of document.querySelectorAll('.dropzone')) {
     const $dropzone = $(el);
@@ -168,7 +168,7 @@ async function initGlobalDropzone() {
   }
 }
 
-function initGlobalLinkActions() {
+export function initGlobalLinkActions() {
   function showDeletePopup() {
     const $this = $(this);
     const dataArray = $this.data();
@@ -278,7 +278,7 @@ function initGlobalLinkActions() {
   });
 }
 
-function initGlobalButtons() {
+export function initGlobalButtons() {
   $('.show-panel.button').on('click', function () {
     $($(this).data('panel')).show();
   });
@@ -304,15 +304,3 @@ function initGlobalButtons() {
     });
   });
 }
-
-export {
-  initHeadNavbarContentToggle,
-  initFootLanguageMenu,
-  initGlobalEnterQuickSubmit,
-  initGlobalFormDirtyLeaveConfirm,
-  initGlobalButtonClickOnEnter,
-  initGlobalCommon,
-  initGlobalDropzone,
-  initGlobalLinkActions,
-  initGlobalButtons,
-};

web_src/js/features/common-issue.js

@@ -1,6 +1,6 @@
 import {updateIssuesMeta} from './repo-issue.js';
 
-function initCommonIssue() {
+export function initCommonIssue() {
   $('.issue-checkbox').on('click', () => {
     const numChecked = $('.issue-checkbox').children('input:checked').length;
     if (numChecked > 0) {
@@ -38,5 +38,3 @@ function initCommonIssue() {
     $(e).trigger('click');
   });
 }
-
-export {initCommonIssue};

web_src/js/features/common-organization.js

@@ -1,11 +1,10 @@
 import {initCompLabelEdit} from './comp/LabelEdit.js';
 
-function initCommonOrganization() {
+export function initCommonOrganization() {
   if ($('.organization').length === 0) {
     return;
   }
 
-  // Options
   if ($('.organization.settings.options').length > 0) {
     $('#org_name').on('keyup', function () {
       const $prompt = $('#org-name-change-prompt');
@@ -23,5 +22,3 @@ function initCommonOrganization() {
   // Labels
   initCompLabelEdit('.organization.settings.labels');
 }
-
-export {initCommonOrganization};

web_src/js/features/comp/ColorPicker.js

@@ -1,6 +1,6 @@
 import createColorPicker from '../colorpicker.js';
 
-function initCompColorPicker() {
+export function initCompColorPicker() {
   createColorPicker($('.color-picker'));
 
   $('.precolors .color').on('click', function () {
@@ -9,5 +9,3 @@ function initCompColorPicker() {
     $('.minicolors-swatch-color').css('background-color', color_hex);
   });
 }
-
-export {initCompColorPicker};