P
@gkilasoniaSubmitted 3 months ago
Latest solutions
Latest comments
- @manish2120Posted 3 months ago
Woah! The size looks perfect. However, there seems to be an issue with the quote icon SVG:
- It looks like the size applied to the author's profile picture is also being applied to the SVG. It's better to use a separate class for the images and SVG where it differs.
- The box-shadow is missing.
- Avoid using px as it sets a fixed size for elements. Instead, use rem or em, whichever works best for you.
0 - @RamirezJordanSubmitted 10 months ago
- @hithisisnickSubmitted over 1 year ago
- @StonyDigiSubmitted about 1 year ago@manish2120Posted 5 months ago
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 - @aliraza732-hubSubmitted about 1 year ago@manish2120Posted 6 months ago
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 - @gopinath-97Submitted 6 months agoWhat 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.👩💻😊
@manish2120Posted 6 months ago-
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 -
