A perfect challenge for beginners

Community feedback

• 

  PhoenixDev22•16,830

  @PhoenixDev22

  Posted almost 3 years ago

  Hi Mohamed Khaled,

  Another great solution. I have some suggestions regarding your solution:

  • The alternate text of the logo should not be Main Logo, it may set alt=”Huddle". Use the website's name as an alternate text.Remember that a website-logo is one of the most meaningful images on a site so use proper alt for it.

  • For social links class="social", the icons are decorative, so you should add aria-hidden=”true” to the icons, to be ignored by screen readers and to avoid redundancy and repetition.

  • Adding rel="noopener" or rel="noreferrer" totarget="_blank"links. When you link to a page on another site using target=”_blank” attribute , you can expose your site to performance and security issues.

  Hopefully this feedback helps.

  12

  Marked as helpful
• 

  David•7,960

  @DavidMorgade

  Posted almost 3 years ago

  Hello Mohamed, congrats on finishing the challenge, pretty good job and it looks that your page is fully responsive, great!

  If you don't mind I would like to give you some feedback.

  It would make your application look smoother and more beautifull if you add some transitions on your buttons and social media icons when hovering, you could add something like transition: all 0.5s ease;, remember to add them inside the class and not in the :hover pseudoclass, I think this is just a little change but will be a boost for the user experience.

  Also would recommend you to use more semantic tags like section or article instead of just dividing all of your main with divs, it will boost the accesibility of your page and also the search engines will position your web better.

  Hope my feedback helps you, great job and keep it going, almost pixel perfect!

  11

  Marked as helpful
• 

  Adam M•550

  @AdamMzkr

  Posted almost 3 years ago

  Good Job but, in mobile view your design as a little bit to big. To see all content you need to scroll horizontally. I think change width container to 90% will help.

  11

  Marked as helpful