Skip to content
OverviewSolutions
9
Comments
7
Jérémy Mouzin
@jeremymouzin

All comments

  • Jérémy Mouzin•160
    @jeremymouzinPosted 13 days ago

    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)
    1
  • sediqa•150
    @Hasani-SediqaSubmitted 15 days ago
    What 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.

    four card feature section using HTML & CSS

    1
    Jérémy Mouzin•160
    @jeremymouzinPosted 15 days ago

    Very 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
    0
  • 4rd2•60
    @4rd2Submitted 18 days ago
    What 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.

    Product Preview Card

    1
    Jérémy Mouzin•160
    @jeremymouzinPosted 18 days ago

    Well 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
    0
  • Jérémy Mouzin•160
    @jeremymouzinPosted 30 days ago

    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
    0
  • Jérémy Mouzin•160
    @jeremymouzinPosted about 1 month ago

    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 👍

    0
  • Jérémy Mouzin•160
    @jeremymouzinPosted about 1 month ago

    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
    0
  • Jérémy Mouzin•160
    @jeremymouzinPosted about 1 month ago

    Great work! Your shadow should not be all around the card though, just at the bottom and slightly on the left and right I think. Double check the design!

    0