matt2282• 310
@matt2282
Posted
You did the email validation and switching of the pages correctly and the general layout of the page is good, but it does have a lot of issues.
The email input has a cursor pointer when it should be a text selection pointer(this is default I believe). Desktop: -background color should be hsl(235, 18%, 26%)
- The padding on the right of the image gets messed up when you switch from mobile back to desktop.
- Missing a space on "Stay updated!"
- I'd recommend setting a fixed height/width for the outside container because things get weird when the container gets resized.
- Also recommend keep the image at a fixed size. -Could use a bit more padding especially on the left.
Mobile:
- The original design has the image at the very top of the page while your solution has it centered and the desktop background visible.
Success Page:
- Misspelled "subscribing!"
- The email should be the same as the email you entered in the box on the first page not [email protected] every time.
- You forgot to make a mobile friendly version. It looks not right on mobile.
0
