uhllie• 30
@hithisisnick
Submitted
Manish
@manish2120
All comments
- Manish• 90
@manish2120
Posted
Nice one!
0 - Laszlo Somossy• 90
@StonyDigi
Submitted
Manish• 90@manish2120
Posted
I like the outcome of your project, but I think you should create a separate stylesheet because there is a lot of CSS included. Is that SCSS you’re using? I’ve never seen nested CSS before.
0 - Ali Raza• 20
@aliraza732-hub
Submitted
After spending a considerable amount of time, I finally created a social links profile using HTML and CSS. Can anyone provide feedback on how I can further improve it?
Manish• 90@manish2120
Posted
As i noticed : Regarding HTML
- Semantics are missing.
Regarding css
- media queries are applied for mobile devices as per they mentioned in there for 375px ig.
Overall its good, it quite looks similar :)
0 - Gobinath• 140
@gopinath-97
Submitted
What are you most proud of, and what would you do differently next time?
Done my third one for the day and used all my past learnings which I thought to forget, but with little of practice, all comes to place. Moving forward to build long single page sites.👩💻😊
Manish• 90@manish2120
Posted
-
i don't know much about semantics, but you can also try using
headerandfootertag here frontend mentor let you know about the landmarks in there report after completing the project. -
Overall its good 🙂, but yea media queries are missing and u can also try color finder extensions to get to know the color of cards as we know we dont have access of figma files after 2 projects so yea we can't make it same as per design 😓.
0 -
- Tran Thi Yen Nhi• 80
@Ttyn08
Submitted
Manish• 90@manish2120
Posted
Did you follow the figma design? it will help you to get the desired output.
Marked as helpful
0 - joyafrikaana• 70
@joyafrikaana
Submitted
What challenges did you encounter, and how did you overcome them?
I learned about using the CSS flex property and media query. When I used the CSS flex property alongside fixed width, my project wasn't centered, but as soon as I removed the fixed width, my project became centered. Also, my mobile version was wonky even though my codes were right. I realised the problem was the order. I had placed the mobile media query before the tablet media query. The solution was to put the mobile media query before the tablet media query.
What specific areas of your project would you like help with?
I currently need help in understanding the relationship between css flex property and fixed width.
Manish• 90@manish2120
Posted
I'm not good in semantics or styling, but here are some points I’ve noticed hope its correct :) :
Regarding HTML:
- You should use semantic tags like <main> and <footer> here to properly define your main content and content information.
- In your boilerplate code, it's better to place the `stylesheets before the closing
</head>tag.
Regarding CSS:
- Consider using min-height on the body or your container elements to ensure the minimum height of the parent element or body to covers the background and i guess it will help you here to center your content.
- Make sure to closely follow the Figma design. This will help you create an output that closely matches the design provided.
Overall, nice work! By the way, it's nice to see that you used an
<h1>tag and applied styling to it separately. I recently got to know about it that we should use heading tags according to their levels for semantic purpose.Marked as helpful
0
