Manage upstream externals about ingress-intel-total-conversion HOT 2 CLOSED

Here will be the list of all externals, where we should specify:

• current version (clear checkbox if it's below upstream)
• bold if the source was modified from upstream (specify exact mods)

(WIP)

jQuery-related components:

• jQuery (jquery-3.3.1.min.js) © JS Foundation: [MIT]
• jQuery UI (jquery-ui-1.12.1.min.js) © JS Foundation:[MIT]
• [2b253c5] jquery-qrcode (jquery.qrcode.min.js) © Jerome Etienne: [MIT]
• [74613a8 mod] jquery-taphold (taphold.js) © Rich Adams: [CC BY 3.0]
  Mod: touchcancel support (added later to upstream too)
  jQuery: fixed deprecated functions (IITC-CE/jquery-taphold@a85050a)
• [~1.2.0 Mod] Spectrum colorpicker (.js/.css) © Brian Grinstead: [MIT]
  jQuery: fixed deprecated functions (McBen a43f785504a73794f3deb85d9e1028a25d9132c4) (fixed in recent upstream too)

Leaflet-related components:

• [1.4.0] Leaflet © Vladimir Agafonkin, CloudMade: [LICENCE]
• [33ea262 + mod] Leaflet.draw © Jon West, Jacob Toye, and Leaflet: [MIT]
• Shramov leaflet-plugins © Pavel Shramov, Bruno Bergot: [MIT]
  • [44935a9] Bing.js
  • [c91e49c] Yandex.js
  • [3d2d4b0] KML.js
• [b68a264 + 1641fd5] toGeoJSON (togeojson.js) © Mapbox: [BSD-2-Clause]
• [04c6d46 + mod] Leaflet.FileLayer (leaflet.filelayer.js) © Makina Corpus: [MIT]
• [0.8.0] Leaflet.GridLayer.GoogleMutant (Leaflet.GoogleMutant.js) © Iván Sánchez Ortega: [LICENCE]
• [v3.4.0 + 78d77aa + 1973792] Leaflet.MiniMap (Control.MiniMap.js) © Norkart AS: [BSD-2-Clause]
• https://github.com/kartena © Kartena AB
  • [7b54411] Leaflet.Pancontrol (L.Control.Pan .js/.css/images): [BSD-2-Clause]
  • [0ca1aba] Leaflet.zoomslider (L.Control.Zoomslider .js/.css): [BSD-2-Clause]
• [mod] Leaflet.Geodesic (L.Geodesic.js) © Kevin Brasier: [BSD-2-Clause]
• [v0.2.6 + 1973792] Overlapping Marker Spiderfier (oms.min.js) © George MacKerron: [MIT]
• [7f807ae + 9077c24] LatLng_Bearings.js [www] © Greg Allensworth: [No licence]

Other:

• [6717ce2] autolink-js © Bryan Woods: [MIT]
• [f828670 + ea46405] delaunay.js © Jay LaPorte: [CC0 1.0]
  Modded (minor refactoring needed during update).
• [3acd55a] load.js © Chris O'Hara: [MIT]
• [5e20625 + 42ed656 + 6f8fbe6] s2-geometry-javascript (s2geometry.js) © Jon Atkins: [LICENSE: same as IITC]

from ingress-intel-total-conversion.

So I've figured out exact versions of externals (as long as required modification), and now we can begin to update most of them (excl. Leaflet.draw, leaflet.filelayer.js, togeojson.js).

Some things still need to be clarified:

• Do we really need 522ef34: #78 (comment)?
  Update: #122

• Leaflet.Geodesic has an alternative:
  (https://github.com/henrythasler/Leaflet.Geodesic) (Leaflet.Geodesic.js) © Henry Thasler: [GPL-3.0]

• s2geometry.js has more advanced fork: s2geometry.js © Jon Atkins, AJ ONeal: [ISC]
  Moved to https://git.daplie.com/Daplie/s2-geometry.js/ (failed to connect)

• Yandex.js is extremely laggy fixed.
  Consider trying https://github.com/kostjerry/leaflet-yandex instead

• Bing.js is extremely blurry enableRetina fixed.

• Shramov leaflet-plugins: files leaflet.filelayer.js, togeojson.js are not present in upstream, need to search out their sources.
  There is an error in ATTRIBUTION.md, that files are not by Shramov

• toGeoJSON (togeojson.js) © Mapbox: [BSD-2-Clause]
  This repo is now developed at https://github.com/tmcw/togeojson: there is no major features added yet. But it is ES2015+.

• delaunay.js upstream is updated to more fast version, but it's still not fastest available, see performance comparison here.
  We should consider switch to https://github.com/mapbox/delaunator, it has pretty the same api, with minor differences.

  // https://unpkg.com/[email protected]/delaunator.js
  var triangles = tidyLinks.Delaunator.from(locations.map(function (latlng) {
    var point = map.project(latlng, tidyLinks.PROJECT_ZOOM);
    return [point.x,point.y];
  })).triangles;
  

• alternative to oms.js: https://github.com/robertleeplummerjr/Leaflet.spiderfy/
  see jawj/OverlappingMarkerSpiderfier-Leaflet#30, jawj/OverlappingMarkerSpiderfier-Leaflet#31

• load.js is used only in basemap-yandex.user.js and sync.user.js, in form of load(yandexApiJs).thenRun(callback);, so I suppose it can be replaced with jQuery.getScript().
  BTW, there is also jQuery.loadScript

• autolink.js is used in single place: (and I don't feel it's necessary):
  

  ingress-intel-total-conversion/code/chat.js

  Line 334 in d77562b

  msg += $('<div/>').text(markup[1].plain).html().autoLink();

• jquery.qrcode.min.js is used in single place too:

  ingress-intel-total-conversion/code/utils_misc.js

  Line 207 in d77562b

  var script = '<script>$(\'#qrcode\').qrcode({text:\'GEO:'+lat+','+lng+'\'});</script>';

  
  Perhaps it should be better to have it as plugin.
  The codebase is rather old. Alternatives: https://www.nayuki.io/page/qr-code-generator-library#javascript, https://www.cssscript.com/tag/qr-code/

• LatLng_Bearings.js can be replaced by gmaps geometry library function spherical.computeHeading.

-.leaflet-control-zoom {
+.leaflet-bar {

+/*
+  override the scale bar and push it over
+*/
+.leaflet-control-scale {
+  left: 63px !important;
+}

 .leaflet-control-zoomslider-slider {
+       background-color: #fff;
+       border-bottom: 1px solid #ccc;

from ingress-intel-total-conversion.