Abhishek Guptaā¢ 150
@Abhishek-G06
Posted
@danielmrz-dev
All the updates have been completed. It's ready for testing.
0
Design comparison
Solution retrospective
Please share your opinions on my code.
Community feedback
Daniel šøā¢ 44,230
@danielmrz-dev
Posted
Hello @Abhishek-G06!
You did a very good job there!
I have a very simple suggestion for improvement:
- Since the title is a clickable element, it's nice to add
cursor: pointerand a color shift hover effect to it.
š This gives the user a visual indication that the element is clickable, as obvious as it may look.
I hope it helps!
Other than that, you did a great job!
0
Abhishek Guptaā¢ 150
@Abhishek-G06
Posted
@danielmrz-dev
Thanks for the suggestion. Could I get a little more elaboration? Since I think I missed something.
1
Daniel šøā¢ 44,230
@danielmrz-dev
Posted
@Abhishek-G06
I don't know how I can elaborate more than that, but in the original design the title is a clickable element, like a link to another page with more content about it.
Since that's the case, it's nice to let the user know that's not just a title, but a link to another page. And one of the ways we can let the user know this is to add effects when they hover over it.
In this case, the recommended hover effects are a color shift and a cursor change.
0
Abhishek Guptaā¢ 150
@Abhishek-G06
Posted
@danielmrz-dev
Oh, Thanks for pointing it out. I mistakingly skipped that part of the code.
1
Please log in to post a comment
Log in with GitHub
Join our Discord community
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