Skip to content
OverviewSolutions
9
Comments
8
Alieu
@alieuk61

All comments

  • Alieu•190
    @alieuk61Posted about 2 years ago

    Also I just realised that I made a few mistakes with the colours of some things...

    0
  • Alieu•190
    @alieuk61Posted over 2 years ago

    also can someone please let me know if the actual website looks different/the same to solution displayed in the design comparison, because to me the solution that i see looks worse than what i see when i click on the website.

    0
  • Alieu•190
    @alieuk61Posted over 2 years ago

    looks great!

    Marked as helpful
    0
  • Alieu•190
    @alieuk61Posted over 2 years ago

    it looks very good, but I think maybe if you made the body have box-sizing: border box, it will allow you to add paddings to the body without allowing it to get bigger in width and height. also in my opinion I think its good practice to have the height have min-height: 100vh; and have the width have max-width: 100vw. Hope this helps and can you also give me some feedback on my page thanks :)

    Marked as helpful
    0
  • Alieu•190
    @alieuk61Posted over 2 years ago

    It's ok, we're all trying to get better, your css isn't showing as Chris said however when you fix it I can come back and try to give some help :)

    0
  • Alieu•190
    @alieuk61Posted over 2 years ago

    This looks so good, literally a replica of the design. Well done :)

    0
  • Alieu•190
    @alieuk61Posted over 2 years ago

    I put the prices in separate divs within a container div and then used margins to try and vertically align them, you can also use flex box to try and do that as well, I hope this helps :)

    0
  • Alieu•190
    @alieuk61Posted over 2 years ago

    Desktop design looks great but the mobile design is supposed to be different, also can you please put the correct link to your code? because when I click it, it comes up with page not found. :)

    1