Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code Modularization and UI changes #4

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

saikiranAnnam
Copy link

This PR focuses on improving the SWE site's structure, maintainability, and user interface. Previously, all logic and functionality were handled in a single swe.js file, making the codebase difficult to maintain and extend. To address this, the following changes were made:

  1. Code Modularization:

    • Separated logic from swe.js into multiple modules based on functionality:
      • map.js for map initialization and layer handling.
      • datepicker.js for datepicker setup and logic.
      • geotiff.js for handling GeoTIFF layers.
      • legend.js for creating and adding legends to the map.
      • events.js for button and map interaction listeners.
      • utils.js for reusable utility functions like copyCoordinates.
    • Improved code readability and ensured easier debugging.
  2. UI Enhancements:

    • Redesigned the control panel and legend for a more polished and user-friendly interface.
    • Adjusted spacing, alignment, and colors to improve aesthetics and usability.
    • Fixed layout issues like overlapping zoom controls and misaligned elements.

Before :
Screenshot 2024-11-18 at 5 18 24 PM

After :
Screenshot 2024-11-18 at 5 18 42 PM

  1. Before and After Screenshots:
    • Included before and after images showcasing the updated UI changes for clarity.

Impact:

  • Improved maintainability of the codebase by modularizing swe.js.
  • Enhanced user experience with a more modern and visually appealing UI.
  • Simplified future updates by logically grouping functionality into separate files.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I moved to index.html into src file

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

moved to src folder

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant