Steven Portas• 440
@satrop
Posted
Nice work! Super clean HTML and nice CSS.
Only have two suggestions. In the HTML layer, I would add a title in the head so your tab reads something a little more interesting than "Document". The other... more a thing to think about than a suggestion. From a front-end professional point adding images as a background-image in the CSS layer can be problematic for some CMS systems to handle. It would be better to have the image either inline on a div or as a standard img-tag either option is easy to style as though it were a background image with CSS/SCSS.
All the best, Steve.
Marked as helpful
0
adr99• 410
@leorichy99
Posted
@satrop Thanks 😍
1
