Merge pull request #52 from 4alltecnologia/fix/header-optional-backlink

🎨 fix header to support absence of backlink
4alltecnologia · Dec 30, 2019 · 36bafd7 · 36bafd7
2 parents 25453a0 + 3eb3393
commit 36bafd7
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 10 deletions.
diff --git a/src/components/Header/index.js b/src/components/Header/index.js
@@ -31,7 +31,7 @@ function Header({ breadcrumbLinks, title, backLink, containerStyles }) {
             </styles.SmallHeader>
             <styles.Title>{title}</styles.Title>
           </styles.LeftHeader>
-          <styles.RightHeader>{renderBackLink(backLink)}</styles.RightHeader>
+          {backLink && <styles.RightHeader>{renderBackLink(backLink)}</styles.RightHeader>}
         </styles.InnerHeader>
       </styles.Container>
     </styles.Header>
@@ -40,15 +40,8 @@ function Header({ breadcrumbLinks, title, backLink, containerStyles }) {
 
 Header.defaultProps = {
   title: '',
-  breadcrumbLinks: [
-    { external: true, link: '/', text: 'Gestão de MKTP' },
-    { link: '/', text: 'Importação' },
-  ],
-  backLink: {
-    text: '',
-    external: false,
-    link: '',
-  },
+  breadcrumbLinks: [{ external: true, link: '/', text: 'Home' }, { link: '/', text: 'Listagem' }],
+  backLink: null,
   containerStyles: {
     width: '100%',
     margin: '0 auto',

diff --git a/src/stories/header.js b/src/stories/header.js
@@ -14,6 +14,12 @@ storiesOf('Header', module)
       backLink={{ text: 'Voltar para Itens', external: true, link: '/' }}
     />
   ))
+  .add('Without backlink', () => (
+    <Header
+      breadcrumbLinks={[{ external: true, link: '/', text: 'Home' }, { link: '/', text: 'Itens' }]}
+      title="Item1"
+    />
+  ))
   .add('Custom style', () => (
     <Header
       breadcrumbLinks={[{ external: true, link: '/', text: 'Home' }, { link: '/', text: 'Itens' }]}