CSS グリッドを使用したレスポンシブランディングページ
Solution retrospective
Greetings
Hello there! 👋🌟
Request
Hey everyone from the Frontend Mentor community, could I ask for your help?
I'm seeking a code review. Do you have any advice?
Project Specification
Currently, I'm working on3-column-preview-card-component-main.
I welcome feedback from various perspectives.
Any suggestions or ideas are greatly appreciated! 🛠️
Points of Concern
In relation to this project, I don't have any specific points of concern,
but I'm open to any suggestions or ideas! 🤔
Appreciation
Your opinions and ideas are warmly welcomed!
Thank you in advance for your contributions! 🚀
Please log in to post a comment
Log in with GitHubCommunity feedback
- @Mitko90
Hi, nice solution.
It looks great. Pixel perfect
Maybe you can check some suggestions I have for you:
- You used three times the
<h1>heading. This heading should only be used once. Maybe change it to<h2>as it can be reused. Of course every page should have anh1heading so I suggest adding avisually-hiddenone to the page, so it doesn't generate an accessibility report. - Your buttons should direct the user to a different part or page of the site. So maybe switch the
<button>with the<a>tag.
I had the exact same mistakes when I did this challenge so I redid it with the changes. You can, if you want, check both of my submissions here and here.
Above all the solution you submitted looks great.
Happy coding.
Marked as helpful - You used three times the
Join 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