feat(icon): deprecate fixedWidth prop

telus · Sep 19, 2017 · 25ce29d · 25ce29d
1 parent 62bc5cc
commit 25ce29d
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/src/old-components/Icon/Icon.jsx b/src/old-components/Icon/Icon.jsx
@@ -18,6 +18,10 @@ const Icon = ({ glyph, variant, fixedWidth, size, className, children, ...rest }
     deprecate('Icon', '\'disabled\' variant is deprecated.')
   }
 
+  if (fixedWidth) {
+    deprecate('Icon', '\'fixedWidth\' prop is deprecated.')
+  }
+
   const classes = `${styles.icon} ${styles[`icon-core-${glyph}`]}`
     + ` ${variant ? styles[`icon--${variant}`] : ''}`
     + `${fixedWidth ? ` ${styles['icon--fw']}` : ''}`

diff --git a/src/old-components/Icon/__tests__/Icon.spec.jsx b/src/old-components/Icon/__tests__/Icon.spec.jsx
@@ -82,5 +82,13 @@ describe('<Icon />', () => {
       expect(icon).toHaveClassName('icon--disabled')
       expect(deprecate).toHaveBeenCalled()
     })
+
+    it('deprecates fixedWidth prop', () => {
+      jest.clearAllMocks()
+      const icon = doShallow({ fixedWidth: true })
+
+      expect(icon).toHaveClassName('icon--fw')
+      expect(deprecate).toHaveBeenCalled()
+    })
   })
 })