Skip to content

Catalog Hover Solution#2009

Open
aRabiej wants to merge 2 commits into
mate-academy:masterfrom
aRabiej:develop
Open

Catalog Hover Solution#2009
aRabiej wants to merge 2 commits into
mate-academy:masterfrom
aRabiej:develop

Conversation

@aRabiej
Copy link
Copy Markdown

@aRabiej aRabiej commented Jul 14, 2023

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.

Add demo link

Comment thread src/styles/variables.scss Outdated
@@ -0,0 +1,3 @@
$cardHeadercolor: #34568b;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Use standard name for naming color variable like color-blue

Comment thread src/styles/blocks/header.scss Outdated
margin-right: 20px;
}
}
transition: color, $transitionDuration;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This is incorrect, comma should separate the transitions group

Suggested change
transition: color, $transitionDuration;
transition: color $transitionDuration;

@aRabiej aRabiej requested a review from DorotaLeniecDev July 18, 2023 12:57
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.

Demo page is still not working :(

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.

2 participants