Huddle-Landing-Page.

Community feedback

• 

  proabre•190

  @proabrePosted over 1 year ago

  hello there saw your page and it looks berfect but i think you miss the active state ...how abt that

  1

  ---

  

  vivekrajput@93•450

  @vivekrajput-93Posted over 1 year ago

  @proabre Thank you ...for noticing that...I did recklesness while making this components...I did not notice the active-state..I will be carefull next time..thank you once again.

  0
• 

  Ahmed El_Bald•1,020

  @Ahmed-ElbaldPosted over 1 year ago

  Hi,

  • I've noticed that you haven't applied any of the hover effects in the page, so try to apply it. You can use the fill property with SVGs to do that.

  • Try to have a main element (only one) in your HTML to improve accessibility (instead of your container for instance), and don't nest it into any parent as it represents a landmark.

  • It's advisable to use semantic elements with meaningful identifiers. For instance, classes like left and right can be replaced by hero-page-img and hero-page-text

  0

  ---

  

  vivekrajput@93•450

  @vivekrajput-93Posted over 1 year ago

  @Ahmed-Elbald Thank you brother for noticing that...It is totally my fault that I did not notice the active state...I will be next time..and correct my mistake...thank you once again.

  0