R A H U L
@owlbuntAll comments
- @omlan99@owlbunt
Everything seems fine you can center the element by adding display :grid and pplace-items:center to body .
- @eldon6219@owlbunt
Congratulations On Completing this challenge,
Here are some improvements you can make like Insted of using
100vh
use100dvh
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 - @addiellucena94@owlbunt
There are some improvements you can make like Insted of using
100vh
use100dvh
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 - @gmenesesdev@owlbunt
There are some improvements you can make like Insted of using
100vh
use100dvh
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 thedesign
folder .Mark This as Helpful If It's Help
Marked as helpful - @xiAvalon@owlbunt
There are some improvements you can make like Insted of using
100vh
use100dvh
to make your website more responsive. And to center a <div> you can also usedisplay: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 thedesign
folder .Mark This as Helpful If It's Help
Marked as helpful - @asaga22@owlbunt
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. - @iamthulasinath@owlbunt
Designs are not similar at all try again and check the design folder and try if you find any difficulty ask me anytime. 😊
- @varadpal@owlbunt
Ah, nice try! Please make it responsive for mobile devices as well. Additionally, center it by using
display: grid; place-items: center;
with100dvh
and100dvw
on the body. - @jcfrancisco17@owlbunt
Everything Is fine Just make it little bigger 🤏
- @Gudelis@owlbunt
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 - @birdbirdmurmur@owlbunt
Congratulations 🎉 on completing this challange.
I have some suggestions for you 📌
-
Always use dvh and dvw instead lf vh and vw for height and width property like this
100vh > 100dvh
-
Delete Unnecessary Files After Completing the challenge like
design folder
to keep your GitHub clean
-
- @ironest
- @Kseylo@owlbunt
Seems perfect just use 100dvh instead of 100vh to make it more responsive also if you like to share how you add those border in corners only. thanks
- @DarkGamerXDOFF@owlbunt
Well , you don't need to make pixel perfect it's all about learning here so do your best and keep going 😉
- @umutcankocamis@owlbunt
You did well if you improve it little bit it would be better otherwise perfect it's all about learning here 😉 . tip : Use 100dvh instead of 100vh to make it more responsive
- @Luiz0tavi0@owlbunt
It seems like you didn't complete this challenge. Did you get stuck somewhere? or need any help?
- @CaceresCallieri@owlbunt
you did it perfectly but there are some issues you can improve, total score circle is not complete circle and also use 100dvh instead of 100vh same goes to width this will make your page more responsive.
- @Sileniz@owlbunt
You can try to implement a js function which allows only 1one question to be opened at a time so when you open another question it disable the previous one in this way you can make more responsive websites and save spaces.