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

#168137051 Implement component for avatar and avatar-group #267

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Luckzman
Copy link
Collaborator

@Luckzman Luckzman commented Aug 29, 2019

What Does This PR Do?

This PR creates Avatar and AvatarGroup Component

Description Of Task To Be Completed

  • create Avatar component
  • create AvatarGroup component
  • style both Avatar component and AvatarGroup component

Any Background Context You Want To Provide?

How can this be manually tested?

  • Fetch the branch locally by running git fetch origin ft-create-avatar-component-168137051 & git checkout ft-create-avatar-component-168137051 respectively
  • import the Avatar Component or AvatarGroup Component to any parent component where it is needed
  • pass in the required props for the component to render properly
  • on the terminal, cd into revampedClient folder and start the app

What are the relevant pivotal tracker stories?

#168137051

Screenshot(s)

avatar_3

@Luckzman Luckzman force-pushed the ft-create-avatar-component-168137051 branch 2 times, most recently from 5c899e9 to 6e4666e Compare August 29, 2019 04:51
@Luckzman Luckzman added the wip label Aug 29, 2019
Copy link
Collaborator

@oluwajuwon oluwajuwon left a comment

Choose a reason for hiding this comment

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

nice work @Luckzman
Please have a look at the comment and also kindly change your PR title to adhere to the team's convention

* @param {object} { imgUrl, classes, altText }
* @returns JSX
*/
export default function Avatar({ imgUrl, classes, altText }) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

any reason why you're not using the fat arrow method?

@Luckzman Luckzman changed the title feat(avatar component): Implement component for avatar and avatar-group #168137051 Implement component for avatar and avatar-group Sep 3, 2019
- Implement component for avatar
- Implement component for avatar group
- Style both avatar and avatar group component

[#168137051]
@Luckzman Luckzman force-pushed the ft-create-avatar-component-168137051 branch 6 times, most recently from b52d4a6 to c00c5e4 Compare September 16, 2019 15:55
@Luckzman Luckzman force-pushed the ft-create-avatar-component-168137051 branch from c00c5e4 to 5bbda24 Compare September 16, 2019 19:10
@Luckzman Luckzman added frontend Ready for review the pr is ready for review and removed wip labels Sep 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Ready for review the pr is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants