@darryncodes
Posted
Hi Adithya,
Really close match, good effort. With a few more little tweaks it could be perfect!
- use
padding: 15px 70px;
on your.container_main
instead of margin, it won't create the border - add your background styles to the
<body>
instead of your.container
and removebackground-position: top;' with
background-size: 100% auto;` - add
cursor: pointer;
to your button styles - be mindful of you semantic html mark up and heading structure
All the best!
Marked as helpful
@ADITHYAADHI06
Posted
@darryncodes thank you great feedback... can u help me to make this card height and width responsive.
@darryncodes
Posted
you're welcome @ADITHYAADHI06. Good question, try to avoid using fixed heights and add max-width
instead of width
.
Two other suggestions:
- use media queries to change the styles per screen width
- use relative units (rem/em) not absolute units (px)
"REM unit makes everything take their size relative to the root font-size. If we change the root font-size, all elements will automatically change their sizes" here is the article the above snippet is from
If you change the root font-size at different break points with media queries everything will reduce in size - it's like magic!
This is a common starting point for me:
html {
font-size: 62.5%;
@media only screen and (max-width: 56.25em) { //900px
font-size: 56.25%;
}
@media only screen and (max-width: 37.5em) { //600px
font-size: 50%;
}
}
Marked as helpful