Skip to content

Comments

Solution Kickstarte#537

Open
RafaelGratton wants to merge 1 commit intomate-academy:masterfrom
RafaelGratton:develop
Open

Solution Kickstarte#537
RafaelGratton wants to merge 1 commit intomate-academy:masterfrom
RafaelGratton:develop

Conversation

@RafaelGratton
Copy link

Copy link

@raulriato raulriato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You have a few changes that could improve your project. But since it's nothing major, I'll approve it.

Some of your images that have a hover effect would look smoothier if you added a cursor: pointer;

Also, be careful with pointer-events: none;, it can be causing some flaws in your code.

id="email"
placeholder="Your email"
class="form__input"
required

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your required is not working as intended. Verify any brech on the code that might be ignoring it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants