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

package/tasks/install.js

@@ -13,7 +13,7 @@
    * Standard installer runs asking for paths to site files etc
 */
 
-var
+const
     gulp           = require('gulp'),
 
     // node dependencies
@@ -54,7 +54,7 @@ var
 
 // Export install task
 module.exports = function (callback) {
-    var
+    let
         currentConfig = requireDotFile('semantic.json', process.cwd()),
         manager       = install.getPackageManager(),
         rootQuestions = questions.root,
@@ -84,7 +84,7 @@ module.exports = function (callback) {
 
     // run update scripts if semantic.json exists
     if (currentConfig && manager.name === 'NPM') {
-        var
+        let
             updateFolder = path.join(manager.root, currentConfig.base),
             updatePaths  = {
                 config: path.join(manager.root, files.config),
@@ -141,21 +141,21 @@ module.exports = function (callback) {
                     .pipe(gulp.dest(manager.root))
                 ;
 
-                console.info('Update complete! Run "\x1b[92mgulp build\x1b[0m" to rebuild dist/ files.');
+                console.info('Update complete! Run "\u001B[92mgulp build\u001B[0m" to rebuild dist/ files.');
 
                 callback();
 
-                return;
-            } else {
-                console.log('Current version of Fomantic UI already installed');
-                callback();
-
                 return;
             }
-        } else {
-            console.error('Cannot locate files to update at path: ', updatePaths.definition);
-            console.log('Running installer');
+
+            console.log('Current version of Fomantic UI already installed');
+            callback();
+
+            return;
         }
+
+        console.error('Cannot locate files to update at path:', updatePaths.definition);
+        console.log('Running installer');
     }
 
     /* --------------
@@ -226,7 +226,7 @@ module.exports = function (callback) {
              Paths
         --------------- */
 
-        var
+        let
             installPaths = {
                 config: files.config,
                 configFolder: folders.config,
@@ -267,10 +267,10 @@ module.exports = function (callback) {
             installFolder = path.join(manager.root, answers.semanticRoot);
 
             // add install folder to all output paths
-            for (var destination in installPaths) {
-                if (installPaths.hasOwnProperty(destination)) {
+            for (let destination in installPaths) {
+                if (Object.prototype.hasOwnProperty.call(installPaths, destination)) {
                     // config goes in project root, rest in install folder
-                    installPaths[destination] = (destination == 'config' || destination == 'configFolder')
+                    installPaths[destination] = destination == 'config' || destination == 'configFolder'
                         ? path.normalize(path.join(manager.root, installPaths[destination]))
                         : path.normalize(path.join(installFolder, installPaths[destination]));
                 }
@@ -286,7 +286,7 @@ module.exports = function (callback) {
                 console.error('NPM does not have permissions to create folders at your specified path. Adjust your folders permissions and run "npm install" again');
             }
 
-            console.log('Installing to \x1b[92m' + answers.semanticRoot + '\x1b[0m');
+            console.log('Installing to \u001B[92m' + answers.semanticRoot + '\u001B[0m');
 
             console.info('Copying UI definitions');
             wrench.copyDirSyncRecursive(source.definitions, installPaths.definition, settings.wrench.overwrite);
@@ -334,7 +334,7 @@ module.exports = function (callback) {
         --------------- */
 
         gulp.task('create theme.config', function () {
-            var
+            let
                 // determine path to site theme folder from theme config
                 // force CSS path variable to use forward slashes for paths
                 pathToSite   = path.relative(path.resolve(installPaths.themeConfigFolder), path.resolve(installPaths.site)).replace(/\\/g, '/'),
@@ -342,7 +342,7 @@ module.exports = function (callback) {
             ;
 
             // rewrite site variable in theme.less
-            console.info('Adjusting @siteFolder to: ', pathToSite + '/');
+            console.info('Adjusting @siteFolder to:', pathToSite + '/');
 
             if (fs.existsSync(installPaths.themeConfig)) {
                 console.info('Modifying src/theme.config (LESS config)', installPaths.themeConfig);
@@ -352,16 +352,16 @@ module.exports = function (callback) {
                     .pipe(replace(regExp.siteVariable, siteVariable))
                     .pipe(gulp.dest(installPaths.themeConfigFolder))
                 ;
-            } else {
-                console.info('Creating src/theme.config (LESS config)', installPaths.themeConfig);
-
-                return gulp.src(source.themeConfig)
-                    .pipe(plumber())
-                    .pipe(rename({ extname: '' }))
-                    .pipe(replace(regExp.siteVariable, siteVariable))
-                    .pipe(gulp.dest(installPaths.themeConfigFolder))
-                ;
             }
+
+            console.info('Creating src/theme.config (LESS config)', installPaths.themeConfig);
+
+            return gulp.src(source.themeConfig)
+                .pipe(plumber())
+                .pipe(rename({ extname: '' }))
+                .pipe(replace(regExp.siteVariable, siteVariable))
+                .pipe(gulp.dest(installPaths.themeConfigFolder))
+            ;
         });
 
         /* --------------
@@ -369,7 +369,7 @@ module.exports = function (callback) {
         --------------- */
 
         gulp.task('create semantic.json', function () {
-            var
+            let
                 jsonConfig = install.createJSON(answers)
             ;
 
@@ -383,16 +383,16 @@ module.exports = function (callback) {
                     .pipe(jsonEditor(jsonConfig))
                     .pipe(gulp.dest(installPaths.configFolder))
                 ;
-            } else {
-                console.info('Creating config file (semantic.json)', installPaths.config);
-
-                return gulp.src(source.config)
-                    .pipe(plumber())
-                    .pipe(rename({ extname: '' })) // remove .template from ext
-                    .pipe(jsonEditor(jsonConfig, { end_with_newline: true }))
-                    .pipe(gulp.dest(installPaths.configFolder))
-                ;
             }
+
+            console.info('Creating config file (semantic.json)', installPaths.config);
+
+            return gulp.src(source.config)
+                .pipe(plumber())
+                .pipe(rename({ extname: '' })) // remove .template from ext
+                .pipe(jsonEditor(jsonConfig, { end_with_newline: true }))
+                .pipe(gulp.dest(installPaths.configFolder))
+            ;
         });
 
         gulp.series('create theme.config', 'create semantic.json')(callback);
@@ -401,7 +401,7 @@ module.exports = function (callback) {
     gulp.task('clean up install', function (callback) {
         // Completion Message
         if (installFolder && !install.shouldAutoInstall()) {
-            console.log('\n Setup Complete! \n Installing Peer Dependencies. \x1b[0;31mPlease refrain from ctrl + c\x1b[0m... \n After completion navigate to \x1b[92m' + answers.semanticRoot + '\x1b[0m and run "\x1b[92mgulp build\x1b[0m" to build');
+            console.log('\n Setup Complete! \n Installing Peer Dependencies. \u001B[0;31mPlease refrain from ctrl + c\u001B[0m... \n After completion navigate to \u001B[92m' + answers.semanticRoot + '\u001B[0m and run "\u001B[92mgulp build\u001B[0m" to build');
             callback();
         } else {
             console.log('');

package/tasks/rtl/build.js

@@ -2,9 +2,7 @@
  *         Build Task
  *******************************/
 
-var
-    gulp = require('gulp')
-;
+const gulp = require('gulp');
 
 // RTL builds are now handled by the default build process
 module.exports = function (callback) {

package/tasks/rtl/watch.js

@@ -2,9 +2,7 @@
  *          Watch Task
  *******************************/
 
-var
-    gulp = require('gulp')
-;
+const gulp = require('gulp');
 
 // RTL watch are now handled by the default watch process
 module.exports = function (callback) {

package/tasks/version.js

@@ -2,9 +2,7 @@
           Version Task
 *******************************/
 
-var
-    release = require('./config/project/release')
-;
+const release = require('./config/project/release');
 
 module.exports = function (callback) {
     console.log(release.title + ' ' + release.version);

package/tasks/watch.js

@@ -2,7 +2,7 @@
  *          Watch Task
  *******************************/
 
-var
+const
     gulp       = require('gulp'),
 
     // node dependencies

package/test/.eslintrc.js

@@ -9,10 +9,6 @@ module.exports = {
     },
     rules: {
         'jest/no-disabled-tests': 'off',
-
-        // TODO rare errors
-        'jest/no-jasmine-globals': 'off',
-        'jest/no-test-prefixes': 'off',
     },
     globals: {
         jQuery: false,

package/test/meteor/assets.js

@@ -1,4 +1,4 @@
-var assets = [
+let assets = [
     'dist/themes/default/assets/images/flags.png',
 ];
 

package/test/modules/module.spec.js

@@ -1,12 +1,12 @@
 function moduleTests(ui) {
-    var
+    let
         module    = ui.module,
         element   = ui.element,
         singleton = ui.singleton,
         name      = $.fn[module].settings.name,
 
         testValue = 'Test',
-        fixtures  = jasmine.getFixtures(),
+        fixtures  = jasmine.getFixtures(), // eslint-disable-line jest/no-jasmine-globals
 
         originalSettings,
         $modules,
@@ -59,18 +59,18 @@ function moduleTests(ui) {
     -------------------- */
     describe('Module', function () {
         it('allows chaining when no settings returned', function () {
-            var $chain = $modules[module]();
+            let $chain = $modules[module]();
             expect($chain).toExist();
             expect($chain.size()).toBe($modules.size());
         });
 
         it('returns a string when one setting returned', function () {
-            var result = $oneModule[module]('setting', 'name');
+            let result = $oneModule[module]('setting', 'name');
             expect(typeof result).toBe('string');
         });
 
         it('returns an array when multiple settings returned', function () {
-            var result = $modules[module]('setting', 'name');
+            let result = $modules[module]('setting', 'name');
             expect($.isArray(result)).toBeTruthy();
         });
 
@@ -90,7 +90,7 @@ function moduleTests(ui) {
                 name: testValue,
             });
 
-            var retrievedValue = $oneModule[module]('setting', 'name');
+            let retrievedValue = $oneModule[module]('setting', 'name');
             expect(retrievedValue).toBe(testValue);
 
             // reinit
@@ -103,7 +103,7 @@ function moduleTests(ui) {
             $.fn[module].settings.name = testValue;
             $oneModule[module]();
 
-            var retrievedValue = $oneModule[module]('setting', 'name');
+            let retrievedValue = $oneModule[module]('setting', 'name');
             $.fn[module].settings.name = name;
 
             expect(retrievedValue).toBe(testValue);
@@ -114,7 +114,7 @@ function moduleTests(ui) {
                 name: testValue,
             });
 
-            var retrievedValue = $oneModule[module]('setting', 'name');
+            let retrievedValue = $oneModule[module]('setting', 'name');
 
             expect(retrievedValue).toBe(testValue);
         });
@@ -122,7 +122,7 @@ function moduleTests(ui) {
         it('allows settings to be changed during runtime', function () {
             $oneModule[module]();
 
-            var retrievedValue = $oneModule[module]('setting', 'name');
+            let retrievedValue = $oneModule[module]('setting', 'name');
 
             expect(retrievedValue).toBe(name);
         });
@@ -137,8 +137,8 @@ function moduleTests(ui) {
             it('creates settings for all instances', function () {
                 $modules[module]('setting', 'name', testValue);
 
-                var retrievedValue = $oneModule[module]('setting', 'name');
-                var clonedSetting  = $clone[module]('setting', 'name');
+                let retrievedValue = $oneModule[module]('setting', 'name');
+                let clonedSetting  = $clone[module]('setting', 'name');
 
                 expect(retrievedValue).toBe(testValue);
                 expect(clonedSetting).toBe(testValue);
@@ -155,8 +155,8 @@ function moduleTests(ui) {
                 $modules[module]();
                 $oneModule[module]('setting', 'name', testValue);
 
-                var retrievedValue = $oneModule[module]('setting', 'name');
-                var clonedSetting  = $clone[module]('setting', 'name');
+                let retrievedValue = $oneModule[module]('setting', 'name');
+                let clonedSetting  = $clone[module]('setting', 'name');
 
                 expect(retrievedValue).toBe(testValue);
                 expect(clonedSetting).toBe(name);
@@ -169,8 +169,8 @@ function moduleTests(ui) {
                     name: testValue,
                 });
 
-                var retrievedValue = $oneModule[module]('setting', 'name');
-                var clonedSetting  = $clone[module]('setting', 'name');
+                let retrievedValue = $oneModule[module]('setting', 'name');
+                let clonedSetting  = $clone[module]('setting', 'name');
 
                 expect(retrievedValue).toBe(testValue);
                 expect(clonedSetting).toBe(name);

package/test/modules/popup.spec.js

@@ -1,4 +1,4 @@
-xdescribe('UI Popup', function () {
+describe.skip('UI Popup', function () {
     moduleTests({
         module: 'popup',
         element: 'i.icon',

package/test/modules/search.spec.js

@@ -1,4 +1,4 @@
-xdescribe('UI Search', function () {
+describe.skip('UI Search', function () {
     moduleTests({
         module: 'search',
         element: '.ui.search',

package/test/modules/tab.spec.js

@@ -1,4 +1,4 @@
-xdescribe('UI Tab', function () {
+describe.skip('UI Tab', function () {
     moduleTests({
         module: 'tab',
         element: '.ui.menu .item',

package/test/modules/transition.spec.js

@@ -1,4 +1,4 @@
-xdescribe('UI Transition', function () {
+describe.skip('UI Transition', function () {
     moduleTests({
         module: 'transition',
         element: '.ui.image',