Looking to hire developers?
Submitted
@Rohitgour03
Hello, Here is my solution to this challenge. π₯π I faced a problem in the animation part. however I managed to do that, but want to know how will tackle that thing and how I can improve? Please provide your valuable feedback!! π
@a1excpunk
Posted
Good job with animation , try with the toggle method on classlist (fewer conditions and redundant code). one thing, background color is different and shadow is too intense.
Marked as helpful
@Rohitgour03
Posted
@a1excpunk Thanks, Alex. I just forgot that I can use toggle π . I will update the background and shadow. π
@anoshaahmed
Posted
To get rid of the accessibility/HTML issues shown in your Report:
<main> ... OR use semantic tags ... OR give role="" to the direct children of your <body> ... Click here to read more<h1> in your codeGreat job! :)
@Madmanden
Posted
Hi @Rohitgour03, the animation looks good on my phone! Nice work π
@Rohitgour03
Posted
@Madmanden Thanks, Christian. π
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord
