New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Break Footer into smaller pieces using composition
#54
Conversation
7ab2751
to
6e907f6
Compare
2def40d
to
62c3f9f
Compare
@@ -1,12 +1,12 @@ | |||
import React, { Component } from 'react' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just moved this file and fix lint
@@ -2,8 +2,8 @@ import PropTypes from 'prop-types' | |||
import React from 'react' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just moved this file and fixed lint
@@ -2,8 +2,8 @@ import React from 'react' | |||
import PropTypes from 'prop-types' | |||
|
|||
import footerList from './footerList' | |||
import withImage from './withImage' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just moved this file and fixed lint
@@ -3,8 +3,8 @@ import React, { Component, Fragment } from 'react' | |||
|
|||
import Accordion from './Accordion' | |||
import FooterLinkList, { FooterLinkItem } from './FooterLinkList' | |||
import { objectLikeLinkArray } from './../propTypes' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just moved this file and fixed lint
@@ -3,11 +3,11 @@ import React, { Component } from 'react' | |||
import classNames from 'classnames' | |||
import { Container } from 'vtex.store-components' | |||
|
|||
import FooterBadgeList from './components/FooterBadgeList' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just moved this file and fixed lint
@@ -1,11 +1,11 @@ | |||
import PropTypes from 'prop-types' | |||
import React, { Component } from 'react' | |||
import { Collapse } from 'react-collapse' | |||
import { injectIntl, intlShape, FormattedMessage } from 'react-intl' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just moved this file and fixed lint
"accepted-payment-methods", | ||
"logo", | ||
"newsletter", | ||
"unstable--menu", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"unstable--menu", | |
"menu", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't right now, since this is not published yet:
vtex-apps/menu#35
error: App build failed with message: I couldn't find an interface "menu". You need to declare it or depend on an app that declares it.
What is the purpose of this pull request?
Fix #46
Break Footer into reusable pieces to make it possible to build a more complex layout.
Components:
What problem is this solving?
Today Footer is a single component with a fixed layout. This makes it pretty strict in layout options.
How should this be manually tested?
Store:
https://breno--storecomponents.myvtexdev.com/
CMS:
https://breno--storecomponents.myvtexdev.com/admin/cms/storefront
Screenshots or example usage
Types of changes