Design comparison
Solution retrospective
Hi, everyone! This time, I just tried to make it responsive without using any media queries. What do you think? I hope you're doing ok!
Community feedback
- @VCaramesPosted about 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
-
The purpose of the Main Element is to identify the main content of your page. It is not the container of you component. After the main element, you want add a container to wrap you separate components in.
-
The “illustration” and Music Icon serves no other purpose than to be decorative; they add no value. Their Alt Tag should left blank and have an aria-hidden=“true” to hide it from assistive technology.
-
The Article Elements are being used ** incorrectly**. You will want to replace them all with Divs.
If you have any questions or need further clarification, let me know.
Happy Coding! 👻🎃
0 -
- @abdullah43577Posted about 2 years ago
It looks nice, you've done a great job here. Keep it up👏👏
0
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