Latest solutions
Latest comments
- @Joshdesign@HectorMartinDama
Hello @Joshdesign the desing is very similiar to the original and the mobile version is very good. The media querys are correct. To be your first proyect in frontendmentor is great.
Marked as helpful - @alexgabriel8@HectorMartinDama
Hello @alexgabriel8, in your request to the api use this:
fetch('https://api.adviceslip.com/advice/' + Math.floor(Math.random() * 200))
The random number is because if you do not indicate a specific advice, it does not work correctly, in your project change the advice when you click in the button five times
Marked as helpful - @Lohuama@HectorMartinDama
Hello @Lohuama, change the property "src" for this:
src="./images/image-qr-code.png"
You are missing the point, this indicates the main folder of the project.
- @CrypticMango@HectorMartinDama
Hello @CrypticMango, your file js is good but I would change the api request into a function. That way you don't need to reload the page every time you click the button.
File js
function getApi(){ fetch('https://api.adviceslip.com/advice/' + Math.floor(Math.random() * 200)) .then(res => res.json()) .then(res => { document.getElementById("phrase").innerHTML= '"' + res.slip.advice + '"' document.getElementById("id").innerHTML= 'ADVICE #'+ res.slip.id }) .catch(err => console.log(err)) }
The random number is because if you do not indicate a specific advice, it does not work correctly.
index.html
<button onclick="getApi()"></button>
Call the function in the button
For the rest it is perfect.
Marked as helpful - @HectorMartinDama@HectorMartinDama
Any feedback is welcome.
Cualquier feedback es bienvenido.
- @drEbison@HectorMartinDama
Perfect 🤗