Design comparison
Solution retrospective
Its make with some basic knowledge and Pure HTML & CSS. and I'll try some more enhanced with media Queries (Maybe).
What challenges did you encounter, and how did you overcome them?None
What specific areas of your project would you like help with?None
Community feedback
- @JunbolPosted 7 months ago
Hi Gaurang! here are some recommendation for your project: PROS: Most of the features are there, it’s almost done.
CONS:
-
README.md: Remember to use the README.md template and adjust it (using Markdown syntax) with your own details and links in GitHub there is still the original README file. Also would be good if you show your final screenshot.
-
Screenshot: Your screenshot doesn’t match the original design, the drawing is out of place in the screenshot.
-
Alignment:
a. The alignment of the "Learning" element don’t follow the design brief given they should be centered 🐞 Reason: Forgot to add display: flex ✅🛠Solution: Add display: flex
but wait there are more issues 🔍 😱🚒🚒 .... I will download your code to test it. I am sorry fixing this will require more of my time. My advice to you is if you use Flexbox always add the beginning display: Flex and especially for this project use flex-direction: column for the rest follow the link bellow to study about using Flexbox. A healthy word of prevention: be careful with the property you use like position:relative and where you use because it could mess all the its children.
- Credits: The photo is missing: 🐞 Reason: missing 'dot' in path: <img src="/assets/images/image-avatar.webp" alt="Author profile"> ✅🛠Solution: Add the 'dot':
<img src="./assets/images/image-avatar.webp"
-
Footer: The footer is not there and it needs to be away from the Name and Photo of the presenter: You could add margin-bottom: 2rem on the .name class.
-
Box-shadow: The **box-shadow **property is not inline with the design brief (unless you wanted to change the design).
RECOMMENDATIONS:
- Semantic HTML: Try to use as much as you can semantic HTML in your index.html file this with readability , accessibility (AOM) (screen readers) (SEO). You could go here and check it out: (https://web.dev/learn/html/semantic-html) I went there again and I realized to add the ARIA ‘role’ attribute (in all my htmls 🤪
<main> <section> <h1>Main title</h1> <p>Paragraph </p> </section> </main> </body>
- Flexbox: I’m sure you use 3wschools to refresh stuff. But these two new kids on the block are really great:
- [Flexbox] (https://web.dev/learn/css/flexbox)
- [Flexbox] (https://www.joshwcomeau.com/css/interactive-guide-to-flexbox/) Good look and Keep going!
Cheers Junier
0 -
- @ImerzionPosted 7 months ago
Hi Guarang,
Looking at your solution to the challenge, there seems to be a few errors when comparing the solution to the design.
One of those issues being the colours that you have chosen do not match those given in the criteria.
There also appears to be an issue with one of your images not pulling through at the bottom of the card (probably due to an error with your folder structure).
Happy coding,
~ Imerzion
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