Skip to content
OverviewSolutions
5
Comments
4
Rihantiana
@LegendaryGhost

All comments

  • Rihantiana•70
    @LegendaryGhostPosted 3 months ago

    Great job ! 🎉 Everything is super accurate but the whole page doesn't appear on screens with small height because you've set the main wrapper's height to 100vh. I'm not sure why you did it but I think you should remove this line :

    main {
  /* ... */
  height: 100vh;
}
    0
  • Rihantiana•70
    @LegendaryGhostPosted 3 months ago

    Great job! 🎉

    It looks prefect but why didn't you write your info instead ?

    0
  • Rihantiana•70
    @LegendaryGhostPosted 3 months ago

    Your screenshot preview of the is pretty close to it but when I opened the link to the live site, it seemed like the CSS file hasn't be loaded.

    Avoid writing "/" in front of the relative path because it makes the server look for the file at its root folder instead of the folder where the file containing the URL is located

    In summary, you should change this:

    <link rel="stylesheet" type="text/css" href="/style.css" />

    into this in order to make it work on the server:

    <link rel="stylesheet" type="text/css" href="style.css" />
    0
  • Rihantiana•70
    @LegendaryGhostPosted 3 months ago

    Great job 🎉

    Some of the padding/margin values are different from the figma design but the whole thing is pretty accurate

    0