Clipboard Landing using HTML y CSS, responsive

Community feedback

• 

  Mahnoor Khan•455

  @mahnoork18Posted almost 3 years ago

  Hey, nice work there! I have noticed a few things that need to be corrected. firstly, the float property in CSS is outdated try using flexbox or grid for layout so that you can have fewer lines of code. also, you need to add alt text in your HTML. Hope this helps!

  Marked as helpful

  0

  ---

  

  Estefanie Peña•40

  @pbatista01Posted almost 3 years ago

  @mahnoork18 Hi Mahnoor, appreciate your advises. I will take them into account and use them to improve this project, and for future ones! Thank you so much for taking the time to reviewed and comment!

  0
• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted almost 3 years ago

  Greetings Estefanie.

  Congratulations for taking this challenge, so far so good. There are issues that have been raised in the automated response, which may include the following:

  • Use semantic html, elements such as main, header, footer etc improves the accessibility of your project that using divs.
  • interactive elements should not be nested, that is a button cannot be a parent of an anchor element or vice-versa. An anchor element is used for navigation purposes - moving to other sections or another page where as a button is associated with form submission, opening modals etc.
  • the social icons are not just images but links to social sites, hence they should be navigation links to the respectful handles. You need to research on how to make an svg accessible.
  • as mentioned by @mahnoork18, look into flexbox and grid, float is rarely used these days

  1