@herieduSubmitted 5 months ago
What specific areas of your project would you like help with?
I would like to know about good naming practices in the case of grid-template.
Latest comments
- @devchsykasPosted 4 months ago
Hello my friend, below is my suggestions for your solution:
- Consider using the figure and figcaption tags for each testimonial card. This would enhance semantics by clearly associating the image (figure) with the text content (figcaption).
- Improve the alt text for profile images. Instead of "avatar," use something more descriptive like "Daniel Clifford's profile picture."
0 - P@Richard1876Submitted 4 months agoWhat specific areas of your project would you like help with?
Feedback is welcome
@devchsykasPosted 4 months agoHello Richard1876,
Suggestions for Improvement
- Improve semantics: Use headings (<h1>, <h2>, etc.) for better hierarchy. Add meaningful alt attributes to images.
- Enhance responsiveness: Utilize CSS grid or flexbox for a two-column or staggered layout on larger screens.
- Refactor repetitive CSS: Extract shared styles into reusable classes to reduce duplication.
0 - @devandytechSubmitted 9 months agoWhat are you most proud of, and what would you do differently next time?
Well am mostly proud of how I use the different images given for the mobile and desktop, and I noticed how start they implement the image not to work on css background image property because of the image storage, and am trying to do the project faster next time
What challenges did you encounter, and how did you overcome them?Using the images for the mobile and desktop, I tackle it using media queries and css display property
What specific areas of your project would you like help with?Any review and feedback is welcome
@devchsykasPosted 9 months agoHi, here are my recomendations for the improvement of your soloution:
- Consider using a more semantic structure. For example, wrapping the content in a <main> tag for the primary content of the page, and using a <header> tag for the logo and heading.
- Consider using modern layout techniques like CSS Grid or Flexbox to make the layout more flexible and easier to manage across different screen sizes.
Marked as helpful0 - @Reza1878Submitted 9 months ago@devchsykasPosted 9 months ago
Hi, here are some suggestions from me
- Ensure you use appropriate HTML elements. For example, use <header> for the top section, <section> for major sections, and <footer> for the footer.
Marked as helpful0 - @noormuhammadrazaSubmitted 9 months agoWhat challenges did you encounter, and how did you overcome them?
I am facing difficulty in creating exact size of the overall layout as well as the containing elements as per the designs.
In the last two challenges I got everything smaller than the design. And in this challenge I think I have made it larger than the design.
I have tried using fixed units as well for the layouts but that also does not seem to work
What specific areas of your project would you like help with?Kindly guide me on how to get the exact size of the overall layout and elements converted to code.
Sometimes I make the layout larger than the design and sometimes smaller.
@devchsykasPosted 9 months agoHi, here are some of my suggestions for further improvement:
1)Use more semantic elements to improve the readability and accessibility of your HTML 2)Ensure that social media links are actual hyperlinks, providing proper navigation and improving user experience.
Marked as helpful0 - @Shreyas235Submitted 9 months ago@devchsykasPosted 9 months ago
Hi, nice work your HTML structure is clean and follows a logical flow. My suggestions to improve semantics and accessibility.
-
Use of Semantic Tags: Replace the <div class="container"> with a <main> tag to indicate the main content of the page. Use an <article> tag for the blog post content to provide semantic meaning.
-
Image Elements: Ensure that the alt attributes for images provide descriptive text. This is essential for accessibility.
Marked as helpful1 -
