@Dannimations
Latest solutions
Animated Rock Paper Scissors Spock Lizard
Submitted 7 months agoThe accessibility part I guess, I'm still learning this. Feel free to reach me if you now any way of improving my project! Thanks for sharing your knowledge.
Four card feature section
Submitted 8 months agoThe box-shadow figma value of cards, I don't have it and it's a nightmare to retro-engineer.
Latest comments
- @jeremymouzin
Ah ah you redesigned the original design 🤣. Why not! Here are some issues I spotted:
- The big quote symbol on violet background testimonial is missing.
- When you have a viewport bigger than 1440px the cards text starts rolling over other cards, it's a mess!
- Your site is not reponsive at all (mobile version is missing)
- @Hasani-SediqaWhat specific areas of your project would you like help with?
If you have any ideas or suggestions for improvement, I'd love to hear them. Your feedback is incredibly valuable and will help me to learn, and grow my skills.
@jeremymouzinVery nice! Here are some improvements you may work on:
- Add some margin at the top of your site so that the slider solution/design is perfectly aligned and you can spot easily differences
- The font-size seems to be off everywhere you may work on that
- Last point the top border colored on cards is NOT rounded, think about a solution to make it straight as in the design (hint: use
card::before!)
Good luck!
Marked as helpful - @4rd2What are you most proud of, and what would you do differently next time?
I am most proud of learning about mobile first design and using it in a project.
What challenges did you encounter, and how did you overcome them?I had many challenges with the font and making the design.
@jeremymouzinWell done!
The design may be improved though:
- Missing padding at the bottom of the content on the right, below the button
- Too much padding around the text
- The font weight of the title doesn't seem right
- You could use letter-spacing on the Perfume line
- @ManojSinghDashauni@jeremymouzin
Very nice start! Some things that needs your attention:
- The background color is not rose but stone-100
- Your layout seems completely translated on the Y axis? We almost can't see the image at the top
- The text color is not black it's stone-600 (maybe add some transparency to get it as close as possible)
Good luck!
Marked as helpful - @gustavoeguedes@jeremymouzin
Very nice, the image seems far too big though (it's a 90px wide image) and the buttons for the links too, they're very wide! The hover effect is nice on the button with the transition of 300ms 👍
- @lgwarda@jeremymouzin
Nice work. Your HTML is semantic and pretty good.
Some things you may want to improve on the design:
- Add a max-width on desktop that matches the design
- The "published" line should be gray-950 not gray-500
- Use the fonts provided by the designer (@font-face to the rescue!)
- Font sizes on mobile are smaller (you may use a media-query or use clamp() to make the font dependant on vw unit (viewport width).
Good luck!
Marked as helpful
