@DarkKiller31
Submitted
@MirrorIce
@DarkKiller31
Submitted
@paramaguru077
Submitted
Looks very neat! Here are some improvements that would help your project look better:
@candebermejo
Submitted
Hello,
The solution looks nice and neat, but the image is not visible for some reason. It might be a good idea reviewing that.
Containing children by selectors, instead of selecting the identifiers, though nice, might hinder remembering which styling is which in the long term. So you could use more ids and classes in your HTML file, to leverage your CSS code.
Also consider a standard font-size in which you could derive your rems. In this project, it was roughly 16px on desktop screens and 14px on mobile.
@Control222
Submitted
The code looks really nice, don't have anything much to the structure.
I think the only thing worth mentioning is that the padding on the image (and maybe on the text) is a bit off. You put it as 1 rem with a base of 15px, but the padding itself is 16px.
The image corners have too much of a radius (1 rem -> 15 px instead of 10px). There are rem to px calculators online, so maybe that can help you if you want to keep using that unit.
The title font should be grayish blue instead of black.