Skip to content

Conversation

@Shadi38
Copy link
Collaborator

@Shadi38 Shadi38 commented Jun 6, 2024

No description provided.

@netlify
Copy link

netlify bot commented Jun 6, 2024

Deploy Preview for avanart failed.

Name Link
🔨 Latest commit 0cd368b
🔍 Latest deploy log https://app.netlify.com/sites/avanart/deploys/666623d45279170008b673e4

@Shadi38 Shadi38 requested review from Bedi06 and KristinaDudnyk June 9, 2024 21:56
});

saveAs(blob, "avatar.png");
if (blob) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Good error handling

<ReactNiceAvatar
className="w-64 h-64 highres:w-80 highres:h-80"
hairColorRandom
shape={this.state.shape}
Copy link
Collaborator

Choose a reason for hiding this comment

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

On line 113 you did destructuring, so you could directly access all of it without "this.state"

Copy link
Collaborator Author

@Shadi38 Shadi38 Jun 10, 2024

Choose a reason for hiding this comment

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

If you are talking about app component, this structure comes from react nice avatar and we didn't change it, but I worked on form page and tried to stick role and region on top of image after clicking the submit btn.
#33

Copy link
Collaborator

Choose a reason for hiding this comment

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

The Form looks all right to me, thank you Shadi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants