All comments

• 

  Abhiram•150

  @abhiram-s21Submitted 5 months ago

  

  blog preview card

  1

  20
  

  Francisco Antonio•60

  @fcooantonioPosted 5 months ago

  Overall layout

  The layout is great, just a few adjustments to the sizes of the components and it will be perfect.

  • Good use of semantic main and footer tags;
  • Good use of the alt argument in images for accessibility;
  • Use media queries to adapt the layout for the mobile version;
  • The use of media queries is appropriate and will help to ensure that the layout adapts well to different screen sizes;

  Additional suggestions

  • Consider using a relative font size for headings to ensure that they scale well on different screen sizes

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  Minerva-xx•60

  @Minerva-xxSubmitted 5 months ago

  

  QR component

  1

  30
  

  Francisco Antonio•60

  @fcooantonioPosted 5 months ago

  Overall:

  • Accessibility: It's great that you're using alt text to describe images! This helps to make your site more accessible to people who use screen readers.
  • Responsiveness: The page currently doesn't adapt well to different screen sizes. Using media queries would help ensure a good user experience across devices.

  Specific Suggestions:

  • Semantic HTML: Consider using semantic HTML tags like <main> to define the main content area of your page. This improves the readability and accessibility of your code.

  • Flexbox: Using Flexbox for layout can help you easily organize and center the elements on your page. It's a flexible and modern layout approach.

  Additional Feedback:

  • Media Queries: Implement media queries to adjust the layout and styling of your page based on different screen sizes. This will make your site responsive and user-friendly on various devices (desktops, tablets, phones).

  Marked as helpful

  0