fix(layout): resolve minify bug with flex css

*  fix minimize bug where `flex: 1 1 0%` incorrectly becomes `flex: 1 1 0`;
*  do not support md-css-only; JS is always required
*  do not remove layout attributes

Author: ThomasBurleson

src/core/services/layout/layout.js

@@ -54,13 +54,8 @@
            * Do NOT remove the original layout Attribute selectors
            * after translation injection; or the media breakpoints will not fire
            */
-          removeAttributes : true,
+          removeAttributes : true
 
-          /**
-           * Special internal flag used to optimize
-           * noop(s) for the directive postLinks below
-           */
-          enablePostLinks : undefined
         };
       })
 
@@ -173,11 +168,9 @@
         return {
             restrict : 'A',
             compile: function(element, attr) {
-              var injected = injectLayoutSpecifier(element, attr);
-              var enabled = isPostLinkEnabled($document[0]);
-
               // Use for postLink to account for transforms after ng-transclude.
-              if ( !injected && enabled ) {
+
+              if ( !injectLayoutSpecifier(element, attr) ) {
                 attributeValueToClass(null, element, attr);
                 return attributeValueToClass;
               }
@@ -289,7 +282,6 @@
         return {
           restrict : 'A',
           compile: function(element, attrs) {
-            if ( !isPostLinkEnabled($document[0]) ) return angular.noop;
 
             attributeToClass(null, element);
 
@@ -327,33 +319,4 @@
 
     }
 
-    /**
-     * Scan the body element. If it has a class 'md-css-only', then do NOT
-     * postLink process the directives for Attribute selectors.
-     * (recall that postlink injects Class selectors based on attribute selector settings)
-     *
-     * Instead the Layout CSS for Attributes is used:
-     * e.g
-     *       .md-css-only [layout=row] {
-     *          flex-direction: row;
-     *          -webkit-flex-direction: row;
-     *       }
-     *
-     * Note: this means that 'md-css-only' will not work for IE (due to performance issues)
-     * In these cases, the Layout translators (directives) should be enabled and the
-     * `angular-material.[min.]js` must be loaded.
-     */
-    function isPostLinkEnabled(document) {
-      var enablePostLinks = $$mdLayout.enablePostLinks;
-
-      // Perform a read-once (1x) check for the `md-css-only` class on the BODY
-
-      if ( angular.isUndefined(enablePostLinks) ) {
-        var body = document && document.body;
-        if (body) enablePostLinks = !body.classList.contains('md-css-only');
-      }
-
-      return $$mdLayout.enablePostLinks = enablePostLinks;
-    }
-
 })();

src/core/services/layout/layout.scss

@@ -182,7 +182,7 @@
     box-sizing: border-box;
   }
 
-  .#{$flexName}             { flex: 1 1 0%;  }
+  .#{$flexName}             { flex: 1;  }   // === 1 1 0%
   .#{$flexName}-grow        { flex: 1 1 100%;  }
   .#{$flexName}-initial     { flex: 0 1 auto;  }
   .#{$flexName}-auto        { flex: 1 1 auto;  }

src/core/services/layout/layout.spec.js

@@ -47,20 +47,6 @@ describe('layout directives', function() {
         var element = $compile('<div ' + attribute + '>Layout</div>')($rootScope.$new());
         expect(element.hasClass(expectedClass)).toBe(true);
       }));
-
-      it('should not add the class ' + expectedClass + ' if the body class has "md-css-only" ' + attribute, inject(function($$mdLayout, $compile, $rootScope, $document) {
-        $$mdLayout.enablePostLinks = undefined;
-
-        var body = angular.element($document[0].body);
-        body.addClass('md-css-only');
-
-        var element = $compile('<div ' + attribute + '>Layout</div>')($rootScope.$new());
-        expect(element.hasClass(expectedClass)).toBe(false);
-
-        body.removeClass('md-css-only');
-        $$mdLayout.enablePostLinks = undefined;
-
-      }));
     }
 
     /** Test directives with 'sm', 'gt-sm', 'md', 'gt-md', 'lg', and 'gt-lg' suffixes */
@@ -146,7 +132,6 @@ describe('layout directives', function() {
       expect(element.hasClass('flex-gt-md-32')).toBe(false);
       expect(element.hasClass('flex-gt-md-fishCheeks')).toBe(true);
 
-      expect(element.attr('flex-gt-md')).toBeFalsy();
     }));
 
   })