Skip to content
OverviewSolutions
46
Comments
46

Akhlak Hossain Jim

@Akhlak-Hossain-JimBangladesh1,325 points

Akhlak is a random guy who loves coding beautiful frontend. Don't love to stay home on weekends or holidays because there is a lot in this world to see.

Latest solutions

View more solutions

Latest comments

  • Akhlak Hossain Jim•1,325
    @Akhlak-Hossain-Jim

    Hey, You have done a great job completing the challenge. Here is some of my suggestions to make it more better.

    • remove the margin from all side in the mobile view (margin: 0;)
    • In the mobile view try adding width: 100%;
    • You can start your from 576px (standard mobile view of bootstrap)
  • Akhlak Hossain Jim•1,325
    @Akhlak-Hossain-Jim

    Hi there, great work so far,

    But here are some things you can do to improve your code.

    1. add line-height: 100%; at your .rare-btn that way it will be a proper circle.
    2. add can add an if else statement to check if someone has selected a rating or not, if so you change the card or do nothing that would be great

    Keep up the good work. Happy coding:)

    Marked as helpful
  • Akhlak Hossain Jim•1,325
    @Akhlak-Hossain-Jim

    Great work so far.

    Here are some of my suggestions,

    1. consider using column gap as well for responsiveness
    2. instead of hard-coded values like px, try using relative values like em, rem, % etc making the site responsive

    And remember practice will make it better so practice more. Happy coding :)

  • Akhlak Hossain Jim•1,325
    @Akhlak-Hossain-Jim

    Great work so far. Everything looks good on the website.

    Here is some of my suggestion,

    1. I found % is best for this kind of scenario where calculation involved
    2. and for the resolution, you should consider using relative values like em, rem, and more or you can use CSS functions like min(), max(), minmax() check this and this resources for reference.
    3. check for typos in code like sectiom

    Keep coding everything just need a bit practice. Happy coding :).

  • Akhlak Hossain Jim•1,325
    @Akhlak-Hossain-Jim

    Great work so far, but

    1. consider using main tag for the card container that would be a great practice
    2. add some space in the mobile view
    3. try using h1 at first and then h2, h3 and all

    Other than that everything looks good. Happy coding:).

  • Akhlak Hossain Jim•1,325
    @Akhlak-Hossain-Jim

    Nice work. For that you can do:

    1. set a fixed bottom padding,
    2. take a div with display: flex; for p and the button and
    3. make flex-direction: column; justify-content: space-between;

    or

    1. in section add display: flex; flex-direction: column;,
    2. select img and add width:64px;height:auto;
    3. in button add margin-top:auto;

    and it should always stay in the same position.

    Happy coding :)

    Marked as helpful
View more comments