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

Total frontend revamp #59

Merged
merged 53 commits into from
Sep 15, 2023
Merged

Total frontend revamp #59

merged 53 commits into from
Sep 15, 2023

Conversation

Nate-Wessel
Copy link
Contributor

@Nate-Wessel Nate-Wessel commented Jun 23, 2023

There area few parallel things going on here that I'll try to document because I may not have a chance to come back to this for a week or two. This is a very big, but necessary overhaul and I'm finding it hard to get it all done in one go.

  • The CSS layout + div containers are dramatically simplified.
  • The map component is totally gone, to be replaced by a react-leaflet implementation that should be much more declarative
  • There is now a class-based data store for spatial data. It should be the ultimate source of all map state.
  • All components that were tightly coupled with the previous map component need to be totally replaced. Unfortunately, that means the whole frontend...

@Nate-Wessel Nate-Wessel marked this pull request as ready for review September 11, 2023 20:51
@Nate-Wessel Nate-Wessel changed the title Simplify the overall layout Total frontend revamp Sep 11, 2023
@Nate-Wessel
Copy link
Contributor Author

This is now about up to the level of functionality of the current deployment... which is to say there's a way to go still. But if this got merged now, no injury would be done.

@Nate-Wessel
Copy link
Contributor Author

It has been deemed fit.

@Nate-Wessel Nate-Wessel merged commit db315b5 into deploy Sep 15, 2023
1 check failed
@Nate-Wessel Nate-Wessel deleted the redo-layout branch September 15, 2023 20:00
@Nate-Wessel Nate-Wessel restored the redo-layout branch September 15, 2023 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant