Skip to content
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

#167454777 create image with article row component #46

Merged
merged 1 commit into from
Jul 25, 2019

Conversation

Sojisoyoye
Copy link
Contributor

What does this PR do?

This PR sets up image with article component

Description of task to be completed?

  • setup container for the atoms
  • include image atom
  • include text and title atom

How should this be manually tested?

$ git clone https://github.com/andela/king-kong-ah-frontend.git
$ npm install
$ npm test

Any background context you want to provide?

You would need npm to run this application

What are the relevant pivotal tracker stories?

PT Story link - #167454777

Screenshots (if appropriate)

image

@akinyeleolat akinyeleolat temporarily deployed to king-kong-ah-frontend-st-pr-46 July 24, 2019 12:35 Inactive
@Sojisoyoye Sojisoyoye force-pushed the ft-image-text-molecule-167454777 branch from b3f542f to 270f156 Compare July 24, 2019 18:19
@akinyeleolat akinyeleolat temporarily deployed to king-kong-ah-frontend-st-pr-46 July 24, 2019 18:19 Inactive
@Sojisoyoye Sojisoyoye force-pushed the ft-image-text-molecule-167454777 branch from 270f156 to f32e954 Compare July 25, 2019 08:00
@akinyeleolat akinyeleolat temporarily deployed to king-kong-ah-frontend-st-pr-46 July 25, 2019 08:00 Inactive
@Sojisoyoye Sojisoyoye added Ready for review This means a PR is ready for review by the teammates and removed Work in Progress labels Jul 25, 2019
Copy link
Contributor

@mosinmiloluwa01 mosinmiloluwa01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job so far, kindly spare some minutes to view the feedback

src/components/UI/atoms/Text/Text.jsx Outdated Show resolved Hide resolved
src/components/UI/atoms/Text/Text.jsx Outdated Show resolved Hide resolved
src/components/pages/Home/HomePage.jsx Outdated Show resolved Hide resolved
@Sojisoyoye Sojisoyoye added Work in Progress and removed Ready for review This means a PR is ready for review by the teammates labels Jul 25, 2019
@Sojisoyoye Sojisoyoye force-pushed the ft-image-text-molecule-167454777 branch from f32e954 to ecb948d Compare July 25, 2019 14:16
@akinyeleolat akinyeleolat temporarily deployed to king-kong-ah-frontend-st-pr-46 July 25, 2019 14:17 Inactive
@Sojisoyoye Sojisoyoye force-pushed the ft-image-text-molecule-167454777 branch from ecb948d to 1df251d Compare July 25, 2019 14:20
@akinyeleolat akinyeleolat temporarily deployed to king-kong-ah-frontend-st-pr-46 July 25, 2019 14:21 Inactive
@Sojisoyoye Sojisoyoye added Ready for review This means a PR is ready for review by the teammates and removed Work in Progress labels Jul 25, 2019
@Sojisoyoye Sojisoyoye force-pushed the ft-image-text-molecule-167454777 branch from 1df251d to 73fe5db Compare July 25, 2019 15:59
@akinyeleolat akinyeleolat temporarily deployed to king-kong-ah-frontend-st-pr-46 July 25, 2019 15:59 Inactive
theme,
}) => `
display: ${display};
text-align: ${textAlign};
color: ${theme.textColors[color]};
padding-bottom: ${theme.spacing[paddingBottom]};
font-size: ${theme.fontSizes[fontSize]};
font-family: ${(content === 'true' && 'Inknut Antiqua') || 'roboto'};

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you try ${content && 'Inknut Antiqua' || 'roboto'};

@benfluleck benfluleck added Work in Progress and removed Ready for review This means a PR is ready for review by the teammates labels Jul 25, 2019
@Sojisoyoye Sojisoyoye force-pushed the ft-image-text-molecule-167454777 branch from 73fe5db to 4d4b0ad Compare July 25, 2019 16:39
@akinyeleolat akinyeleolat temporarily deployed to king-kong-ah-frontend-st-pr-46 July 25, 2019 16:39 Inactive
- setup container for the atoms
- include image atom
- include text and title atom
- write test for component

[Finishes #167454777]
@Sojisoyoye Sojisoyoye force-pushed the ft-image-text-molecule-167454777 branch from 4d4b0ad to 3a0329f Compare July 25, 2019 16:50
@Sojisoyoye Sojisoyoye added Ready for review by TTL PR has been approved by at least two team members and ready to be reviewed by the TTL and removed Work in Progress labels Jul 25, 2019
@benfluleck benfluleck merged commit a6ec791 into develop Jul 25, 2019
@benfluleck benfluleck deleted the ft-image-text-molecule-167454777 branch July 25, 2019 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready for review by TTL PR has been approved by at least two team members and ready to be reviewed by the TTL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants