search

fac25 / week2-db-Sumithra-Joe-Paz-Abdullah

Data base project
https://london-attractions.herokuapp.com/
0 stars 0 forks source link

Unnecessary p tags #40

Open mariaalouisaa opened 2 years ago

mariaalouisaa commented 2 years ago

It isn't necessary to wrap the the form inputs and labels in p tags. As you begin to work on bigger projects, it's nice to keep clutter out of the code.

Amazing work on this project! Love the design and your code is nice to follow. Great work for two days! ✨ 🤖

https://github.com/fac25/week2-db-Sumithra-Joe-Paz-Abdullah/blob/5c90628a0e65ca599c277ea11df621d59cd97c25/src/templates.js#L109-L121

mariaalouisaa commented 2 years ago

Another note on this bit of code - it's interesting UX to have the the placeholder be 3.5 but then only allow users to input whole integers... 🤔