While working on the profile popup to match the designs in #13, I noticed navbar.js became super messy. For the sake of future coding in this area, I have generalized many aspects of the navbar code to make it more scalable, improve readability and reduce redundancy.
navbar.js:
Generalized 'Opening' and 'Closing' popups to toggle and depend on the pop-up ID.
Extracted 'Setting Tabs' functionality into a separate module to outline distinct features.
Reworked 'close pop-up' cases to fit with new code.
notes.css:
Replaced '.about-pop-up button' styling to '.close-button' for consistency in formatting across all pop-ups.
Moved code around in notes.css for clear distinction of objects.
notes.html:
Utilises existing code for 'about pop-up' to create another pop-up for 'Profile Settings'.
This is quite a large overhaul to the entire navbar code, and wanted to confirm if this is okay before properly implementing the profile formatting as stated in #18. Unsure if this would affect any aspect of anyone elses branches, but if so I'm happy to work on rewiring any conflicts this may cause.
While working on the profile popup to match the designs in #13, I noticed navbar.js became super messy. For the sake of future coding in this area, I have generalized many aspects of the navbar code to make it more scalable, improve readability and reduce redundancy.
navbar.js:
notes.css:
notes.html:
This is quite a large overhaul to the entire navbar code, and wanted to confirm if this is okay before properly implementing the profile formatting as stated in #18. Unsure if this would affect any aspect of anyone elses branches, but if so I'm happy to work on rewiring any conflicts this may cause.