Design comparison
Solution retrospective
I know Html & CSS, although not very well. I would appreciate if you review my codes and give feedback. Thank you very much in advance.
Community feedback
- @cyberspatialPosted about 3 years ago
Solution is implemented very well. Further I would like to add:
-
You have used section inside a div. instead u can use div. Section is used to separate the content on the page, But here you are using it to separate the content inside a div , semantically it can be improved by replacing section with div.
-
U have used heading even for a rate and annual plan. Instead of using <h4> for annual plan and rate u can use <span> or <p>.
-
You can remove the accessibility issues by changing <div class="container"> to <main class="container"> and <div class="attribution"> to <footer class="attribution">.
Good luck!
Marked as helpful0 -
- @afrusselPosted about 3 years ago
Good work. I would like to request you please take a look at my solution it may help you to find your issues
1 - @ChesarfieldPosted about 3 years ago
Thank you for your return. I will implement your recommendations. I wish you good work :)
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