All comments

• 

  brammilevs•60

  @brammilevsSubmitted 7 days ago

  

  responsive landing of recipe card

  1

  10
  

  eugenederii•40

  @eugenederiiPosted 7 days ago

  1. Yes, your code uses some semantic elements like <main>, <h1>, <h3>, <ul>, <ol> and <table>. However, there are some points that could be improved:

  2. For a table, you can add column headings to improve accessibility for screen readers. Using <th> for headings will help.

  3. yes, but different from the design

  4. the code is poorly structured and not readable

  5. fundamentally different from the layout, almost no css was used

  0
• 

  josephXplorer•50

  @josephXplorerSubmitted 22 days ago

  

  Social Links Proof using flex-box

  1

  20
  

  eugenederii•40

  @eugenederiiPosted 8 days ago

  1. Heading Levels: Using <h1> for "Jessica Randall" is not ideal, as this is the user’s name and not the primary heading of the page. You should use a separate <h1> tag for the main page title, such as "Social Links Profile," outside the profile card. The user’s name can be an <h2> tag instead.

  2. Links: Instead of using <div class="website">GitHub</div>, you should use <a href="#" class="website">GitHub</a> to make these clickable and semantically indicate they are links.

  In general, yes, but it is possible to improve the names of the classes and bring out all the styles in CSS 5. -

  0
• 

  xbroindia•60

  @xbroindiaSubmitted 29 days ago

  

  Blog-preview-card

  1

  20
  

  eugenederii•40

  @eugenederiiPosted 8 days ago

  1. Semantic elements such as <article>, <header>, <footer> are missing in the code. It is recommended to replace <div class="Blog-Card"> with <article> and class="info" with <section> to improve the structure.

  2. Add alt text for author profile image (<img src="./assets/images/image-avatar.webp" alt="Avatar of Greg Hooper">) to improve accessibility.

  4.The .highlight class can be made more versatile for any highlighted text. 5Visually, the project looks similar to the given example.

  0
• 

  Yiftach-Yaakov•40

  @Yiftach-YaakovSubmitted 8 days ago

  

  QR code

  1

  20
  

  eugenederii•40

  @eugenederiiPosted 8 days ago

  1. There is a semantic error in this solution, for example, p is used instead of h1, it is better to use <article class="card"> instead of <div class="card">
  2. the alt is missing in the img
  3. omitted indents
  4. The code is well structured, readable
  5. yes

  0