Hi @Drallas
This looks a bit broken on mobile because of height 100vh instead of min-height. You always want backgrounds to be able to grow just like anything else when working responsively.
In the html
- logo image alt should just be the company/product name, not say logo
- the other company logos are communicating valuable information so should not be aria hidden and should have alt descriptions. I would also add a visually hidden (screenreader only) heading above them to say something like 'companies that use Pod'
- the input needs a label
Positioning decorative motifs like those dots absolutely is ideal, I don't think they need to be in the document flow unless you're having trouble with the positioning in some way. Not sure I get the problem (if there is one)
Keep going though, it's looking good overall
@Drallas
Posted
@grace-snow Hello Grace
Thank you for the review. As always i will look into the remarks, trying to make it a bit better and learning for next one. :)
I see you've added the validation. Nice
Just make sure there's a way to proceed with success and to remove the error message when the email is valid ☺
