Skip to content

Add feature: Local map overlays for mission planner#2164

Closed
paul-fornage wants to merge 37 commits intoiNavFlight:masterfrom
paul-fornage:master
Closed

Add feature: Local map overlays for mission planner#2164
paul-fornage wants to merge 37 commits intoiNavFlight:masterfrom
paul-fornage:master

Conversation

@paul-fornage
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.

paul-fornage 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
}
}
}
} No newline at end of file
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

@paul-fornage paul-fornage 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.

@MrD-RC
Copy link
Member

MrD-RC commented Nov 7, 2024

Please resolved the conflicts. Also, it's Mission Control, not Mission Planner ;)

@sensei-hacker sensei-hacker added this to the 9.1 milestone Dec 23, 2025
@sensei-hacker
Copy link
Member

Thanks for this.
Are you familiar with "squash" in git?

@paul-fornage
Copy link
Author

No tbh :) I can figure it out and make it not 37 commits lol but I have long since forgotten about this and would have to relearn the project context for the merge conflicts. I'm not opposed but I'm busy for the holidays and the change wouldn't be able to come now. I think I've also improved since then too so I would probably end up cleaning it up in other ways. And also I changed package lock? silly behavior.

@erikbohnsack
Copy link

erikbohnsack commented Jan 14, 2026

Nice work @paul-fornage! I suggest closing this and going with #2519, which I created based on this, but based on a fresh master and a pair of extra eyes. Basically same functionality straight off, with some usability improvements IMO.

@paul-fornage
Copy link
Author

Nice work @paul-fornage! I suggest closing this and going with #2519, which I created based on this, but based on a fresh master and a pair of extra eyes. Basically same functionality straight off, with some usability improvements IMO.

Sounds good. Thank you for this, I'm glad it got cleaned up

@sensei-hacker
Copy link
Member

Superseded by #2519 which reimplements this feature with usability improvements.

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.

6 participants