@Cechr
Latest solutions
Latest comments
- @rob-the-automator
Good job! I would try to change as many divs to semantic HTML tags. For example changing the container div to a <main> tag, the "centrado" div to <section> and the "attribution" div to <footer>. I would also change the h2 to and h1 again for semantic reasons. But other than that, it looks good!
- @hectorRperez@rob-the-automator
Good job! I would replace the div with the attribution class to a footer and I would wrap the entire code in a <main> tag and I would change the h2 to an h1 for accessibility reasons.
Marked as helpful - @catherineisonline@rob-the-automator
This looks great! One thing I noticed is that the 'Please provide an email address' popup disappears inconsistently. Sometimes it'll disappear after a couple milliseconds and sometimes it takes a few seconds to disappear. And on a 4k monitor the popup appears on the page mockup(right below the three buttons on the page mockup). But other than that, it looks amazing!
