Skip to content

catalog_hover#2021

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

catalog_hover#2021
usernameluke wants to merge 3 commits into
mate-academy:masterfrom
usernameluke:develop

Conversation

@usernameluke

Copy link
Copy Markdown

@anastasiia-tilikina anastasiia-tilikina left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Almost done!
let's just add some improvements

Comment thread src/styles/blocks/catalog.scss Outdated
border-radius: 5px;
margin-bottom: 48px;

transition-duration: 300ms;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

it's better to set transitioned properties explictly

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

not fixed

Comment thread src/styles/blocks/header.scss Outdated
text-transform: uppercase;
font-size: 12px;
font-weight: 500;
transition-duration: 300ms;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

fix here as well

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

not fixed

@lerastarynets lerastarynets left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pls fix prev comments

Comment thread src/styles/blocks/catalog.scss Outdated
border-radius: 5px;
margin-bottom: 48px;

transition-duration: 300ms;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

not fixed

Comment thread src/styles/blocks/header.scss Outdated
text-transform: uppercase;
font-size: 12px;
font-weight: 500;
transition-duration: 300ms;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

not fixed

@anastasiia-tilikina anastasiia-tilikina left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

GJ!

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