Byron• 2,180
@byronbyron
Posted
Hey dude, here's a couple of suggestions:
- I would stick a
max-width
andmargin: 0 auto;
on the.container
as well as some padding: - Could put the background on the
body
instead of the.container
- The quotation marks background may need a bit off attention as well on
.item-1
. - And the line height might be better off being
1.5
instead of1.5rem
.container {
max-width: 1158px;
margin: 0 auto;
padding: 4.375rem 1.5rem;
background: none;
line-height: 1.5;
}
body {
background: hsl(210deg 46% 95%);
}
.item1 {
background: url('images/bg-pattern-quotation.svg') no-repeat top 0px right 80px hsl(263, 55%, 52%);
}
You've got the grid template areas working nice!
Keep coding 👍
Marked as helpful
0
abdulrahman badamasi• 430
@papilo-cloud
Posted
@byronbyron Thanks man I really appreciate your feedback
0