fomantic-ui 2.9.1-beta.23 → 2.9.1-beta.24

package/.eslint/index.js

@@ -0,0 +1,23 @@
+module.exports = {
+    rules: {
+        'no-extra-parens': require('./no-extra-parens-rule'),
+    },
+    configs: {
+        recommended: {
+            plugins: [
+                '@internal/eslint-plugin',
+            ],
+            rules: {
+                // https://github.com/eslint/eslint/issues/16626
+                // https://github.com/airbnb/javascript/blob/eslint-config-airbnb-v19.0.4/packages/eslint-config-airbnb-base/rules/errors.js#L66
+                '@internal/no-extra-parens': ['error', 'all', {
+                    conditionalAssign: true,
+                    enforceForArrowConditionals: false,
+                    ignoreJSX: 'all',
+                    nestedBinaryExpressions: false,
+                    returnAssign: false,
+                }],
+            },
+        },
+    },
+};

package/.eslint/no-extra-parens-rule.js

@@ -0,0 +1,17 @@
+const eslint = require('eslint');
+const ruleComposer = require('eslint-rule-composer');
+
+const rule = new eslint.Linter().getRules().get('no-extra-parens');
+
+module.exports = ruleComposer.filterReports(
+    rule,
+    (problem) => {
+        if (problem.node.type === 'ConditionalExpression'
+            && (problem.node.parent.type === 'ConditionalExpression' || problem.node.parent.type === 'SpreadElement')
+        ) {
+            return false;
+        }
+
+        return problem;
+    }
+);

package/.eslint/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "@internal/eslint-plugin",
+  "version": "1.0.0",
+  "private": true,
+  "dependencies": {
+    "eslint-rule-composer": "^0.3.0"
+  },
+  "peerDependencies": {
+    "eslint": "*"
+  }
+}

package/.eslintrc.js

