feat: allow to add step attribute on every field

Author: Kamil Kisiela

src/types/input/input.js

@@ -15,6 +15,11 @@ export default (formlyConfigProvider) => {
         mdMaxlength: {
           bound: 'md-maxlength'
         },
+        // XXX angular-formly#8042d2a so we want to keep it compatible
+        // with angular-formly releases before that commit
+        step: {
+          attribute: 'step'
+        },
         disabled: {
           bound: 'ng-disabled'
         },
@@ -38,15 +43,4 @@ export default (formlyConfigProvider) => {
       };
     }
   });
-
-  // add only step attribute because min and max are both built-in
-  formlyConfigProvider.extras.fieldTransform.push((fields) => {
-    return ngModelAttrsTransformer(fields, (field) => (
-      field.type === 'input' &&
-      field.templateOptions &&
-      field.templateOptions.type === 'number'
-    ), 'step', {
-      attribute: 'step'
-    });
-  });
 };

tests/types/input-spec.js

@@ -118,15 +118,17 @@ describe('formlyMaterial - input type', () => {
 
   describe('number type specific', () => {
     describe('step attribute', () => {
-      it('should not be available on non number type', () => {
+      // XXX BE instead of NOT BE because `step` attr is now fully supported
+      // angular-formly#8042d2a
+      it('should BE available on non number type', () => {
         compile({
           templateOptions: {
             type: 'text',
             step: 2
           }
         });
 
-        expect(element.attr('step')).toBeUndefined();
+        expect(parseInt(element.attr('step'))).toBe(field.templateOptions.step);
       });
 
       it('should be available on number type', () => {