doc/plugins/osm/discussion.mdwn

   1 ### CGI requirement when using \[\[\!waypoint\]\] on pages?
   2
   3 Most of the osm plugin works well without cgi. The link from waypoints however use ikiwiki.cgi, at least in my configuration. Is this actually required or is it possible to use a pre rendered page to avoid running cgi on the server?
   4
   5 At the moment I'd prefer not running cgi and one of the advantages of a wiki compiler is that you could run without cgi on the server.
   6
   7 This is a minor issue but I'd be interested if you think it's possible to prerender waypoint maps. I.e maps that center on a specific waypoint whilst having all waypoints of that map visible.
   8
   9 > I think it would be possible, but it would require a patch. I do think however that this is the way it works: all waypoints are shown, and the only difference is that the map is centered. --[[anarcat]]
  10
  11 ### Configure all osm tags to use same icon?
  12
  13 Setting the default `osm_tag_default_icon` does not seem to work? All tagged waypoint pages now want their own unique icon and display broken image if not present. Populating the tag folder with identical icons gets a bit much when there are a lot of tags.
  14
  15 > That looks like a bug. Please file it as such in [[bugs]]. --[[anarcat]]
  16
  17 ### \[Wishlist\] Setting unigue icon for "active waypoint"
  18
  19 For usability it would be great if it was possible to display the active waypoint with a different icon. So that clicking a waypoint map symbol takes you to a map with lots of waypoints but the waypoint from the sources page is centered (as per current behaviour) **and** has a different icon.
  20
  21 > This *would* indeed be awesome. --[[anarcat]]
  22
  23 *PS. The osm plugin is amazing!*
  24
  25 > Thanks! --[[anarcat]]
  26
  27 -----
  28
  29 ## Updated plugin needs review and merge
  30
  31 [[!template id=gitbranch branch=tincho-osm author="[[tincho]]"]]
  32
  33 [[schmonz]] here. I recently tried to use this plugin, had some trouble, and discovered on IRC that [[tincho]] has a largely [rewritten version](https://blog.tincho.org/posts/OSM_in_IkiWiki/) that looks good [on his site](https://blog.tincho.org/Mingle/), but hadn't gotten around to submitting for merge. So we remote-paired on it today, improved a few things, and wrote down what we noticed:
  34
  35 ### Features removed
  36
  37 - Google Maps API
  38 - Multiple layers
  39 - Customized waypoint icons
  40 - Full-screen map (via the CGI)
  41 - Whatever "editable" did (maybe something interactive?)
  42 - OpenLayers -> Leaflet (if anyone was somehow depending on OpenLayers)
  43
  44 ### Features added
  45
  46 - Maps actually work again
  47 - Maps work when embedded in HTTPS sites
  48 - Multiple maps and multiple waypoints in a page work properly
  49 - Maps _do_ appear in inlines
  50 - Pagestate hash gets cleaned up better after edit/preview/delete
  51 - Unigue icon for "active waypoint" works, also possible to select manually any waypoint to highlight
  52 - Good test coverage, including inlines and multiple maps/waypoints per page
  53
  54 ### Wishlist
  55
  56 - Optionally do something (render a static image?) for RSS
  57 - When drawing lines between waypoints that are linked, optionally draw an arrow
  58 - Performance/ikiwiki-idiomaticness: generate one GeoJSON file per waypoint, then rely on getting changed waypoints from the `changes()` hook
  59 - Treat optional waypoint description as Markdown (or whatever input format you're using)
  60 - Simplify scrubbing (maybe avoid needing HTML::Scrubber)
  61
  62 ### Questions for reviewers
  63
  64 - Given this is backward-incompatible, dhould we call it something other than "osm"?
  65 - What needs scrubbing? Have we covered all the bases? Too many bases?
  66 - Should we vendor Leaflet into an underlay, instead of needing a URL to load it from a CDN? [[schmonz]] somewhat prefers this, so we avoid needing external resources by default, avoid breaking when the Leaflet CDN is down, etc.
  67 - Should we write some tests before merging? `osm.pm` hadn't had any, FWIW -- [[Tincho]] Done
  68
  69 Bump! Tincho would like to see us merge his effort, and FWIW I'd also
  70 rather not have to carry around a local copy of his work to get a map
  71 with waypoints on my HTTPS site. [[smcv]], can you spare some round
  72 tuits to give us your thoughts? --[[schmonz]]
  73
  74 ----
  75
  76 Just stumbled onto this.
  77
  78 With regards to features lost my only concern would be customizable waypoint icons. It would be really great to be able to at least change colour from the config. The wishlist point of generating one file per waypoint might solve an issue I have with osm only showing the most recently added waypoints unless I rebuild. I have however a huge number of waypoints.
  79
  80 Looks like good changes to me!
  81
  82 > did a grep `Placemark pois.kml|wc -l` which returned 3468. Which perhaps isn't that much? I'm thinking about how individual poi files might affect performance. My performance troubles are more likely to be with my tweaked album and img plugins though.
  83
  84 --[[kjs]]
  85
  86 > The issue about not getting all the waipoints until you rebuild has been solved, the current plugin had issues with keeping track of updated and deleted waypoints which is now fixed in my branch. --[[Tincho]]