@ZenitsuAg
Posted
Hello Nayan, how you doing today? Your code is amazing!! I like the animation, looks great, you can fix the issue you're facing in firefox with the piece of code.
- On line 8 of your
script.js
, you have this:
fetch('https://api.adviceslip.com/advice')
Update it to this
fetch('https://api.adviceslip.com/advice', { method: 'GET', mode: 'cors', cache: 'no-cache' })
The browser will stop retaining the old advice and give a new advice each time you ask.
-
It'll be better if all your code is inside a
landmark
tag likemain
footer
, you did this but you left out thebutton
's div. -
You can also center your content with
<body class="...other-classes flex justify-center items-center">
- The node_modules folder is not necessary in your repo, so you can remove it.
And that's all.
Happy Coding :)
Marked as helpful
@freaky4wrld
Posted
@ZenitsuAg I'm doing great buddy, and with your help it's even better than great..... once again thanks for your remarks and I appreciate your effort..... Btw.........how you doing pal??
@ZenitsuAg
Posted
@freaky4wrld I'm doing great! Thanks for asking. I'm so happy that my remarks were useful to you and I see that you have also made changes. You may have to redeploy your site so that we can see the changes in your live site.
Have a nice day!
Happy Coding :)