Skip to content

Catalog-hovers#1997

Open
Kacper-Leman wants to merge 4 commits into
mate-academy:masterfrom
Kacper-Leman:develop
Open

Catalog-hovers#1997
Kacper-Leman wants to merge 4 commits into
mate-academy:masterfrom
Kacper-Leman:develop

Conversation

@Kacper-Leman
Copy link
Copy Markdown

Copy link
Copy Markdown

@DorotaLeniecDev DorotaLeniecDev left a comment

Choose a reason for hiding this comment

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

One small change needed

Comment thread src/styles/blocks/header.scss Outdated
Comment on lines +44 to +45
transition: color;
transition-duration: $delayValue;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Add transition to origin element (not :hover) so the transisiton is smooth in both directions, you can use short version:

Suggested change
transition: color;
transition-duration: $delayValue;
transition: color $delayValue

Copy link
Copy Markdown

@DorotaLeniecDev DorotaLeniecDev left a comment

Choose a reason for hiding this comment

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

One more fix needed

Comment thread src/styles/blocks/header.scss Outdated
Comment on lines +43 to +46
&__link:hover {
transition: color $delayValue;
color: #00acdc;
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Put transition to origin element not hover to make animation smooth in both directions:

Suggested change
&__link:hover {
transition: color $delayValue;
color: #00acdc;
}
&__link {
transition: color $delayValue;
}

@darokrk darokrk self-requested a review June 8, 2023 06:08
Copy link
Copy Markdown

@darokrk darokrk left a comment

Choose a reason for hiding this comment

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

Approved 👍🏽 good job 🥇

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