Skip to content
OverviewSolutions
9
Comments
4
Shadab Khan
@bentekku

All comments

  • Shadab Khan•130
    @bentekkuPosted about 1 year ago

    Don't mind the screenshot taken by FrontendMentor, the real design is not off-looking.

    0
  • Shadab Khan•130
    @bentekkuPosted about 1 year ago

    Hey Matthew, nice work. Your design looks identical to the challenge. As for the community question you asked, the answer would be, eye-balling the correct border-radius and padding.

    Use chrome dev tools to tweak the css in the browser itself to save time and frustration switching back and forth between the code editor and the browser

    Cheers,

    Shadab

    Marked as helpful
    1
  • Shadab Khan•130
    @bentekkuPosted about 1 year ago

    Your design looks good, however, here are some pointers:

    • The Design is not aligned center vertically, which could be done using,
    display: flex;
justify-content: center;
align-items: center;

    and yes, do give height of 100vh to the main parent, which could be the body element in your case.

    • The summary column elements are bit off color, which could be simply solved by changing the hsl to hsla, and adding 0.1 as the alpha for the hsla or tweak around till you see it fit. e.g.,
     hsl(210, 28, 32, 0.1 );

    Overall, you really nailed the design. Good work!

    Marked as helpful
    0
  • Shadab Khan•130
    @bentekkuPosted about 1 year ago

    I decide to go with wider container for the Desktop version, as it seems to make use of the empty space the best.

    0