Responsive multi-page using grid and flex

Community feedback

• 

  Manish Patel•740

  @savvystrider

  Posted over 2 years ago

  Looks amazing!

  Here are some suggestions to improve the JS:

  • Use const instead of let for variables that are not reassigned.

  • Use textContent instead of innerHTML since you're only setting text content.

  • Use input event instead of click event for input fields: Instead of attaching the click event to increase or decrease the number of visitors, you can attach the input event to the visitorNumber input field. This allows users to change the value by typing directly into the input field.

  • Instead of showDiv1(), showDiv2(), and showDiv3(), you could use more descriptive function names in camelCase, such as showFamilyBooking(), showSpecialBooking(), and showSocialBooking().

  • Also, try to consolidate repeated code: The code that checks if a value is valid and shows or hides the label could be consolidated into a separate function that can be reused for each input field.

  11