P
Thomas Dimnet• 250
@tdimnet
Posted
Hi creatored,
Nice project so far! It looks nice both on mobile and desktop devices.
I think the line below be a h1 tag instead of a span:
<span>Improve your front-end skills by building projects</span>
You could add a better descriptive description for the alt tag. Don't forget it used for screen readers.
Why did you create a JavaScript file? You don't seem to use it.
You did a good job for your first project :).
1
creatored• 50
@creatored
Posted
@tdimnet Thanks for pointing out the use of span element and alt tag! I forgot to remove the Javascript file before submitting it. Thanks for the feedback!
0
