Changeset 3884:0cea47e6b892 for admin/js

Timestamp:

09/21/18 12:56:21 (7 years ago)

Author:

franck <carnet.franck.paul@…>

Branch:

default

Message:

Use let and const rather than var (ES2015/ES6)

File:

• admin/js/jquery/jquery.pageTabs.js (modified) (12 diffs)

admin/js/jquery/jquery.pageTabs.js

@@ +1 @@
+/*global jQuery, chainHandler */
+'use strict';
+
 (function($) {
-  'use strict';
 
   $.pageTabs = function(start_tab, opts) {
-    var defaults = {
+    const defaults = {
       containerClass: 'part-tabs',
       partPrefix: 'part-',
@@ -12 +14 @@
 
     $.pageTabs.options = $.extend({}, defaults, opts);
-    var active_tab = start_tab || '';
-    var hash = $.pageTabs.getLocationHash();
-    var subhash = $.pageTabs.getLocationSubhash();
+    let active_tab = start_tab || '';
+    const hash = $.pageTabs.getLocationHash();
+    const subhash = $.pageTabs.getLocationSubhash();
 
     if (hash !== undefined && hash) {
@@ -25 +27 @@
     createTabs();
 
-    $('ul li', '.' + $.pageTabs.options.containerClass).click(function(e) {
+    $('ul li', '.' + $.pageTabs.options.containerClass).click(function() {
       if ($(this).hasClass($.pageTabs.options.activeClass)) {
         return;
@@ -34 +36 @@
       $('.' + $.pageTabs.options.contentClass + '.active').removeClass('active').hide();
 
-      var part_to_activate = $('#' + $.pageTabs.options.partPrefix + getHash($(this).find('a').attr('href')));
+      const part_to_activate = $('#' + $.pageTabs.options.partPrefix + getHash($(this).find('a').attr('href')));
 
       part_to_activate.addClass('active').show();
@@ -45 +47 @@
     });
 
-    $(window).bind('hashchange onhashchange', function(e) {
+    $(window).bind('hashchange onhashchange', function() {
       $.pageTabs.clickTab($.pageTabs.getLocationHash());
     });
@@ -52 +54 @@
 
     if (subhash !== undefined) {
-      var elt = document.getElementById(subhash);
+      const elt = document.getElementById(subhash);
       // Tab displayed, now scroll to the sub-part if defined in original document.location (#tab.sub-part)
-      elt.scrollIntoView()
+      elt.scrollIntoView();
       // Give focus to the sub-part if possible
       $('#' + subhash).addClass('focus').focusout(function() {
@@ -65 +67 @@
   };
 
-  var createTabs = function createTabs() {
-    var lis = [],
-      li_class = '';
+  const createTabs = function createTabs() {
+    let lis = [];
 
     $('.' + $.pageTabs.options.contentClass).each(function() {
@@ -80 +81 @@
   };
 
-  var getHash = function getHash(href) {
-    var href = href || '';
+  const getHash = function getHash(href) {
+    href = href || '';
 
     return href.replace(/.*#/, '');
@@ -92 +93 @@
       // try to find anchor in a .multi-part div
       if ($('#' + tab).length == 1) {
-        var div_content = $('#' + tab).parents('.' + $.pageTabs.options.contentClass);
+        const div_content = $('#' + tab).parents('.' + $.pageTabs.options.contentClass);
         if (div_content.length == 1) {
           tab = div_content.attr('id').replace($.pageTabs.options.partPrefix, '');
@@ -110 +111 @@
   $.pageTabs.getLocationHash = function() {
     // Return the URL hash (without subhash — #hash[.subhash])
-    var h = getHash(document.location.hash).split('.');
+    const h = getHash(document.location.hash).split('.');
     return h[0];
   };
   $.pageTabs.getLocationSubhash = function() {
     // Return the URL subhash if present (without hash — #hash[.subhash])
-    var sh = getHash(document.location.hash).split('.');
+    const sh = getHash(document.location.hash).split('.');
     return sh[1];
   };
@@ -123 +124 @@
   this.each(function() {
     if (f) {
-      chainHandler(this, 'tabload', f)
+      chainHandler(this, 'tabload', f);
     } else {
-      var h = this.tabload;
+      const h = this.tabload;
       if (h) {
         h.apply(this);
@@ -139 +140 @@
       chainHandler(this, 'onetabload', f);
     } else {
-      var h = this.onetabload;
+      const h = this.onetabload;
       if (h != null) {
         h.apply(this);