Skip to content
OverviewSolutions
9
Comments
3
P
Ryan Abney
@RCAbney

All comments

  • Ryan Abney•170
    @RCAbneyPosted over 2 years ago

    Where are you supposed to place an H1? The design didn't have a logical place for one.

    0
  • Ryan Abney•170
    @RCAbneyPosted almost 5 years ago

    ugh formatting problems with the comment sorry - try & # 8 2 2 6 ; remove the spaces

    0
  • Felix Anducho•400
    @PleopleqSubmitted almost 5 years ago

    I did the toggle function with JS. I dont know if it is possible to make it with CSS. Also, i dont know why, but when i open my page from Github the background is gone. It works with live server (VSC).

    Pricing component.

    3
    Ryan Abney•170
    @RCAbneyPosted almost 5 years ago

    @Pleopleq Nice job on this project.

    I see in the network tab of my console that the bg images are returning a 404 error. Try adjusting the url on your background-image rule to: url("../images/bg-top.svg") (along with the others adjusted as necessary) and see if that helps you out.

    One piece of feedback for your CSS: Your initial media query of min-width: 375px isn't necessary. I believe your intent is to have everything inside that as your global styles. As is, any browser width less than 375px (Samsung Galaxy S8/9 - 360px) won't include them.

    2