Submitted about 2 years ago
using flex-box was the best choice for completing this challenge to me
@Maryan121
Design comparison
SolutionDesign
Community feedback
- @johnny064Posted about 2 years ago
@Maryan121 Hello bro. Though you have not yet added the script file. It's hard to say. But I have looked at your code. Here is some issue I have found.
*** You should use the <h> by likewise like:
<h1><\h1> <h2><\h2> <h3><\h3> <h4><\h4> <h5><\h5>
*** You should use this format:
<header></header> <main></main> <footer></footer>
it will give your code more accessibility.
*** And please write comments to your code. All HTML, CSS, Javascript. So that anyone can understand your code easily.
Thanks If have any issue you can tell me here.
Marked as helpful0@Maryan121Posted about 2 years ago@johnny064 thank you for the feedback i appreciate it .
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