@@ -6,6 +6,7 @@ module.exports = {
     extends: [
         'airbnb-base',
         'plugin:unicorn/recommended',
+        'plugin:@internal/eslint-plugin/recommended',
     ],
     parserOptions: {
         ecmaVersion: '2020',
@@ -22,10 +23,10 @@ module.exports = {
         'class-methods-use-this': 'off',
         'comma-dangle': ['error', {
             arrays: 'always-multiline',
-            objects: 'always-multiline',
+            exports: 'always-multiline',
             functions: 'never',
             imports: 'always-multiline',
-            exports: 'always-multiline',
+            objects: 'always-multiline',
         }],
         'consistent-return': 'off',
         curly: ['error', 'all'],
@@ -56,8 +57,8 @@ module.exports = {
         'object-shorthand': ['error', 'never'],
         'padding-line-between-statements': ['error', {
             blankLine: 'always',
-            prev: '*',
             next: ['continue', 'break', 'export', 'return', 'throw'],
+            prev: '*',
         }],
         'prefer-destructuring': 'off',
         'prefer-template': 'off',
@@ -95,6 +96,7 @@ module.exports = {
         'one-var': 'off',
         'prefer-const': 'off',
         'prefer-exponentiation-operator': 'off',
+        'prefer-rest-params': 'off',
         'prefer-spread': 'off',
         'semi-style': 'off',
         'unicorn/no-array-for-each': 'off',
@@ -114,15 +116,8 @@ module.exports = {
         eqeqeq: 'off', // about 300 errors to be fixed manually
         'global-require': 'off', // about 30 errors to be fixed manually
         'no-shadow': 'off', // about 220 errors to be fixed manually
+        'no-shadow-restricted-names': 'off', // TODO https://github.com/fomantic/Fomantic-UI/pull/2604
         'prefer-arrow-callback': 'off', // about 350 errors (all autofixable)
-        'prefer-rest-params': 'off', // about 180 errors to be fixed manually
-        'unicorn/no-nested-ternary': 'off', // about 80 errors (except 14 nested ternary operators autofixable)
-
-        // TODO
-        'no-return-assign': 'off',
-        'no-shadow-restricted-names': 'off',
-        'no-use-before-define': 'off',
-        'unicorn/prefer-negative-index': 'off',
     },
     reportUnusedDisableDirectives: true,
     globals: {

package/dist/components/accordion.css

@@ -1,5 +1,5 @@
 /*!
- * # Fomantic-UI 2.9.1-beta.23+27384cb - Accordion
+ * # Fomantic-UI 2.9.1-beta.24+029436f - Accordion
  * https://github.com/fomantic/Fomantic-UI/
  *
  *

package/dist/components/accordion.js

@@ -1,5 +1,5 @@
 /*!
- * # Fomantic-UI 2.9.1-beta.23+27384cb - Accordion
+ * # Fomantic-UI 2.9.1-beta.24+029436f - Accordion
  * https://github.com/fomantic/Fomantic-UI/
  *
  *
@@ -15,7 +15,7 @@
         return typeof obj === 'function' && typeof obj.nodeType !== 'number';
     }
 
-    window = (window !== undefined && window.Math === Math)
+    window = window !== undefined && window.Math === Math
         ? window
         : globalThis;
 
@@ -27,14 +27,14 @@
             performance     = [],
 
             query           = arguments[0],
-            methodInvoked   = (typeof query === 'string'),
+            methodInvoked   = typeof query === 'string',
             queryArguments  = [].slice.call(arguments, 1),
 
             returnedValue
         ;
         $allModules.each(function () {
             var
-                settings        = ($.isPlainObject(parameters))
+                settings        = $.isPlainObject(parameters)
                     ? $.extend(true, {}, $.fn.accordion.settings, parameters)
                     : $.extend({}, $.fn.accordion.settings),
 
@@ -121,16 +121,16 @@
 
                 toggle: function (query) {
                     var
-                        $activeTitle = (query !== undefined)
-                            ? (typeof query === 'number')
+                        $activeTitle = query !== undefined
+                            ? (typeof query === 'number'
                                 ? $title.eq(query)
-                                : $(query).closest(selector.title)
+                                : $(query).closest(selector.title))
                             : $(this).closest(selector.title),
                         $activeContent = $activeTitle.next($content),
                         isAnimating = $activeContent.hasClass(className.animating),
                         isActive    = $activeContent.hasClass(className.active),
-                        isOpen      = (isActive && !isAnimating),
-                        isOpening   = (!isActive && isAnimating)
+                        isOpen      = isActive && !isAnimating,
+                        isOpening   = !isActive && isAnimating
                     ;
                     module.debug('Toggling visibility of content', $activeTitle);
                     if (isOpen || isOpening) {
@@ -146,15 +146,15 @@
 
                 open: function (query) {
                     var
-                        $activeTitle = (query !== undefined)
-                            ? (typeof query === 'number')
+                        $activeTitle = query !== undefined
+                            ? (typeof query === 'number'
                                 ? $title.eq(query)
-                                : $(query).closest(selector.title)
+                                : $(query).closest(selector.title))
                             : $(this).closest(selector.title),
                         $activeContent = $activeTitle.next($content),
                         isAnimating = $activeContent.hasClass(className.animating),
                         isActive    = $activeContent.hasClass(className.active),
-                        isOpen      = (isActive || isAnimating)
+                        isOpen      = isActive || isAnimating
                     ;
                     if (isOpen) {
                         module.debug('Accordion already open, skipping', $activeContent);
@@ -216,16 +216,16 @@
 
                 close: function (query) {
                     var
-                        $activeTitle = (query !== undefined)
-                            ? (typeof query === 'number')
+                        $activeTitle = query !== undefined
+                            ? (typeof query === 'number'
                                 ? $title.eq(query)
-                                : $(query).closest(selector.title)
+                                : $(query).closest(selector.title))
                             : $(this).closest(selector.title),
                         $activeContent = $activeTitle.next($content),
                         isAnimating    = $activeContent.hasClass(className.animating),
                         isActive       = $activeContent.hasClass(className.active),
-                        isOpening      = (!isActive && isAnimating),
-                        isClosing      = (isActive && isAnimating)
+                        isOpening      = !isActive && isAnimating,
+                        isClosing      = isActive && isAnimating
                     ;
                     if ((isActive || isOpening) && !isClosing) {
                         module.debug('Closing accordion content', $activeContent);
@@ -278,7 +278,7 @@
 
                 closeOthers: function (index) {
                     var
-                        $activeTitle = (index !== undefined)
+                        $activeTitle = index !== undefined
                             ? $title.eq(index)
                             : $(this).closest(selector.title),
                         $parentTitles    = $activeTitle.parents(selector.content).prev(selector.title),
@@ -298,7 +298,7 @@
                         $openTitles = $openTitles.not($nestedTitles);
                         $openContents = $openTitles.next($content);
                     }
-                    if (($openTitles.length > 0)) {
+                    if ($openTitles.length > 0) {
                         module.debug('Exclusive enabled, closing other content', $openTitles);
                         $openTitles
                             .removeClass(className.active)
@@ -482,7 +482,7 @@
                         query = query.split(/[ .]/);
                         maxDepth = query.length - 1;
                         $.each(query, function (depth, value) {
-                            var camelCaseValue = (depth != maxDepth)
+                            var camelCaseValue = depth != maxDepth
                                 ? value + query[depth + 1].charAt(0).toUpperCase() + query[depth + 1].slice(1)
                                 : query;
                             if ($.isPlainObject(object[camelCaseValue]) && (depth != maxDepth)) {
@@ -533,7 +533,7 @@
             }
         });
 
-        return (returnedValue !== undefined)
+        return returnedValue !== undefined
             ? returnedValue
             : this;
     };

package/dist/components/accordion.min.css

@@ -1,5 +1,5 @@
 /*!
- * # Fomantic-UI 2.9.1-beta.23+27384cb - Accordion
+ * # Fomantic-UI 2.9.1-beta.24+029436f - Accordion
  * https://github.com/fomantic/Fomantic-UI/
  *
  *

package/dist/components/accordion.min.js

@@ -1,5 +1,5 @@
 /*
- * # Fomantic UI - 2.9.1-beta.23+27384cb
+ * # Fomantic UI - 2.9.1-beta.24+029436f
  * https://github.com/fomantic/Fomantic-UI
  * https://fomantic-ui.com/
  *

package/dist/components/ad.css

@@ -1,5 +1,5 @@
 /*!
- * # Fomantic-UI 2.9.1-beta.23+27384cb - Ad
+ * # Fomantic-UI 2.9.1-beta.24+029436f - Ad
  * https://github.com/fomantic/Fomantic-UI/
  *
  *

package/dist/components/ad.min.css

@@ -1,5 +1,5 @@
 /*!
- * # Fomantic-UI 2.9.1-beta.23+27384cb - Ad
+ * # Fomantic-UI 2.9.1-beta.24+029436f - Ad
  * https://github.com/fomantic/Fomantic-UI/
  *
  *

package/dist/components/api.js

@@ -1,5 +1,5 @@
 /*!
- * # Fomantic-UI 2.9.1-beta.23+27384cb - API
+ * # Fomantic-UI 2.9.1-beta.24+029436f - API
  * https://github.com/fomantic/Fomantic-UI/
  *
  *
@@ -19,7 +19,7 @@
         return typeof obj === 'function' && typeof obj.nodeType !== 'number';
     }
 
-    window = (window !== undefined && window.Math === Math)
+    window = window !== undefined && window.Math === Math
         ? window
         : globalThis;
 
@@ -34,7 +34,7 @@
             performance    = [],
 
             query          = arguments[0],
-            methodInvoked  = (typeof query === 'string'),
+            methodInvoked  = typeof query === 'string',
             queryArguments = [].slice.call(arguments, 1),
 
             returnedValue
@@ -42,7 +42,7 @@
 
         $allModules.each(function () {
             var
-                settings          = ($.isPlainObject(parameters))
+                settings          = $.isPlainObject(parameters)
                     ? $.extend(true, {}, $.fn.api.settings, parameters)
                     : $.extend({}, $.fn.api.settings),
 
@@ -62,7 +62,7 @@
                 $form           = $module.closest(selector.form),
 
                 // context used for state
-                $context        = (settings.stateContext)
+                $context        = settings.stateContext
                     ? ([window, document].indexOf(settings.stateContext) < 0 ? $(document).find(settings.stateContext) : $(settings.stateContext))
                     : $module,
 
@@ -274,13 +274,13 @@
 
                 should: {
                     removeError: function () {
-                        return (settings.hideError === true || (settings.hideError === 'auto' && !module.is.form()));
+                        return settings.hideError === true || (settings.hideError === 'auto' && !module.is.form());
                     },
                 },
 
                 is: {
                     disabled: function () {
-                        return ($module.filter(selector.disabled).length > 0);
+                        return $module.filter(selector.disabled).length > 0;
                     },
                     expectingJSON: function () {
                         return settings.dataType === 'json' || settings.dataType === 'jsonp';
@@ -289,14 +289,14 @@
                         return $module.is('form') || $context.is('form');
                     },
                     mocked: function () {
-                        return (settings.mockResponse || settings.mockResponseAsync || settings.response || settings.responseAsync);
+                        return settings.mockResponse || settings.mockResponseAsync || settings.response || settings.responseAsync;
                     },
                     input: function () {
                         return $module.is('input');
                     },
                     loading: function () {
-                        return (module.request)
-                            ? (module.request.state() == 'pending')
+                        return module.request
+                            ? module.request.state() == 'pending'
                             : false;
                     },
                     abortedRequest: function (xhr) {
@@ -311,7 +311,7 @@
                         return false;
                     },
                     validResponse: function (response) {
-                        if ((!module.is.expectingJSON()) || !isFunction(settings.successTest)) {
+                        if (!module.is.expectingJSON() || !isFunction(settings.successTest)) {
                             module.verbose('Response is not JSON, skipping validation', settings.successTest, response);
 
                             return true;
@@ -331,16 +331,16 @@
 
                 was: {
                     cancelled: function () {
-                        return (module.cancelled || false);
+                        return module.cancelled || false;
                     },
                     successful: function () {
-                        return (module.request && module.request.state() == 'resolved');
+                        return module.request && module.request.state() == 'resolved';
                     },
                     failure: function () {
-                        return (module.request && module.request.state() == 'rejected');
+                        return module.request && module.request.state() == 'rejected';
                     },
                     complete: function () {
-                        return (module.request && (module.request.state() == 'resolved' || module.request.state() == 'rejected'));
+                        return module.request && (module.request.state() == 'resolved' || module.request.state() == 'rejected');
                     },
                 },
 
@@ -359,16 +359,16 @@
                                 $.each(requiredVariables, function (index, templatedString) {
                                     var
                                         // allow legacy {$var} style
-                                        variable = (templatedString.indexOf('$') !== -1)
+                                        variable = templatedString.indexOf('$') !== -1
                                             ? templatedString.slice(2, -1)
                                             : templatedString.slice(1, -1),
-                                        value   = ($.isPlainObject(urlData) && urlData[variable] !== undefined)
+                                        value   = $.isPlainObject(urlData) && urlData[variable] !== undefined
                                             ? urlData[variable]
-                                            : ($module.data(variable) !== undefined)
+                                            : ($module.data(variable) !== undefined
                                                 ? $module.data(variable)
-                                                : ($context.data(variable) !== undefined)
+                                                : ($context.data(variable) !== undefined // eslint-disable-line unicorn/no-nested-ternary
                                                     ? $context.data(variable)
-                                                    : urlData[variable]
+                                                    : urlData[variable]))
                                     ;
                                     // remove value
                                     if (value === undefined) {
@@ -379,7 +379,7 @@
                                     }
 
                                     module.verbose('Found required variable', variable, value);
-                                    value = (settings.encodeParameters)
+                                    value = settings.encodeParameters
                                         ? module.get.urlEncodedValue(value)
                                         : value;
                                     url = url.replace(templatedString, value);
@@ -390,16 +390,16 @@
                                 $.each(optionalVariables, function (index, templatedString) {
                                     var
                                         // allow legacy {/$var} style
-                                        variable = (templatedString.indexOf('$') !== -1)
+                                        variable = templatedString.indexOf('$') !== -1
                                             ? templatedString.slice(3, -1)
                                             : templatedString.slice(2, -1),
-                                        value   = ($.isPlainObject(urlData) && urlData[variable] !== undefined)
+                                        value   = $.isPlainObject(urlData) && urlData[variable] !== undefined
                                             ? urlData[variable]
-                                            : ($module.data(variable) !== undefined)
+                                            : ($module.data(variable) !== undefined
                                                 ? $module.data(variable)
-                                                : ($context.data(variable) !== undefined)
+                                                : ($context.data(variable) !== undefined // eslint-disable-line unicorn/no-nested-ternary
                                                     ? $context.data(variable)
-                                                    : urlData[variable]
+                                                    : urlData[variable]))
                                     ;
                                     // optional replacement
                                     if (value !== undefined) {
@@ -536,15 +536,15 @@
                         done: function (response, textStatus, xhr) {
                             var
                                 context            = this,
-                                elapsedTime        = (Date.now() - requestStartTime),
-                                timeLeft           = (settings.loadingDuration - elapsedTime),
-                                translatedResponse = (isFunction(settings.onResponse))
-                                    ? module.is.expectingJSON() && !settings.rawResponse
+                                elapsedTime        = Date.now() - requestStartTime,
+                                timeLeft           = settings.loadingDuration - elapsedTime,
+                                translatedResponse = isFunction(settings.onResponse)
+                                    ? (module.is.expectingJSON() && !settings.rawResponse
                                         ? settings.onResponse.call(context, $.extend(true, {}, response))
-                                        : settings.onResponse.call(context, response)
+                                        : settings.onResponse.call(context, response))
                                     : false
                             ;
-                            timeLeft = (timeLeft > 0)
+                            timeLeft = timeLeft > 0
                                 ? timeLeft
                                 : 0;
                             if (translatedResponse) {
@@ -565,10 +565,10 @@
                         fail: function (xhr, status, httpMessage) {
                             var
                                 context     = this,
-                                elapsedTime = (Date.now() - requestStartTime),
-                                timeLeft    = (settings.loadingDuration - elapsedTime)
+                                elapsedTime = Date.now() - requestStartTime,
+                                timeLeft    = settings.loadingDuration - elapsedTime
                             ;
-                            timeLeft = (timeLeft > 0)
+                            timeLeft = timeLeft > 0
                                 ? timeLeft
                                 : 0;
                             if (timeLeft > 0) {
@@ -748,17 +748,17 @@
                     },
                     responseFromXHR: function (xhr) {
                         return $.isPlainObject(xhr)
-                            ? (module.is.expectingJSON())
+                            ? (module.is.expectingJSON()
                                 ? module.decode.json(xhr.responseText)
-                                : xhr.responseText
+                                : xhr.responseText)
                             : false;
                     },
                     errorFromRequest: function (response, status, httpMessage) {
-                        return ($.isPlainObject(response) && response.error !== undefined)
+                        return $.isPlainObject(response) && response.error !== undefined
                             ? response.error // use json error message
-                            : (settings.error[status] !== undefined) // use server error message
+                            : (settings.error[status] !== undefined // use server error message
                                 ? settings.error[status]
-                                : httpMessage;
+                                : httpMessage);
                     },
                     request: function () {
                         return module.request || false;
@@ -795,7 +795,7 @@
                             return runSettings;
                         }
 
-                        return (runSettings !== undefined)
+                        return runSettings !== undefined
                             ? $.extend(true, {}, runSettings)
                             : $.extend(true, {}, settings);
                     },
@@ -803,7 +803,7 @@
                         var
                             decodedValue   = window.decodeURIComponent(value),
                             encodedValue   = window.encodeURIComponent(value),
-                            alreadyEncoded = (decodedValue !== value)
+                            alreadyEncoded = decodedValue !== value
                         ;
                         if (alreadyEncoded) {
                             module.debug('URL value is already encoded, avoiding double encoding', value);
@@ -836,11 +836,11 @@
                         }
                         if (settings.on == 'auto') {
                             if ($module.is('input')) {
-                                return (element.oninput !== undefined)
+                                return element.oninput !== undefined
                                     ? 'input'
-                                    : (element.onpropertychange !== undefined)
+                                    : (element.onpropertychange !== undefined
                                         ? 'propertychange'
-                                        : 'keyup';
+                                        : 'keyup');
                             }
                             if ($module.is('form')) {
                                 return 'submit';
@@ -1004,7 +1004,7 @@
                         query = query.split(/[ .]/);
                         maxDepth = query.length - 1;
                         $.each(query, function (depth, value) {
-                            var camelCaseValue = (depth != maxDepth)
+                            var camelCaseValue = depth != maxDepth
                                 ? value + query[depth + 1].charAt(0).toUpperCase() + query[depth + 1].slice(1)
                                 : query
                             ;
@@ -1057,7 +1057,7 @@
             }
         });
 
-        return (returnedValue !== undefined)
+        return returnedValue !== undefined
             ? returnedValue
             : this;
     };

package/dist/components/api.min.js

@@ -1,5 +1,5 @@
 /*
- * # Fomantic UI - 2.9.1-beta.23+27384cb
+ * # Fomantic UI - 2.9.1-beta.24+029436f
  * https://github.com/fomantic/Fomantic-UI
  * https://fomantic-ui.com/
  *

package/dist/components/breadcrumb.css

@@ -1,5 +1,5 @@
 /*!
- * # Fomantic-UI 2.9.1-beta.23+27384cb - Breadcrumb
+ * # Fomantic-UI 2.9.1-beta.24+029436f - Breadcrumb
  * https://github.com/fomantic/Fomantic-UI/
  *
  *

package/dist/components/breadcrumb.min.css

@@ -1,5 +1,5 @@
 /*!
- * # Fomantic-UI 2.9.1-beta.23+27384cb - Breadcrumb
+ * # Fomantic-UI 2.9.1-beta.24+029436f - Breadcrumb
  * https://github.com/fomantic/Fomantic-UI/
  *
  *

package/dist/components/button.css

@@ -1,5 +1,5 @@
 /*!
- * # Fomantic-UI 2.9.1-beta.23+27384cb - Button
+ * # Fomantic-UI 2.9.1-beta.24+029436f - Button
  * https://github.com/fomantic/Fomantic-UI/
  *
  *

package/dist/components/button.min.css

@@ -1,5 +1,5 @@
 /*!
- * # Fomantic-UI 2.9.1-beta.23+27384cb - Button
+ * # Fomantic-UI 2.9.1-beta.24+029436f - Button
  * https://github.com/fomantic/Fomantic-UI/
  *
  *

package/dist/components/calendar.css

@@ -1,5 +1,5 @@
 /*!
- * # Fomantic-UI 2.9.1-beta.23+27384cb - Calendar
+ * # Fomantic-UI 2.9.1-beta.24+029436f - Calendar
  * https://github.com/fomantic/Fomantic-UI/
  *
  *