search

IITC-CE / ingress-intel-total-conversion

intel.ingress.com total conversion user script with some new features. Should allow easier extension of the intel map.
https://iitc.app
ISC License
285 stars 110 forks source link

Fly links not working #450

Closed Ren-Noir closed 2 years ago

Ren-Noir commented 3 years ago

I have never had an issue like this. For the life of me, I cannot get fly links to work for. Ivload and reload plugins, force update, but nothing seems to work. Please help

MysticJay commented 3 years ago

@Ren-Noir do you have an active draw?

le-jeu commented 3 years ago

since last release, you need to draw at least one polygon to include all the portals you want to field with. https://github.com/IITC-CE/ingress-intel-total-conversion/pull/310#issue-362714616

  • region selection with the polygon draw tool
  • exclusion of portals with the marker draw tool
  • steady drawing (portals is not excluded from the drawing if they are moved out of boundaries)

image

johnd0e commented 3 years ago

https://github.com/IITC-CE/ingress-intel-total-conversion/blob/2daad393c75f45e9f121136c4717d4dec51fca4f/plugins/fly-links.js#L344

I see that plugin now silently breaks if draw-tools is not available. That is not good at all. Perhaps better solution would be fallback to map bounds.

@Fly33

johnd0e commented 3 years ago

Another bug: fly-links does not react on drawing. So we have to pan slightly to initiate fly-links update. Annoying.

johnd0e commented 3 years ago

Also: I'd like to use Circle too, not only Polyline.

crystalwizard commented 3 years ago

Might consider a popup if there's no polygon to inform the user that a polygon must be drawn before flylinks can activate