style(eslint): enable spaced-comment rule (#11585)

reformat .eslintrc.json

Author: Splaktar

.eslintrc.json

@@ -16,7 +16,7 @@ module.exports = function(config) {
     // Maximum 10 browsers - SauceLabs limit
     // Those pre-configured browsers will always run in the CI Release Mode to confirm, that all
     // previous jobs have passed.
-    browsers: ['SL_CHROME'],   //, 'SL_FIREFOX', 'SL_IE11'],
+    browsers: ['SL_CHROME'],   // , 'SL_FIREFOX', 'SL_IE11'],
     customLaunchers: sauceBrowsers,
 
     captureTimeout: 180 * 1000,

config/karma-sauce.conf.js

@@ -6,7 +6,7 @@ module.exports = function(config) {
 
     // To enable use of `gulp karma-watch`,
     // don't use the dist/angular-material.js.
-    //'dist/angular-material.js',   // Un-minified source
+    // 'dist/angular-material.js',   // Un-minified source
 
 
     // Test utilities, source, and specifications.

config/karma.conf.js

@@ -28,7 +28,7 @@ function buildScanner(pattern) {
 
     if (depsMatch) {
       dependencies = depsMatch.split(/\s*,\s*/).map(function(dep) {
-        dep = dep.trim().slice(1, -1); //remove quotes
+        dep = dep.trim().slice(1, -1); // remove quotes
         return dep;
       });
     }

config/ngModuleData.js

@@ -614,7 +614,7 @@ function($scope, COMPONENTS, BUILDCONFIG, $mdSidenav, $timeout, $mdDialog, menu,
   $rootScope.$on('$locationChangeSuccess', openPage);
   $scope.focusMainContent = focusMainContent;
 
-  //-- Define a fake model for the related page selector
+  // Define a fake model for the related page selector
   Object.defineProperty($rootScope, "relatedPage", {
     get: function () { return null; },
     set: angular.noop,

docs/app/js/app.js

@@ -68,7 +68,7 @@ function($q, $compile, $timeout) {
         return file.contentsPromise;
       }))
       .then(function(styles) {
-        styles = styles.join('\n'); //join styles as one string
+        styles = styles.join('\n'); // join styles as one string
 
         var styleElement = angular.element('<style>' + styles + '</style>');
         document.body.appendChild(styleElement[0]);

docs/app/js/demoInclude.js

@@ -4,7 +4,7 @@ angular.module('docsApp')
     restrict: 'E',
     compile: function(element, attr) {
       var code;
-      //No attribute? code is the content
+      // No attribute? code is the content
       if (!attr.code) {
         code = element.html();
         element.empty();

docs/app/js/highlight-angular.js

@@ -80,7 +80,7 @@ module.exports = function componentsGenerateProcessor() {
         });
 
       })
-      .filter() //remove null items
+      .filter() // remove null items
       .value();
 
     const EXPOSED_CORE_SERVICES = '$mdMedia';

docs/config/processors/componentsData.js

@@ -7,10 +7,8 @@ const kDisallowedFunctions = [
   // Allow xit/xdescribe --- disabling tests is okay
   'fit',
   'iit',
-  //'xit',
   'fdescribe',
   'ddescribe',
-  //'xdescribe',
   'describe.only',
   'it.only'
 ];

gulp/tasks/ddescribe-iit.js

@@ -1,10 +1,10 @@
 const gulp = require('gulp');
 const fs = require('fs');
 
-//-- include docs gulpfile (should eventually be factored out)
+// include docs gulpfile (should eventually be factored out)
 require('./docs/gulpfile');
 
-//-- read in all files from gulp/tasks and create tasks for them
+// read in all files from gulp/tasks and create tasks for them
 fs.readdirSync('./gulp/tasks')
     .filter(function (filename) {
       return filename.match(/\.js$/i);

gulpfile.js

@@ -56,7 +56,7 @@
     log('If you would like to cancel this release, please run "./abort"');
   }
 
-  //-- utility methods
+  // utility methods
 
   /** confirms that you will be able to perform the release before attempting */
   function validate () {
@@ -146,7 +146,7 @@
           }
         } else {
           version[ type ]++;
-          //-- reset any version numbers lower than the one changed
+          // reset any version numbers lower than the one changed
           switch (type) {
             case 'minor': version.patch = 0;
             case 'patch': version.rc = 0;
@@ -216,13 +216,13 @@
     const options = { cwd: './bower-material' };
     const bower   = require(options.cwd + '/bower.json'),
           pkg     = require(options.cwd + '/package.json');
-    //-- update versions in config files
+    // update versions in config files
     bower.version = pkg.version = newVersion;
     fs.writeFileSync(options.cwd + '/package.json', JSON.stringify(pkg, null, 2));
     fs.writeFileSync(options.cwd + '/bower.json', JSON.stringify(bower, null, 2));
     done();
     start('Building bower files...');
-    //-- build files for bower
+    // build files for bower
     exec([
       'rm -rf dist',
       'gulp build',
@@ -232,15 +232,15 @@
      ]);
     done();
     start('Copy files into bower repo...');
-    //-- copy files over to bower repo
+    // copy files over to bower repo
     exec([
            'cp -Rf ../dist/* ./',
            'git add -A',
            `git commit -m "release: version ${newVersion}"`,
            'rm -rf ../dist'
          ], options);
     done();
-    //-- add steps to push script
+    // add steps to push script
     pushCmds.push(
       comment('push to bower (master and tag) and publish to npm'),
       'cd ' + options.cwd,
@@ -262,14 +262,14 @@
     const options = { cwd: './code.material.angularjs.org' };
     writeDocsJson();
 
-    //-- build files for bower
+    // build files for bower
     exec([
         'rm -rf dist',
         'gulp docs'
     ]);
     replaceFilePaths();
 
-    //-- copy files over to site repo
+    // copy files over to site repo
     exec([
         `cp -Rf ../dist/docs ${newVersion}`,
         'rm -rf latest && cp -Rf ../dist/docs latest',
@@ -280,12 +280,12 @@
     replaceBaseHref(newVersion);
     replaceBaseHref('latest');
 
-    //-- update firebase.json file
+    // update firebase.json file
     updateFirebaseJson();
     exec([ 'git commit --amend --no-edit -a' ], options);
     done();
 
-    //-- add steps to push script
+    // add steps to push script
     pushCmds.push(
         comment('push the site'),
         'cd ' + options.cwd,
@@ -323,15 +323,15 @@
       const config = require(options.cwd + '/docs.json');
       config.versions.unshift(newVersion);
 
-      //-- only set to default if not a release candidate
+      // only set to default if not a release candidate
       config.latest = newVersion;
       fs.writeFileSync(options.cwd + '/docs.json', JSON.stringify(config, null, 2));
     }
   }
 
   /** replaces localhost file paths with public URLs */
   function replaceFilePaths () {
-    //-- handle docs.js
+    // handle docs.js
     const filePath = path.join(__dirname, '/dist/docs/docs.js');
     const file = fs.readFileSync(filePath);
     const contents = file.toString()
@@ -342,7 +342,7 @@
 
   /** replaces base href in index.html for new version as well as latest */
   function replaceBaseHref (folder) {
-    //-- handle index.html
+    // handle index.html
     const filePath = path.join(__dirname, '/code.material.angularjs.org/', folder, '/index.html');
     const file = fs.readFileSync(filePath);
     const contents = file.toString().replace(/base href="\//g, 'base href="/' + folder + '/');

release.js

@@ -69,7 +69,7 @@ function MdAutocompleteCtrl ($scope, $element, $mdUtil, $mdConstant, $mdTheming,
 
   return init();
 
-  //-- initialization methods
+  // initialization methods
 
   /**
    * Initialize the controller, setup watchers, gather elements
@@ -325,7 +325,7 @@ function MdAutocompleteCtrl ($scope, $element, $mdUtil, $mdConstant, $mdTheming,
     return obj;
   }
 
-  //-- event/change handlers
+  // event/change handlers
 
   /**
    * Handles changes to the `hidden` property.
@@ -610,7 +610,7 @@ function MdAutocompleteCtrl ($scope, $element, $mdUtil, $mdConstant, $mdTheming,
     }
   }
 
-  //-- getters
+  // getters
 
   /**
    * Returns the minimum length needed to display the dropdown.
@@ -780,7 +780,7 @@ function MdAutocompleteCtrl ($scope, $element, $mdUtil, $mdConstant, $mdTheming,
     return ($scope.searchText || '').length >= getMinLength();
   }
 
-  //-- actions
+  // actions
 
   /**
    * Defines a public property with a handler and a default value.
@@ -804,7 +804,7 @@ function MdAutocompleteCtrl ($scope, $element, $mdUtil, $mdConstant, $mdTheming,
    * @param {number} index to select
    */
   function select (index) {
-    //-- force form to update state for validation
+    // force form to update state for validation
     $mdUtil.nextTick(function () {
       getDisplayValue(ctrl.matches[ index ]).then(function (val) {
         var ngModel = elements.$.input.controller('ngModel');

src/components/autocomplete/js/autocompleteController.js

@@ -133,7 +133,7 @@ function MdButtonDirective($mdButtonInkRipple, $mdTheming, $mdAria, $mdInteracti
     if (isAnchor(attr)) {
       return '<a class="md-button" ng-transclude></a>';
     } else {
-      //If buttons don't have type="button", they will submit forms automatically.
+      // If buttons don't have type="button", they will submit forms automatically.
       var btnType = (typeof attr.type === 'undefined') ? 'button' : attr.type;
       return '<button class="md-button" type="' + btnType + '" ng-transclude></button>';
     }

src/components/button/button.js

@@ -112,7 +112,7 @@
     };
   }
 
-  /*** Initialization ***/
+  /** Initialization **/
 
   /**
    * Initialize the controller by saving a reference to the calendar and

src/components/datepicker/js/calendarMonth.js

@@ -61,7 +61,7 @@ function MdDialogDirective($$rAF, $mdTheming, $mdDialog) {
         if (content) {
           images = content.getElementsByTagName('img');
           addOverflowClass();
-          //-- delayed image loading may impact scroll height, check after images are loaded
+          // delayed image loading may impact scroll height, check after images are loaded
           angular.element(images).on('load', addOverflowClass);
         }
 

src/components/dialog/dialog.js

@@ -1428,9 +1428,9 @@ describe('$mdDialog', function() {
         template: '<md-dialog class="two">',
         parent: parent
       });
-      //First run is for the old dialog being hidden.
+      // First run is for the old dialog being hidden.
       runAnimation();
-      //Second run is for the new dialog being shown.
+      // Second run is for the new dialog being shown.
       runAnimation();
       $mdDialog.hide();
       runAnimation();

src/components/dialog/dialog.spec.js

@@ -180,7 +180,7 @@
 
       // TODO: On desktop, we should be able to reset the indexes so you cannot tab through, but
       // this breaks accessibility, especially on mobile, since you have no arrow keys to press
-      //resetActionTabIndexes();
+      // resetActionTabIndexes();
     }
 
     function disableKeyboard() {

src/components/fabSpeedDial/fabController.js

@@ -2,7 +2,7 @@ angular
   .module('material.components.menu')
   .provider('$mdMenu', MenuProvider);
 
-/*
+/**
  * Interim element provider for the menu.
  * Handles behavior for a menu while it is open, including:
  *    - handling animating the menu opening/closing
@@ -180,7 +180,7 @@ function MenuProvider($$interimElementProvider) {
         angular.extend(opts, {
           alreadyOpen: false,
           isRemoved: false,
-          target: angular.element(opts.target), //make sure it's not a naked dom node
+          target: angular.element(opts.target), // make sure it's not a naked DOM node
           parent: angular.element(opts.parent),
           menuContentEl: angular.element(element[0].querySelector('md-menu-content'))
         });

src/components/menu/js/menuServiceProvider.js

@@ -149,9 +149,6 @@ function MdProgressLinearDirective($mdTheming, $mdUtil, $log) {
         var hasValue = angular.isDefined(attr.value);
         var mode = hasValue ? MODE_DETERMINATE : MODE_INDETERMINATE;
         var info = "Auto-adding the missing md-mode='{0}' to the ProgressLinear element";
-
-        //$log.debug( $mdUtil.supplant(info, [mode]) );
-
         element.attr("md-mode", mode);
         attr.mdMode = mode;
       }

src/components/progressLinear/progress-linear.js

@@ -1406,7 +1406,7 @@ function SelectProvider($$interimElementProvider) {
 
         angular.extend(options, {
           isRemoved: false,
-          target: angular.element(options.target), //make sure it's not a naked dom node
+          target: angular.element(options.target), // make sure it's not a naked DOM node
           parent: angular.element(options.parent),
           selectEl: selectEl,
           contentEl: element.find('md-content'),

src/components/select/select.js

@@ -217,7 +217,7 @@ describe('<md-select>', function() {
       $material.flushInterimElement();
 
       // FIXME- does not work with minified, jquery
-      //expect($document[0].activeElement).toBe(select[0]);
+      // expect($document[0].activeElement).toBe(select[0]);
 
       // Clean up the DOM after the test.
       $document[0].body.removeChild(select[0]);

src/components/select/select.spec.js

@@ -119,7 +119,7 @@ function MdSticky($mdConstant, $$rAF, $mdUtil, $compile) {
     var self;
     return self = {
       prev: null,
-      current: null, //the currently stickied item
+      current: null, // the currently stickied item
       next: null,
       items: [],
       add: add,
@@ -194,8 +194,9 @@ function MdSticky($mdConstant, $$rAF, $mdUtil, $compile) {
       while (current && current !== contentEl[0]) {
         item.top += current.offsetTop;
         item.left += current.offsetLeft;
-        if (current.offsetParent){
-          item.right += current.offsetParent.offsetWidth - current.offsetWidth - current.offsetLeft; //Compute offsetRight
+        if (current.offsetParent) {
+          // Compute offsetRight
+          item.right += current.offsetParent.offsetWidth - current.offsetWidth - current.offsetLeft;
         }
         current = current.offsetParent;
       }

src/components/sticky/sticky.js

@@ -133,7 +133,7 @@ function MdSwitch(mdCheckboxDirective, $mdUtil, $mdConstant, $parse, $$rAF, $mdG
 
         var percent = ev.pointer.distanceX / drag.width;
 
-        //if checked, start from right. else, start from left
+        // if checked, start from right. else, start from left
         var translate = ngModel.$viewValue ?  1 + percent : percent;
         // Make sure the switch stays inside its bounds, 0-1%
         translate = Math.max(0, Math.min(1, translate));

src/components/switch/switch.js

@@ -636,10 +636,10 @@ function MdTabsController ($scope, $element, $window, $mdConstant, $mdTabInkRipp
     var width = 0;
 
     angular.forEach(tabs, function (tab) {
-      //-- Uses the larger value between `getBoundingClientRect().width` and `offsetWidth`.  This
-      //   prevents `offsetWidth` value from being rounded down and causing wrapping issues, but
-      //   also handles scenarios where `getBoundingClientRect()` is inaccurate (ie. tabs inside
-      //   of a dialog)
+      // Uses the larger value between `getBoundingClientRect().width` and `offsetWidth`.  This
+      // prevents `offsetWidth` value from being rounded down and causing wrapping issues, but
+      // also handles scenarios where `getBoundingClientRect()` is inaccurate (ie. tabs inside
+      // of a dialog).
       width += Math.max(tab.offsetWidth, tab.getBoundingClientRect().width);
     });
 

src/components/tabs/js/tabsController.js

@@ -382,7 +382,7 @@ describe('<md-tabs>', function () {
       expect(tabContent.attr('role')).toBe('tabpanel');
       expect(tabContent.attr('aria-labelledby')).toBe(tabItem.attr('id'));
 
-      //Unique ids check
+      // Unique ids check
       expect(tabContent.attr('id')).not.toEqual(tabItem.attr('id'));
     });
 

src/components/tabs/tabs.spec.js

@@ -470,7 +470,7 @@ function MdToastProvider($$interimElementProvider) {
       options.element = element;
 
       options.onSwipe = function(ev, gesture) {
-        //Add the relevant swipe class to the element so it can animate correctly
+        // Add the relevant swipe class to the element so it can animate correctly
         var swipe = ev.type.replace('$md.','');
         var direction = swipe.replace('swipe', '');