All comments

• 

  Carmen•490

  @CarmenyoSubmitted over 2 years ago

  All feedback is welcome. Many thanks!

  

  Order summary component

  3

  40
  

  Lesego•95

  @lesego16Posted over 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

  @oyeyinkaojoraSubmitted over 2 years ago

  Any suggestion on how I can improve this?

  

  Profile component

  1

  40
  

  Lesego•95

  @lesego16Posted over 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

  @ImmuneMoonSubmitted about 3 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

  20
  

  Lesego•95

  @lesego16Posted about 3 years ago

  1. Try fixing ACCESSIBILITY ISSUES and HTML ISSUES

  0

  This comment has received 1 replyShow
• 

  Daniel Veluswamy•10

  @danjayasSubmitted about 3 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

  20
  

  Lesego•95

  @lesego16Posted about 3 years ago

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

  0
• 

  Kerolous Amged•130

  @Kokosalah45Submitted about 3 years ago

  approaches to apply the background svgs ty

  

  Order summary component challenge hub

  2

  20
  

  Lesego•95

  @lesego16Posted about 3 years ago

  1. try to fix accessibility issues and html issues

  0
• 

  Taha•310

  @thaykrglSubmitted about 3 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

  20
  

  Lesego•95

  @lesego16Posted about 3 years ago

  2. 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

  @thaykrglSubmitted about 3 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

  20
  

  Lesego•95

  @lesego16Posted about 3 years ago

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

  1

  This comment has received 1 replyShow
• 

  Asbyx•10

  @AsbyxSubmitted about 3 years ago

  If I made classic mistakes be free to point them.

  

  Order summary component using html css

  1

  10
  

  Lesego•95

  @lesego16Posted about 3 years ago

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

  0
• 

  Lucas Avendaño•90

  @LucasAvenSubmitted about 3 years ago

  Any feedback is appreciated!

  

  3 Colums Responsive Cards with Grid

  1

  10
  

  Lesego•95

  @lesego16Posted about 3 years ago

  1. try to rectify width > 765

  1

  This comment has received 5 repliesShow
• 

  Marina•35

  @TulipSoupSubmitted about 3 years ago

  Hi there! It's my first challenge :D

  

  Component CSS Flexbox - Updated

  2

  30
  

  Lesego•95

  @lesego16Posted about 3 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

  @lesego16Submitted over 3 years ago

  feedback will be highly appreciated thank you.

  

  Insure Landing page using HTML and CSS

  2

  31
  

  Lesego•95

  @lesego16Posted over 3 years ago

  Thank you, I will try to correct it

  1