Design comparison
Solution retrospective
How can I improve the way I code?
Community feedback
- @devjhexPosted over 1 year ago
@judgemongcal Congratulations on completing this challenge, it really shows that you put a lot of effort into it, keep it up and keep on practicing. On the side of suggestions though I just wanted to talk about the position of the component itself which could be centered more like the design by adding the following lines to your body styles.
body{ padding: 0; font-family: 'Outfit', sans-serif; background-color: #0D192C; /*Add the following lines to your code*/ displaty:flex; justify-content:center; align-items:center; flex-direction:column; height:100vh; }
Otherwise to be honest, You really did a great job. Happy coding👌👌👌
0 - @Kamlesh0007Posted over 1 year ago
Congratulations on completing the challenge🎉! That's a great achievement, and I'm sure you put a lot of effort into it. I really liked the way you approached the challenge and the code you wrote. You demonstrated a good understanding of the concepts and applied them effectively to solve the problem.I have a few suggestions to improve your code further. You need to add Semantic HTML tags to your code
The HTML structure should be semantically correct. The current structure doesn't provide any semantic information about the content. A better approach would be to use appropriate HTML tags such as <header>, <main>, <section>, <article>, etc. to define the sections of the page.<main> tag to indicate that it represents the main content of the page. You can wrap up .attribution class within <footer> as a part of semantics.Wrapping the .attribution class within a <footer> element is a great way to improve the semantics and structure of your code. here is the code that u need to add to include semantic tags in ur page
<main> <div class="card-div"> ...... </div> </main> <footer> <div class="attribution"> Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>. Coded by <a href="#">Judge Mongcal</a>. </div> </footer>
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