Skip to content

catalog hovers#2020

Open
Sadeni-09 wants to merge 3 commits into
mate-academy:masterfrom
Sadeni-09:develop
Open

catalog hovers#2020
Sadeni-09 wants to merge 3 commits into
mate-academy:masterfrom
Sadeni-09:develop

Conversation

@Sadeni-09
Copy link
Copy Markdown

Copy link
Copy Markdown

@anastasiia-tilikina anastasiia-tilikina left a comment

Choose a reason for hiding this comment

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

Let's fix some code issues.
check my comments.

Also, please read this and follow best practices here and in the future

Comment thread src/styles/blocks/catalog.scss Outdated
Comment thread src/styles/blocks/header.scss Outdated
@Sadeni-09
Copy link
Copy Markdown
Author

Hello i deployed again but I can't see the track of deployment
Uploading image.png…

image

Copy link
Copy Markdown

@lerastarynets lerastarynets left a comment

Choose a reason for hiding this comment

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

Prev comments were not fixed, maybe u forgot to push your code to github?

@Sadeni-09
Copy link
Copy Markdown
Author

Can you please check latest version?

Copy link
Copy Markdown

@lerastarynets lerastarynets left a comment

Choose a reason for hiding this comment

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

Great job overall! Just one tiny thing needs to be fixed before approval)

Comment thread src/styles/main.scss Outdated
Copy link
Copy Markdown

@lerastarynets lerastarynets left a comment

Choose a reason for hiding this comment

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

You've got this, keep pushing forward!

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.

3 participants