Snigdha Sukun
@snigdha-sukunAll comments
- @waquintero-iaSubmitted 3 months ago@snigdha-sukunPosted about 23 hours ago
Your solution is amazing. It matches the design perfectly. Also, your code was easy to understand. I am a beginner & I learned a lot from your solution!
0 - @akramkumarSubmitted 10 days ago@snigdha-sukunPosted 9 days ago
Your solution is very good. The only thing missing is that we need to use 2 different fonts within the page.
To import both, try:
@import url('https://fonts.googleapis.com/css2?family=Outfit:[email protected]&family=Young+Serif&display=swap');
Then use it for the headings!
Marked as helpful0 - @S-AlifSubmitted 15 days ago@snigdha-sukunPosted 9 days ago
Your solution seems perfect. I learned a lot from your solution.
If I had to find some improvement area, I would suggest you to use
role=main
in the page to help with accessibility.0 - @Miguel03nxSubmitted 13 days agoWhat are you most proud of, and what would you do differently next time?
con la practica me fue un poco mas sencillo y rápido acabar este desafío
What challenges did you encounter, and how did you overcome them?poder hacer la letra mas robusta en ciertas partes del blog
What specific areas of your project would you like help with?como aprender a usar mejor el @media para poder hacer sitios responsive
@snigdha-sukunPosted 13 days agoYour overall solution looks good, specially the positioning of all html elements.
Here are a few things that might be helpful to make your solution better:
- The avatar image is going outside the box. To fix it, you should try using:
.container{ display: block; }
- add the hover effect for the heading
h3:hover { cursor: pointer; color: hsl(47, 88%, 63%); }
- You should use atleast 1
h1
in the page for accessibility
Marked as helpful1 - @AnjaniKavithaKadiyalaSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
I am proud of my self, that I cretaed the first application and deployed as live application. Though it is simple application I did it my first time. I took online refrence to complte this solution, from Next time onwords I will put my own efforts to complete the solution. While doing this soltuion I learned CSS stuff and GIT.
What challenges did you encounter, and how did you overcome them?I faced the difficulty while writing the CSS code. I couldn't able to figurout the exact matched solotion according to the desgin. so I took refrences from internet and try to replicate those.
What specific areas of your project would you like help with?CSS techiniques.
@snigdha-sukunPosted 3 months ago-
Semantic HTML:
The solution effectively uses semantic HTML, which is great to see. -
Accessibility:
To enhance accessibility, I recommend addingrole="main"
to the<div>
that contains the primary content. This will help screen readers better understand the structure of the page. -
Responsiveness:
Unfortunately, I was unable to assess how well the layout adapts to different screen sizes, as the live website link provided did not work for me. -
Code Quality:
The code is well-structured, readable, and designed with reusability in mind. Excellent work on this front. -
Design Consistency:
Similar to the responsiveness, I couldn't evaluate the design accuracy due to the live website link not working.
If you fix the issue with the live website link, I'd be happy to provide further feedback on the layout and design consistency. Overall, you're on the right track!
0 -