change(copyright): removed color variant

- the copyright will be styled according to the parent component, there is no need to give it a different color from the copyright
eea · Dec 5, 2022 · 747b8a2 · 747b8a2
1 parent 7b6c0ae
commit 747b8a2
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 39 deletions.
diff --git a/src/ui/Banner/Banner.stories.jsx b/src/ui/Banner/Banner.stories.jsx
@@ -138,10 +138,7 @@ const Template = (args) => (
           </>
         </Banner.Metadata>
       )}
-      <Copyright
-        copyrightPosition={args.copyrightPosition}
-        colorVariant={args.copyrightVariant}
-      >
+      <Copyright copyrightPosition={args.copyrightPosition}>
         <Copyright.Icon>
           <Icon className={args.copyrightIcon} />
         </Copyright.Icon>

diff --git a/src/ui/Copyright/Copyright.jsx b/src/ui/Copyright/Copyright.jsx
@@ -3,26 +3,12 @@ import PropTypes from 'prop-types';
 
 Copyright.propTypes = {
   copyrightPosition: PropTypes.oneOf(['left', 'right']),
-  colorVariant: PropTypes.oneOf([
-    'default',
-    'primary',
-    'secondary',
-    'tertiary',
-  ]),
 };
 
 function Copyright({ children, ...rest }) {
   return (
     <div className={`eea copyright ${rest.copyrightPosition}`}>
-      <div
-        className={`${
-          rest.colorVariant === 'default'
-            ? 'wrapper'
-            : 'wrapper color-fg-' + rest.colorVariant
-        }`}
-      >
-        {children}
-      </div>
+      <div className={'wrapper'}>{children}</div>
     </div>
   );
 }

diff --git a/src/ui/Copyright/Copyright.stories.jsx b/src/ui/Copyright/Copyright.stories.jsx
@@ -6,16 +6,6 @@ export default {
   title: 'Components/Copyright',
   component: Copyright,
   argTypes: {
-    colorVariant: {
-      name: 'Variation',
-      defaultValue: 'default',
-      options: ['default', 'primary', 'secondary', 'tertiary'],
-      control: { type: 'select' },
-      table: {
-        defaultValue: { summary: 'default' },
-        type: { summary: 'string' },
-      },
-    },
     copyrightPosition: {
       name: 'Position',
       control: {
@@ -32,11 +22,8 @@ export default {
 
 export const Default = (args) => (
   <Container>
-    <Segment className={args.colorVariant === 'default' ? 'inverted' : ''}>
-      <Copyright
-        copyrightPosition={args.copyrightPosition}
-        colorVariant={args.colorVariant}
-      >
+    <Segment className={'inverted'}>
+      <Copyright copyrightPosition={args.copyrightPosition}>
         <Copyright.Icon>
           <Icon className={args.icon} />
         </Copyright.Icon>
@@ -47,7 +34,6 @@ export const Default = (args) => (
 );
 
 Default.args = {
-  colorVariant: 'default',
   copyrightPosition: 'left',
   icon: 'ri-copyright-line',
   text: 'Image copyright: Velit fusce sed sem ut.',

diff --git a/src/ui/Hero/Hero.jsx b/src/ui/Hero/Hero.jsx
@@ -49,10 +49,7 @@ function Hero({
             {children}
 
             {has_copyright && (
-              <Copyright
-                copyrightPosition={copyrightPosition}
-                colorVariant={copyrightVariant}
-              >
+              <Copyright copyrightPosition={copyrightPosition}>
                 <Copyright.Icon>
                   <Icon className={copyrightIcon} />
                 </Copyright.Icon>