@tarasis
Posted
Oh fudge, my comment failed, apologies this will be more terse. Congrats on finishing the challenge, it looks good. I don't see the problem with the preview image.
I can't really comment on the CSS/Tailwind, I haven't used it and don't like how cluttered it makes the HTML look. I find it hard to grok.
First I know these are just challenges, but get into the habit of providing useful alt text. Don't leave it as simply alt=""
as you have here.
Where you can use semantic html rather than divs. For instance the first div has a role of main, in which case, why not use the main
tag?
If an icon isn't important for screenreaders and such then add aria-hidden=true
to them. Consider aria-label
to provide more info that a sighted person might not need. For instance have an label that said "The price of the NTF image is 0.041 ethereum"
Best wishes for your next challenge.
Marked as helpful