@vraj1091
Posted
To remove warnings please put your all containt of body in the main tag and please remove border from css for that please use css property borfer:none;
@vraj1091
Posted
To remove warnings please put your all containt of body in the main tag and please remove border from css for that please use css property borfer:none;
@SoniBasant
Posted
Hello mate, You made a decent card solution. It also have some scope to improve.
<section class="main-body">
to just <main>
.Layout
The designs were created to the following widths:
Mobile: 375px
Desktop: 1440px
So use media query at 375px. You used at 600px.
Change h3 to h1 then change font-size of h1 as you need from CSS. As one heading (h1) is necessary.
Use Dark-blue: hsl(218, 44%, 22%);
for h1
and Grayish-blue: hsl(220, 15%, 55%);
for p
border-radius
for .outer-div and .outer-div img to make them more similar as you can see in design comparison. You can do this in .outer-div img
. Remove padding-top
..outer-div img{
width: 88%;
border-radius: 10px;
}
Hope I am helpful. Good Luck. : )
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord