Skip to content
OverviewSolutions
2
Comments
3
BringsCoding
@BringsCoding

All comments

  • Tarun•140
    @XENO2410Submitted over 2 years ago

    Will improve more in future learning grid & importance of positioning point out some changes & ideas to improve for me thnks :)

    HTML CSS

    1
    BringsCoding•20
    @BringsCodingPosted over 2 years ago

    hey

    First of all, congratulations on taking a task and working on it!

    The container is not centered on the desktop Here I would margin-left:auto marign-right:auto take so it is in the middle :)

    Songs it is not mobile optimized Media queries are the magic word here

    otherwise is good!

    Keep it up and don't lose the fun even if it can be hard to develop at times

    1
  • BringsCoding•20
    @BringsCodingPosted over 2 years ago

    First of all, congratulations that you have set yourself a task and that you have finished it!

    Looks good, but on mobile I see the following Distance between navbar and hero image is too big

    The get start button has a border at the top when you press it, then only at the bottom

    The distance from the cards to the upper section is too much

    The cards are not centered At the very bottom the input field is unstyled

    Otherwise everything looks good 👍

    1
  • BringsCoding•20
    @BringsCodingPosted almost 3 years ago

    Good work!

    The only thing that is not perfect is that the button "Download for IOS" has no hover effect, and in the footer between 680 and 668 the Facbook icon covers the text.

    0