Manage landing page using HTML CSS Vanilla JavaScript

Community feedback

• 

  Adriano•38,040

  @AdrianoEscarabotePosted about 2 years ago

  Hi David Pokrajac, how are you?

  I really liked the result of your project, but I have some tips that I think you will enjoy:

  • images must have alt text unless it is a decorative image, for any decorative image each IMG tag must have empty alt="" and add aria-hidden="true" attributes to make all the assistive technologies of the Web , the screen readers.
  • Heading levels should only increase by one so follow the sequence h1-h5

  The rest is great!

  I hope it helps... 👍

  Marked as helpful

  1

  ---

  

  David Pokrajac•370

  @DavidPokrajacPosted about 2 years ago

  @AdrianoEscarabote Hey, thank you very much for your help!!! I'm glad you like the overall solution

  0
• 

  Adhran•110

  @adhranPosted about 2 years ago

  The problem is your using "img" tag in html instead copy and paste the svg code directly into your html, To change the svg color you need to select the "path" tag of the svg and set the "fill" property to white. (eg:- 👉 svg> g > path { fill: white; })

  Here I selected the "g" tag first and then the "path" because in the logo svg code the "path" tag is located inside the "g" tag but in other svgs(facebook logo, insta logo) the "path" is the directly located inside the "svg" tag

  Marked as helpful

  1

  ---

  

  David Pokrajac•370

  @DavidPokrajacPosted about 2 years ago

  @adhran Hey, thank you very much for your help!!!

  0
• 

  David Pokrajac•370

  @DavidPokrajacPosted about 2 years ago

  Also, to get mobile version of this, you can in console.log set window width to be at 360 pixels and than refresh the page.

  0