search

jslovers / jslovers-branding-page

MIT License
6 stars 22 forks source link

gallery section added #40

Open ksubramanyeshwara opened 1 year ago

Neha commented 11 months ago

Thank you for your PR. Could you please add description

Neha commented 11 months ago

Please take the pull before doing the changes. I have integrated prettier.

Below are the requested changes:

  1. Please rename the images name eg: jslovers-delhi-meetup, jslovers-speaker, etc. instead of 01, 02, names
  2. Please add relevant alt tags
  3. We don't need to add relative, and absolute on the gallery divs.
  4. We don't need to wrap the gallery heading with div. Please check how all the sections heading is done
  5. Please change h1 to h2 of gallery heading
  6. Please make the images centre aligned as the sponsors and rest of the sections are
  7. We can optimize the HTML of the gallery and get rid of extra divs

Thank you for your contribution @ksubramanyeshwara