feat(paragraph): Rename the base class as noSpacing to better describ…

…e its purpose Also remove the alignment from it because we have an explicit alignment prop now.
telus · Aug 24, 2017 · 8782b2d · 8782b2d
1 parent 14bd906
commit 8782b2d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/src/components/Typography/Paragraph/Paragraph.jsx b/src/components/Typography/Paragraph/Paragraph.jsx
@@ -11,7 +11,7 @@ const Paragraph = ({ bold, size, align, invert, children, ...rest }, context) =>
   const paragraphColor = invert ? textStyles.colorInverted : textStyles.color
 
   const classes = classnames(
-    styles.base,
+    styles.noSpacing,
     context.inheritColor ? styles.inheritColor : paragraphColor,
     textStyles[size],
     bold ? textStyles.boldFont : textStyles[`${size}Font`],

diff --git a/src/components/Typography/Paragraph/Paragraph.modules.scss b/src/components/Typography/Paragraph/Paragraph.modules.scss
@@ -1,8 +1,6 @@
-.base {
+.noSpacing {
   padding: 0;
   margin: 0;
-
-  text-align: left;
 }
 
 .inheritColor {

diff --git a/src/components/Typography/Paragraph/__tests__/__snapshots__/Paragraph.spec.jsx.snap b/src/components/Typography/Paragraph/__tests__/__snapshots__/Paragraph.spec.jsx.snap
@@ -2,7 +2,7 @@
 
 exports[`Paragraph renders 1`] = `
 <p
-  className="base color medium mediumFont leftAlign"
+  className="noSpacing color medium mediumFont leftAlign"
 >
   Some content
 </p>