Skip to content
OverviewSolutions
13
Comments
29
R A H U L
@owlbunt

All comments

  • R A H U L•360
    @owlbuntPosted 9 months ago

    Everything seems fine you can center the element by adding display :grid and pplace-items:center to body .

    1
  • R A H U L•360
    @owlbuntPosted 10 months ago

    Congratulations On Completing this challenge,

    Here are some improvements you can make like Insted of using 100vh use 100dvh to make your website more responsive.

    and you can delete all the unnecessary files which is not required after the challenge to keep your code and GitHub clean like the design folder .

    Mark This as Helpful If It's Help

    Marked as helpful
    0
  • R A H U L•360
    @owlbuntPosted 10 months ago

    There are some improvements you can make like Insted of using 100vh use 100dvh to make your website more responsive.

    you can delete all the unnecessary files which is not required after the challenge to keep your code and GitHub clean like the design folder .

    you can add a js function which allows you to open a single question at a time close the others for reference you can checkout my code.

    Mark This as Helpful If It's Help

    Marked as helpful
    0
  • R A H U L•360
    @owlbuntPosted 10 months ago

    There are some improvements you can make like Insted of using 100vh use 100dvh to make your website more responsive. and you can delete all the unnecessary files which is not required after the challenge to keep your code and GitHub clean like the design folder .

    Mark This as Helpful If It's Help

    Marked as helpful
    1
  • R A H U L•360
    @owlbuntPosted 10 months ago

    There are some improvements you can make like Insted of using 100vh use 100dvh to make your website more responsive. And to center a <div> you can also use display:grid; place-items: center; . and last you can delete all the unnecessary files which is not required after the challenge to keep your code and GitHub clean like the design folder .

    Mark This as Helpful If It's Help

    Marked as helpful
    0
  • R A H U L•360
    @owlbuntPosted 10 months ago

    Design Seems Good And Ig We can reduce the use of js where things can be done using html and css only to increase the page speed which also effect in SEO and also you don't have to add fonts additionally there is an font folder which includes required fonts in assets folder.

    0
  • R A H U L•360
    @owlbuntPosted 10 months ago

    Designs are not similar at all try again and check the design folder and try if you find any difficulty ask me anytime. 😊

    0
  • R A H U L•360
    @owlbuntPosted 10 months ago

    Ah, nice try! Please make it responsive for mobile devices as well. Additionally, center it by using display: grid; place-items: center; with 100dvh and 100dvw on the body.

    0
  • R A H U L•360
    @owlbuntPosted 10 months ago

    Everything Is fine Just make it little bigger 🤏

    0
  • Ben•20
    @GudelisSubmitted 10 months ago

    First time doing accordion any feedback will be appreciated

    FAQ Accordion

    1
    R A H U L•360
    @owlbuntPosted 10 months ago

    Everything is fine and i checked that you added a function which allows to open only question at a time and that's good only make it responsive for mobile device otherwise perfect. 🤗

    Marked as helpful
    0
  • R A H U L•360
    @owlbuntPosted 10 months ago

    Congratulations 🎉 on completing this challange.

    I have some suggestions for you 📌

    1. Always use dvh and dvw instead lf vh and vw for height and width property like this 100vh > 100dvh

    2. Delete Unnecessary Files After Completing the challenge like design folder to keep your GitHub clean

    1
  • R A H U L•360
    @owlbuntPosted 10 months ago

    Use dvh and dvw instead of vh and vw to make more responsive

    Marked as helpful
    0