Skip to content
OverviewSolutions
3
Comments
3
Antoine-thz
@Antoine-thz

All comments

  • Antoine-thz•90
    @Antoine-thzPosted 10 months ago

    Hi, I have 2 comments ont your work, first there is a problem with the font you pick, it really didn't work well with the project, so I recommend you to pick one on google font and to use it. Just spend 5 minutes to pick a new font do all the difference. The other problem is the responsive of your page, the top background of your site does not take up the full width of my screen and it's really weird and if you reduce the size screen the withe rectangle also reduce and you don't want that.

    I hope that will help you ! :)

    Marked as helpful
    0
  • KOMA TOZ•110
    @Maksym-PaselskySubmitted over 1 year ago

    It was little tricky to make design responsible🧐. I used a lot of rem/em but than realise it was too much. I literaly didn't have any px values🥶 I'm quite confident about the code🧑‍💻. But I still far away from understanding best pratices for responsive design and mobile first approach⏳

    Any feedback is much appreciated same for likes and follows. Let's get in touch and learn together🎉.

    Responsive Summary Component

    1
    Antoine-thz•90
    @Antoine-thzPosted over 1 year ago

    Hi,

    I thin you can use more of display flex so your summary part will be more responsive and for less effort ( don't have to use position relative to each child of summary) try somethin like for your summary class :

    display: flex; flex-direction: column; /* align-content: center; */ align-items: center; justify-content: space-evenly; } And be carrefull to small details like the border radius of the summary left bottom corner they are easy to correct but have a big impact to the design.

    You can also try to do some javascript by automatically calculate and display the average rate.

    Good luck !

    Marked as helpful
    0
  • Antoine-thz•90
    @Antoine-thzPosted over 1 year ago

    Hi, If you want to aligne your text you can simply use : text-align: center;

    0