Design comparison
Solution retrospective
Hi πββοΈ Frontend Mentor community!
The active state was a challenge. But it made me learn more. π
Used: -Semantic HTML5 markup -Flexbox -Media-queries
π Any advice is appreciated π
Community feedback
- @RioCantrePosted almost 3 years ago
Hello there! Awesome work with this project. Viewing your solution, you did well in implementing the design. I would only suggest this for you...
- Add
cursor: pointer
to the hover state of the design
Overall you did a great job and Keep it going!
0@Ahmed-ElhebirPosted almost 3 years ago@RioCantre Hi (:
I appreciate it π
Thanks for the suggestion π this will be a great add to the project.
I will πͺπΌ and I'm looking forward for your feedback. π
0@RioCantrePosted almost 3 years ago@Ahmed-Elhebir No worries. I would appreciate it if you mark it as helpful and I can give you feedbacks anytime. Cheers!
0 - Add
- @rsrclabPosted almost 3 years ago
Hi, @Ahmed-Elhebir ~
Congratulate on your solution to the challenge on FM platform. I have studied your work carefully and learned a lot from it.
Here are some of the tips I like to provide.
- I suggest you to try transition on hoverable elements like image, heading and creator name.
- Please try BEM for naming element classes. It will help you a lot on bigger projects.
https://www.frontendmentor.io/solutions/my-first-solution-on-chanllenge-V-4IzAivH
Here is my solution to this challenge, and if it can help you even a bit, it would be happy to me.
Cheers ~
0@Ahmed-ElhebirPosted almost 3 years agoHi, @tymren608
I appreciate that you saw my work and learned from it. π
thanks for your tips I'm working on the transition now and about the BEM I'm still learning it but I will think about add them in my projects.
I'll definitely see your solution and I'm sure I will learn from it π
Thanks a lot π
0 - @catherineisonlinePosted almost 3 years ago
Looks really nice! I would add a smoother transition for hover states :))
0@Ahmed-ElhebirPosted almost 3 years ago@catherineisonline Thanks I appreciate it π
That's a great idea I will try it now π
Thanks a lot π
0
Please log in to post a comment
Log in with GitHubJoin 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