Skip to content
OverviewSolutions
3
Comments
2
Md Rafsan Jani Rafin
@itsrafsanjani

All comments

  • Paul•10
    @ChasusaSubmitted about 3 years ago

    This is my first submission on Frontend Mentor.

    Would appreciate feedback on my HTML structure and CSS as well. I feel that I wrote too much code and could maybe have simplified it better.

    Thanks for any feedback!

    Order Summary Card with CSS FLexbox

    4
    Md Rafsan Jani Rafin•70
    @itsrafsanjaniPosted about 3 years ago

    Your design looks perfect. But did you manually calculate those bottom, left, right, top? If yes then stop doing that. Try to use flex or grid as much as possible. So that other developers can read and modify your code easily. And use href="#" if you don't know the link.

    Marked as helpful
    0
  • Md Rafsan Jani Rafin•70
    @itsrafsanjaniPosted about 3 years ago

    body { background-repeat: no-repeat; background-size: cover; }

    Use this and also add hover state for the buttons.

    And also add some box-shadow for the card.

    Marked as helpful
    0