This challenge is very good to understand how display grid and display flex works. Especially knowing how to separate the sections of your layout. A very fun challenge for us beginners! I am open to hearing new solutions and feedback!
Hi @davidFreelance19,
Your solution is awesome 👌
Tip to avoid warnings in the accessibility report :
1.Wrap all the content in the <main> tag so you couldn't get the lanmark warnings
I have seen all your solutions are nice. Can we work together?
This is my first solution to try in this platform I'm beginner in web dev I would like to know if I did it according to the challenge, I'll appreciate your feedback.
Hi @MarwaanAbdalla,
Welcome to Frontend mentor community, Your first solution is awesome and keep it up.
Some advices
1.Wrap all the content in the <main> tag so you couldn't get an landmark error.
2.Use the alt attributes for images
Hello, I have a question:
Hey @taldr27,
Your Solution is really good!
1.Use atleast one <h1> tag so you couldn't get the warning.
Hi @sbetav,
Your Solution is really good!
Tips :
1.Wrap all the content in the <main> tag so you couldn't get an landmark error
2.Use atleast one <h1> tag
Thanks!
Thank you for looking.
Hi @NTWayys,
Your Solution is Great 👍 Some Tips,
1.Wrap all the content in the <main> tag so you couldn't get an landmark error.
Hey @yabeshyj
Welcome to Frontend mentor community,
Your Solution is great👍 :
Wrap the all the content in the <main>
Every comment is helpful for me
Hi Mohamed, Your Solution is looks Good!
Some of my suggestions are :
1.Change the background color of body to hsl(212, 45%, 89%) because this color is in the design
2.Wrap all your content in <main> tag, so you couldn't get any landmark errors
I hope this will helpful to you, Thanks!
Hi there, I’m Melvin and this is my solution for this challenge. 👋
Any feedback on how I can improve and reduce unnecessary code is welcome!
Thank you. :)
Hi @MelvinAguilar,
Your solution is really awesome 🎉 but one thing box shadow for dice button is showing only in bottom but in the design it should be all around the dice add this box-shadow:0 0px 35px 1px #52ffa8; CSS box-shadow code to dice button it will make box-shadow all around the dice button
Hi @abdulrehmandev,
Your solution looks Good! Some of My suggestions,
1.Change Background color white to #fafafa because this color is mentioned in design
2.In learn more button It seems your adding a border when hovering just add a border to learn more button and make border color to #141414 when hovering change the border color to #fafafa
3.Reduce the padding size of the learn more button
4.Give more padding top to the header
That's All!
Hello, Frontend Mentor community! Here is my solution to the Advice generator app. Happy to hear any feedback and advice 👋🏻😉
Hello @AndreaDev14,
Your is solution really looks great 👍 and only thing you missed is box-shadow when hovering the dice.
Add this CSS code to the dice button box-shadow:0 0px 35px 1px #52ffa8;
Any advice is accepted.
Hi @Cachilox,
It looks good, but one advice change the background color white to #fafafa.
Hi @tascintra,
Your Solution is really great👍 Make the background color to #fafafa because design contains this color as bg color for a body.
