Design comparison
SolutionDesign
Solution retrospective
I've resubmitted this about five times because I couldn't figure out Github haha. Would love feedback on both the HTML and CSS. If I can refactor anything to be more simple, I'd love to know!
Community feedback
- @vanzasetiaPosted over 2 years ago
Hi there! š
Congratulations on finishing your challenge! š
Some suggestions.
- This is already simple enough. But, there are some things that need to fix.
- First, I recommend wrapping all the elements with a
main
tag. - After that, remove all the
div
. - Make the
main
element as the card element instead. - Then, I suggest putting the
img
element as the first child of themain
element followed by theh2
and thep
element. - Next, I recommend taking a look at the issues that have been generated and then trying to learn more about the issues.
- Finally, you can update the code in the repository by pushing the latest changes. Then, try to generate a new report.
- First, I recommend wrapping all the elements with a
- If you still see any issues then try to fix them. š
- I notice a
.DS_Store
inside theresources/
folder. This can be prevented by keeping the.gitignore
file both on your local machine and the GitHub repo. (For your information, by the time you downloaded the "Starter files", it already came with the.gitignore
file).
That's it! Let me know if you have any questions. Hope this helps. š
1@EngineerHamzieyPosted over 2 years ago@vanzasetia thanks for your regular helpful comments š¢ Seriously my eyes always ignore the learn more button. I'll instead be waiting for people to comment how I can solve those issues which is bad.
But I'll start using the learn more
1 - This is already simple enough. But, there are some things that need to fix.
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