Skip to content
OverviewSolutions
50
Comments
5

Yassine Ghezal

@Ryusuke66Algeria870 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

I’m currently learning...

Typescript, React.js

Latest solutions

View more solutions

Latest comments

  • Yassine Ghezal•870
    @Ryusuke66Posted over 1 year ago

    Congrats on finishing the challenge your solution looks great, one thing i found is when i reload the website the advice changes twice sometimes, what causes that is this code in your Javascript file:

    window.onload = () => {
    getAdvice();
}

    it's not needed since the API changes the advice every 2 seconds, so you can remove it and it will fix the issue. Good luck and happy coding!

    Marked as helpful
    0
  • Yassine Ghezal•870
    @Ryusuke66Posted over 1 year ago

    Hi Hamid, your solution looks great, but there's a little issue, when the screen width is between 741px and 809px the page overflows sideways, to fix that you can change .navLink padding to padding: 15px; instead of padding: 15px 30px, Happy coding!

    Marked as helpful
    0
  • Yassine Ghezal•870
    @Ryusuke66Posted over 3 years ago
    • the background isn't the same as the design.
    • using header image as a background is better than putting it as an element.
    • the font family for the design is "Red Hat Display".
    Marked as helpful
    1
  • Yassine Ghezal•870
    @Ryusuke66Posted over 3 years ago
    • Looks good, the issue is that from 500px to 900px the design is overflowing, you have to scroll sideways to see the rest.
    Marked as helpful
    0
  • Yassine Ghezal•870
    @Ryusuke66Posted over 3 years ago
    • Good work, you should add background-size: contain and change the background-repeat property to no-repeat in the body element.
    Marked as helpful
    1