@Faris-ThibaniSubmitted about 2 years ago
The solution is part of the refactoring and review of previous solutions. As always, all feedback is welcome!
Marc
@chocolateflightAll comments
- @chocolateflightPosted about 2 years ago
Nicely done! You did a good job copying the design. It is responsive, it is clear. That's very good.
Just one or two suggestions to maybe improve it a little bit:
-
Add a :hover effect to the .cta button, so that when I hover over the button I can see a change which indicated that it's an interactive element. You could even add an :active class that changes the button during my click
-
Maybe this is a browser thing, but in my Chrome-based browser, I see an arrow cursor when hovering over the CTA button. Maybe add a declaration to the CSS that changes the cursor to a pointer
-
In the design suggestion from FrontEnd Mentor, the corners of the container are rounded a bit. All it takes to do that is one simple CSS declaration to make this happen. Maybe you can make this work if you'd prefer the rounded design as well.
Overall, nicely done!
Marked as helpful1 -
