Design comparison
Community feedback
- @MargauxArmanetPosted 5 months ago
Hi there!
I’ve taken a look at your project, and I’m really impressed with your efforts. I have a few suggestions that might help improve it even further:
For index.html:
-
Consider using semantic tags: Using elements like <article>, <figure>, and <figcaption> can improve accessibility and SEO.
-
Setting default width and height for images: It can be helpful to set these attributes on your images to ensure they display correctly, even if your CSS doesn’t load for some reason.
-
Making the blog preview card clickable: If you wrap your .container-main in an a tag, it makes the whole card clickable. To style the title so it changes color on hover, you can try adding this CSS rule: a:hover h1 { color: yellow; }.
For main.css:
- Using a mobile-first approach: Especially for this challenge, it will make your life so much easier.
- Applying media queries: They allow you to style your site differently depending on the screen size.
These are just small ideas but I hope you find them useful!
0 -
- @JJ-codes-9Posted 5 months ago
Nice work, though I suggest to spend some time to fill out the read.me as it will be part of your work as you'll get more experienced. As a beginner, I try that too:)
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