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

Add feature: Local map overlays for mission planner #2164

Open
wants to merge 37 commits into
base: master
Choose a base branch
from

Conversation

paulwrath1223
Copy link

This PR makes it possible for the user to drag and drop a GEO file containing property lines, trails, waypoints, flight restricted areas, or anything else on to the map. The change does not affect anything about the actual mission and is purely for visual reference.

While the feature is similar to GEO-fencing, an upcoming rumored addition to iNav, this allows the user to easily import existing data in almost any applicable format (GEOJSON, KML, GPX, TOPOJSON, and IGC) to be able to more easily identify or plan around various landmarks in mission planner.

To put it bluntly, I am not a JS dev, and some conventions may be missing. Especially those specific to this project. But I did my best to emulate the style in the rest of the project, if not with more comments. If there are any issues I am in the iNav discord under the name BeanTime, and I'm always open to suggestions or changes that need to be made.

paulwrath1223 and others added 30 commits July 22, 2024 19:58
fixed issue where panning would not work when mouse-down was over a feature imported by the user in mission planner
Not working, but I want to keep working on a different machine
Not working, but I want to keep working on a different machine
@@ -9649,4 +9649,4 @@
}
}
}
}
}
Copy link
Author

Choose a reason for hiding this comment

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

I accidentally pushed some changes my IDE made to this file, and then undid them manually, but for some reason this is still in the diff. Feel free to not merge this file

Copy link
Collaborator

Choose a reason for hiding this comment

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

no big deal. probably extra empty line at the end of file

@paulwrath1223 paulwrath1223 marked this pull request as ready for review August 8, 2024 20:10
@b14ckyy
Copy link
Collaborator

b14ckyy commented Aug 8, 2024

@Scavanger maybe worth to keep in mind when porting GF to 8.0.

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.

3 participants