@artimys
Posted
Nice job Alexei, the card looks very similar to the original.
Some feedback:
-
To help get a more clean/consistent naming pattern to your CSS components. This card challenge I think is perfect for. Look into BEM link here
-
I highly recommend using
background-image
for multiple images on thebody
. Example take from here for more info. It will be tricky but once you get thebackground-image
setup. Focus on gettingbackground-size
and finallybackground-position
#example1 {
background-image: url(img_flwr.gif), url(paper.gif);
background-position: right bottom, left top;
background-repeat: no-repeat, no-repeat;
}
-
Another method instead of using
<hr>
for a border is to wrap the upper section of your card to adiv
and add a bottom border to it. -
For your
<section>
container. They normally require a heading tag. Starting betweenh2
toh6
.
Good work and keep on coding!! 👍
@Batareika007
Posted
Thanks very much for your work, it's very helpful to get that useful feedback !