-
Notifications
You must be signed in to change notification settings - Fork 4.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test solution #5040
base: master
Are you sure you want to change the base?
test solution #5040
Changes from 5 commits
5bc9acb
86632c9
2eaa6b0
7e7285a
2fca7ca
8eb04d6
0262f01
fe37e20
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -1,6 +1,28 @@ | ||||||||||||||||||||||||||||||||||||||||||||||
<!doctype html> | ||||||||||||||||||||||||||||||||||||||||||||||
<html lang="en"> | ||||||||||||||||||||||||||||||||||||||||||||||
<head> | ||||||||||||||||||||||||||||||||||||||||||||||
<link | ||||||||||||||||||||||||||||||||||||||||||||||
rel="preconnect" | ||||||||||||||||||||||||||||||||||||||||||||||
href="https://fonts.googleapis.com" | ||||||||||||||||||||||||||||||||||||||||||||||
/> | ||||||||||||||||||||||||||||||||||||||||||||||
<link | ||||||||||||||||||||||||||||||||||||||||||||||
rel="preconnect" | ||||||||||||||||||||||||||||||||||||||||||||||
href="https://fonts.gstatic.com" | ||||||||||||||||||||||||||||||||||||||||||||||
crossorigin="use-credentials" | ||||||||||||||||||||||||||||||||||||||||||||||
/> | ||||||||||||||||||||||||||||||||||||||||||||||
<link | ||||||||||||||||||||||||||||||||||||||||||||||
rel="preconnect" | ||||||||||||||||||||||||||||||||||||||||||||||
href="https://fonts.googleapis.com" | ||||||||||||||||||||||||||||||||||||||||||||||
/> | ||||||||||||||||||||||||||||||||||||||||||||||
<link | ||||||||||||||||||||||||||||||||||||||||||||||
rel="preconnect" | ||||||||||||||||||||||||||||||||||||||||||||||
href="https://fonts.gstatic.com" | ||||||||||||||||||||||||||||||||||||||||||||||
crossorigin="use-credentials" | ||||||||||||||||||||||||||||||||||||||||||||||
/> | ||||||||||||||||||||||||||||||||||||||||||||||
<link | ||||||||||||||||||||||||||||||||||||||||||||||
href="https://fonts.googleapis.com/css2?family=Roboto:wght@400;500;700&display=swap" | ||||||||||||||||||||||||||||||||||||||||||||||
rel="stylesheet" | ||||||||||||||||||||||||||||||||||||||||||||||
/> | ||||||||||||||||||||||||||||||||||||||||||||||
<meta charset="UTF-8" /> | ||||||||||||||||||||||||||||||||||||||||||||||
<meta | ||||||||||||||||||||||||||||||||||||||||||||||
name="viewport" | ||||||||||||||||||||||||||||||||||||||||||||||
|
@@ -13,6 +35,43 @@ | |||||||||||||||||||||||||||||||||||||||||||||
/> | ||||||||||||||||||||||||||||||||||||||||||||||
</head> | ||||||||||||||||||||||||||||||||||||||||||||||
<body> | ||||||||||||||||||||||||||||||||||||||||||||||
<h1>Product cards</h1> | ||||||||||||||||||||||||||||||||||||||||||||||
<div | ||||||||||||||||||||||||||||||||||||||||||||||
class="card" | ||||||||||||||||||||||||||||||||||||||||||||||
data-qa="card" | ||||||||||||||||||||||||||||||||||||||||||||||
> | ||||||||||||||||||||||||||||||||||||||||||||||
<img | ||||||||||||||||||||||||||||||||||||||||||||||
class="card__image" | ||||||||||||||||||||||||||||||||||||||||||||||
src="./images/imac.jpeg" | ||||||||||||||||||||||||||||||||||||||||||||||
alt="Product" | ||||||||||||||||||||||||||||||||||||||||||||||
/> | ||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__title"> | ||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__title-text"> | ||||||||||||||||||||||||||||||||||||||||||||||
APPLE A1419 iMac 27" Retina 5K Monoblock (MNED2UA/A) | ||||||||||||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__title-product-code">Product code: 195434</div> | ||||||||||||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__stars"> | ||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__stars-star"></div> | ||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__stars-star"></div> | ||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__stars-star"></div> | ||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__stars-star"></div> | ||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__stars-star"></div> | ||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__stars-reviews">Reviews: 5</div> | ||||||||||||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. if you have separate file stars then you should have stars block and you need container for star elements so you could use space-between
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__price"> | ||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__price-text">Price:</div> | ||||||||||||||||||||||||||||||||||||||||||||||
<div class="card__price-sum">$2,199</div> | ||||||||||||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||
<a | ||||||||||||||||||||||||||||||||||||||||||||||
class="card__button" | ||||||||||||||||||||||||||||||||||||||||||||||
href="#" | ||||||||||||||||||||||||||||||||||||||||||||||
data-qa="hover" | ||||||||||||||||||||||||||||||||||||||||||||||
> | ||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||
buy | ||||||||||||||||||||||||||||||||||||||||||||||
</a> | ||||||||||||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||||||||||||
</body> | ||||||||||||||||||||||||||||||||||||||||||||||
</html> |
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,23 @@ | ||||||||||
.card__button { | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||
@include center; | ||||||||||
@include roboto700; | ||||||||||
|
||||||||||
height: 40px; | ||||||||||
width: $primary-width; | ||||||||||
margin-top: 15px; | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||
border-radius: 5px; | ||||||||||
border-color: transparent; | ||||||||||
KrawczykDamiann marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||
text-decoration: none; | ||||||||||
cursor: pointer; | ||||||||||
font-size: 14px; | ||||||||||
line-height: 16px; | ||||||||||
text-transform: uppercase; | ||||||||||
color: #fff; | ||||||||||
background-color: #00acdc; | ||||||||||
|
||||||||||
KrawczykDamiann marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||
&:hover { | ||||||||||
background-color: #fff; | ||||||||||
border: 1px solid #00acdc; | ||||||||||
color: #00acdc; | ||||||||||
} | ||||||||||
} |
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
@@ -0,0 +1,38 @@ | ||||||
.card { | ||||||
@include center; | ||||||
|
||||||
padding: 16px; | ||||||
border: 1px solid #f3f3f3; | ||||||
border-radius: 5px; | ||||||
flex-direction: column; | ||||||
|
||||||
KrawczykDamiann marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
&__image { | ||||||
margin-top: 16px; | ||||||
width: 160px; | ||||||
height: 134px; | ||||||
background-image: url('../images/imac.jpeg'); | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
} | ||||||
|
||||||
&__title { | ||||||
margin-top: 40px; | ||||||
|
||||||
&-text { | ||||||
@include roboto500; | ||||||
|
||||||
width: 166px; | ||||||
display: flex; | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
font-size: 12px; | ||||||
line-height: 18px; | ||||||
color: #060b35; | ||||||
} | ||||||
|
||||||
&-product-code { | ||||||
@include roboto400; | ||||||
|
||||||
color: #616070; | ||||||
font-size: 10px; | ||||||
line-height: 14px; | ||||||
margin-top: 4px; | ||||||
} | ||||||
} | ||||||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
@mixin center { | ||
display: flex; | ||
align-items: center; | ||
justify-content: center; | ||
} | ||
|
||
@mixin roboto400 { | ||
font-family: Roboto, serif; | ||
font-weight: 400; | ||
font-style: normal; | ||
} | ||
|
||
@mixin roboto500 { | ||
font-family: Roboto, serif; | ||
font-weight: 500; | ||
font-style: normal; | ||
} | ||
|
||
@mixin roboto700 { | ||
font-family: Roboto, serif; | ||
font-weight: 700; | ||
font-style: normal; | ||
} |
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,20 @@ | ||||||||||
.card__price { | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||
display: flex; | ||||||||||
justify-content: space-between; | ||||||||||
align-items: center; | ||||||||||
margin-top: 22px; | ||||||||||
width: 166px; | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||
|
||||||||||
&-text { | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||
@include roboto400; | ||||||||||
|
||||||||||
color: #616070; | ||||||||||
font-size: 12px; | ||||||||||
} | ||||||||||
|
||||||||||
&-sum { | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||
@include roboto700; | ||||||||||
|
||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||
font-size: 16px; | ||||||||||
} | ||||||||||
} |
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,27 @@ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
.card__stars { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
@include center; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||
line-height: 14px; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
width: 100%; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
margin-top: 17px; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||
&-star { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
KrawczykDamiann marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||
width: 16px; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
height: 16px; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
margin-right: 4px; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
background-image: url('../images/star-active.svg'); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
background-repeat: no-repeat; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||
&-star:nth-child(n + 5) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||
background-image: url('../images/star.svg'); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||
&-reviews { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
@include roboto400; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||
margin-left: 16px; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
KrawczykDamiann marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||
color: #060b35; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
font-size: 10px; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
} |
Original file line number | Diff line number | Diff line change | ||
---|---|---|---|---|
@@ -0,0 +1 @@ | ||||
$primary-width: 166px; | ||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. create variables for colors
Suggested change
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. you don't need this file |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,131 @@ | ||
.card { | ||
display: flex; | ||
align-items: center; | ||
justify-content: center; | ||
padding: 16px; | ||
border: 1px solid #f3f3f3; | ||
border-radius: 5px; | ||
flex-direction: column; | ||
} | ||
|
||
.card__image { | ||
margin-top: 16px; | ||
width: 160px; | ||
height: 134px; | ||
background-image: url('../images/imac.jpeg'); | ||
} | ||
|
||
.card__title { | ||
margin-top: 40px; | ||
} | ||
|
||
.card__title-text { | ||
font-family: Roboto, serif; | ||
font-weight: 500; | ||
font-style: normal; | ||
width: 166px; | ||
display: flex; | ||
font-size: 12px; | ||
line-height: 18px; | ||
color: #060b35; | ||
} | ||
|
||
.card__title-product-code { | ||
font-family: Roboto, serif; | ||
font-weight: 400; | ||
font-style: normal; | ||
color: #616070; | ||
font-size: 10px; | ||
line-height: 14px; | ||
margin-top: 4px; | ||
} | ||
|
||
.card__stars { | ||
display: flex; | ||
align-items: center; | ||
justify-content: center; | ||
line-height: 14px; | ||
width: 100%; | ||
margin-top: 17px; | ||
} | ||
|
||
.card__stars-star { | ||
width: 16px; | ||
height: 16px; | ||
margin-right: 4px; | ||
background-image: url('../images/star-active.svg'); | ||
background-repeat: no-repeat; | ||
} | ||
|
||
.card__stars-star:nth-child(n + 5) { | ||
background-image: url('../images/star.svg'); | ||
} | ||
|
||
.card__stars-reviews { | ||
font-family: Roboto, serif; | ||
font-weight: 400; | ||
font-style: normal; | ||
margin-left: 16px; | ||
color: #060b35; | ||
font-size: 10px; | ||
} | ||
|
||
.card__price { | ||
display: flex; | ||
justify-content: space-between; | ||
align-items: center; | ||
margin-top: 22px; | ||
width: 166px; | ||
} | ||
|
||
.card__price-text { | ||
font-family: Roboto, serif; | ||
font-weight: 400; | ||
font-style: normal; | ||
color: #616070; | ||
font-size: 12px; | ||
} | ||
|
||
.card__price-sum { | ||
font-family: Roboto, serif; | ||
font-weight: 700; | ||
font-style: normal; | ||
font-size: 16px; | ||
} | ||
|
||
.card__button { | ||
display: flex; | ||
align-items: center; | ||
justify-content: center; | ||
font-family: Roboto, serif; | ||
font-weight: 700; | ||
font-style: normal; | ||
height: 40px; | ||
width: 166px; | ||
margin-top: 15px; | ||
border-radius: 5px; | ||
border-color: transparent; | ||
text-decoration: none; | ||
cursor: pointer; | ||
font-size: 14px; | ||
line-height: 16px; | ||
text-transform: uppercase; | ||
color: #fff; | ||
background-color: #00acdc; | ||
} | ||
|
||
.card__button:hover { | ||
background-color: #fff; | ||
border: 1px solid #00acdc; | ||
color: #00acdc; | ||
} | ||
|
||
body { | ||
display: flex; | ||
align-items: center; | ||
justify-content: center; | ||
font-family: Roboto, serif; | ||
font-weight: 400; | ||
font-style: normal; | ||
margin: 0; | ||
} /*# sourceMappingURL=index.css.map */ |
KrawczykDamiann marked this conversation as resolved.
Show resolved
Hide resolved
|
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change | ||
---|---|---|---|---|
@@ -1,3 +1,13 @@ | ||||
@import './mixins'; | ||||
@import './variables'; | ||||
@import './card'; | ||||
@import './stars'; | ||||
@import './price'; | ||||
@import './button'; | ||||
|
||||
body { | ||||
@include center; | ||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||
@include roboto400; | ||||
|
||||
margin: 0; | ||||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are duplicate preconnect links for 'https://fonts.googleapis.com' and 'https://fonts.gstatic.com'. It's redundant to have multiple preconnect links for the same URL, as it can lead to unnecessary network requests. Consider removing the duplicates to optimize performance.