All comments

• 

  Carmen• 490

  @Carmenyo

  Submitted about 2 years ago

  All feedback is welcome. Many thanks!

  

  Order summary component

  3

  4

  0

  

  Lesego• 95

  @lesego16

  Posted about 2 years ago

  Try to fix accessibility issues:

  1.your heading should follow h1 - h6

  2.wrap everything inside a <main></main> tag after the body tag

  3.add an aria-label attribute to your <a> tags

  4.add a role attribute to your div classes

  Marked as helpful

  0
• 

  Oyeyinka Ojora• 170

  @oyeyinkaojora

  Submitted about 2 years ago

  Any suggestion on how I can improve this?

  

  Profile component

  1

  4

  0

  

  Lesego• 95

  @lesego16

  Posted about 2 years ago

  Try fixing accessibility issues:

  1.Your heading should begin from h1 - h6

  2.add aria-label attribute to your links especially <a> tags

  0

  This comment has received 1 replyShow
• 

  Alex• 95

  @ImmuneMoon

  Submitted over 2 years ago

  New to HTML/CSS, I'm sure this is probably messy and there are better ways to do this. Still working on the accessibility/ HTML issues, if you have suggestions I'd love to hear them, thanks!

  

  Huddle responsive landing page with flexbox

  2

  2

  0

  

  Lesego• 95

  @lesego16

  Posted over 2 years ago

  1. Try fixing ACCESSIBILITY ISSUES and HTML ISSUES

  0

  This comment has received 1 replyShow
• 

  Daniel Veluswamy• 10

  @danjayas

  Submitted over 2 years ago

  My first project. I am sure it's not 100%, but I learned a lot. Thank you, Frontend Mentors. I would love your feedback.

  

  Only an Atom code editor

  1

  2

  0

  

  Lesego• 95

  @lesego16

  Posted over 2 years ago

  1. Well done!! everything seems to look good, but try to fix accessibility issues

  0
• 

  Kerolous Amged• 130

  @Kokosalah45

  Submitted over 2 years ago

  approaches to apply the background svgs ty

  

  Order summary component challenge hub

  2

  2

  0

  

  Lesego• 95

  @lesego16

  Posted over 2 years ago

  1. try to fix accessibility issues and html issues

  0
• 

  Taha• 310

  @thaykrgl

  Submitted over 2 years ago

  I would be very happy if you could help me with the parts I need to improve. Thank you for your feedback.

  

  Profile Card Component

  2

  2

  0

  

  Lesego• 95

  @lesego16

  Posted over 2 years ago

  1. Have a look at your media queries because when I change to different widths it seems to break.

  0

  This comment has received 3 repliesShow
• 

  Taha• 310

  @thaykrgl

  Submitted over 2 years ago

  I would be very happy if you could help me with the parts I need to improve. Thank you for your feedback.

  

  Profile Card Component

  2

  2

  0

  

  Lesego• 95

  @lesego16

  Posted over 2 years ago

  1. Well done!! everything looks good but try to fix accessibility issues

  1

  This comment has received 1 replyShow
• 

  Asbyx• 10

  @Asbyx

  Submitted over 2 years ago

  If I made classic mistakes be free to point them.

  

  Order summary component using html css

  1

  1

  0

  

  Lesego• 95

  @lesego16

  Posted over 2 years ago

  1.Well done!! everything looks good but try fixing accessibility issues.

  0
• 

  Lucas Avendaño• 90

  @LucasAven

  Submitted over 2 years ago

  Any feedback is appreciated!

  

  3 Colums Responsive Cards with Grid

  1

  1

  0

  

  Lesego• 95

  @lesego16

  Posted over 2 years ago

  1. try to rectify width > 765

  1

  This comment has received 5 repliesShow
• 

  Marina• 35

  @TulipSoup

  Submitted over 2 years ago

  Hi there! It's my first challenge :D

  

  Component CSS Flexbox - Updated

  2

  3

  0

  

  Lesego• 95

  @lesego16

  Posted over 2 years ago

  1. Learn "flex-box" and then use it on the plan container.

  Marked as helpful

  2

  This comment has received 1 replyShow
• 

  Lesego• 95

  @lesego16

  Submitted almost 3 years ago

  feedback will be highly appreciated thank you.

  

  Insure Landing page using HTML and CSS

  2

  3

  1

  

  Lesego• 95

  @lesego16

  Posted almost 3 years ago

  Thank you, I will try to correct it

  1