Submitted over 2 years ago
Responsive Webpage | Interacts with API
@sachdevavaibhav
Design comparison
SolutionDesign
Solution retrospective
Please review the webpage and give some feedback.
Community feedback
- @RioCantrePosted over 2 years ago
Hello there! Awesome work with this project. Looking at your solution, I would like to suggest the following for you…
- Import the
attribution
style in CSS file and remove thestyle
tag - Instead of
div
, alternatively wrap the whole content with semantic tags, for HTML structures, refer it with this one Semantics. Like this line<div class="attribution">
can be wrap withfooter
tag or placed it inside themain
tag - Include description with the
alt
in image tags. Like this line<img src="images/pattern-divider-mobile.svg" alt="" srcset="" class="divider-svg">
- Clean the whitespaces in the code
- Try increasing the width size into
max-width: 30rem;
in.card-section
rule set
ADVICE #38
"The most delicious cocktails often have the highest alcohol content. Always pace yourself to preserve your dignity."
Above all, the design is well implemented. Keep up the good work!
Marked as helpful1@sachdevavaibhavPosted over 2 years ago@RioCantre Thanks for taking out the time and provide valuable feedback. Will surely work on your suggestions. Thank you!!
0 - Import the
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord