trailheadlabs / outerspatial-map-library Goto Github PK

• Leaflet.GeometryUtil
• Leaflet.Snogylop

the link to OuterSpatial Map Builder in the README 404s. i thought maybe you might still be in the process of re-open sourcing it, but it looks like it was already up in March...

https://github.com/trailheadlabs/outerspatial.js#outerspatial-map-builder

ps. i'm here because @jmoe showed me your current ArcGIS Service integration while we were up in Tacoma. very cool!

Seems you can clear the measure tools with hitting escape. Same with the line tool in draw. But with polygons, can't seem to clear them.

The layer objects in this example contains a maki element. Did this once give the clustered icons a maki icon perhaps?

maki: {
  color: '#609321',
  name: 'park'
}

Was wondering if this functionality was broken at some point and we didn't realize it or if it's supposed to be:

 styles: {
      point: {
        'marker-color': '#609321',
        'marker-symbol': 'park'
      }
    }

e.g.
https://github.com/trailheadlabs/outerspatial.js/blob/master/examples/geojson-layer-clustered.js#L11-L13
https://github.com/trailheadlabs/outerspatial.js/blob/master/examples/geojson-layer-clustered.js#L25-L28

1. activate measure
2. choose area
3. click around to create polygon
4. try to complete polygon, it won't let you.

When the print template is deployed on OuterSpatial:

• https://github.com/trailheadlabs/outerspatial.js/blob/master/src/control/print.js#L9
• References in https://github.com/trailheadlabs/outerspatial.js/blob/master/api/index.md

This will improve accessibility and eliminate a bunch of overly-complicated code that's really unnecessary at this point. We'll need to work on the styling some though.

https://staging.outerspatial.com/builder_maps/23/embed#11/37.3027/-122.0918

https://outerspatial-utilities.herokuapp.com

1. activate draw tool
2. choose area
3. click around to create polygon
4. try to complete polygon, it won't let you.

Now that we've "forked" Mapbox Outdoors, we should change name to "OuterSpatial Outdoors" (or something like that). We should do the same with a Satellite basemap, and potentially others as well.

This has long been a usability issue, and I really like how the Google Maps API is handling it now:

How do you feel about defaulting to using fractional zoom levels?

Here's a jsfiddle to get a sense of the feel of it:
https://jsfiddle.net/ryanjhodge/ay01dnxp/

e.g.

wheelPxPerZoomLevel: 120
zoomDelta: 0.5
zoomSnap: 0.5

https://gist.github.com/danswick/c280d49d7dae91dc351a8f0d4da99af0

We're going to license OuterSpatial.js under MIT for now, but should investigate other licenses that might make it easier for government usage.

Should have options to link out to current providers of directions like Google, Bing, etc.

https://developers.google.com/maps/documentation/urls/guide#directions-action

https://staging.outerspatial.com/builder_maps/23/embed#11/37.3199/-122.1165

Now that we're on Leaflet > 1.0, we should implement better support for z-index between all of a map's overlays.

This issue stems from #2

https://staging-manager.outerspatial.com/5185/maps/builder/23

Reproduce on the "examples/styling-vectors.html" page.

It looks like we're missing MultiPolygon and MultiPoint.

Contribute any appropriate fixes back into Leaflet so others can use the work.

Docs

• Create detailed documentation outlining the findings of the accessibility audit and publish via a blog post
• Write a VPAT (example: https://government.github.com/accessibility/)

Fixes

• Change markup of popup title to a heading element (h1, h2, or h3?)
• Wrap the popup close button (“x”) in a span with aria-hidden=“true” attribute and add a sibling span with descriptive text (“close popup”) that is visually hidden^
• Add pan buttons to navigation controls (http://kartena.github.io/Leaflet.Pancontrol/)
• Switch button images over to foreground images (from background images)
• Focus this title header element when popup is shown, setting the tabindex to -1 before the element is focused
• Remove title attributes from control buttons - adding the title text to alt text
• Set tabIndex of div with npmap-map class to 0 rather than 1
• Disable scrollWheelZoom by default and document the reason for this decision via contextual help in OuterSpatial Map Builder
• Add more contrast to :hover and :focus states for control buttons (background-color:#333;) and also override default outlines/glows for browsers to improve the contrast
• Use anchor elements, rather than divs, for markers and point the href attribute to the id of the popup’s div (this will require a persistent popup rather than the current approach which is destroying old and creating new popups)
• Also place the id of the marker (or other “shape”) that opened the popup into the href attribute of the close button and give focus back to this “shape” when the popup is closed
• Implement color palette contrast recommendations

^ Implement upstream in Leaflet

Testing on paper maps as requested.

https://github.com/trailheadlabs/outerspatial.js/blob/master/src/map.js#L22

This will allow organizations to pull in data from OuterSpatial without having to manually format popups, specify symbology, etc.

Going into fullscreen mode adds inline styles for fixed positioning. That's fine, but it will be nice to also add a class to the map container element so we can style externally based on that state.

Example of fullscreen mode right now:
<div class="outerspatial" style="left: 0px; position: fixed; top: 0px;"><otherMapHTML /></div>

Something like this would work:
<div class="outerspatial fullscreen" style="left: 0px; position: fixed; top: 0px;"><otherMapHTML /></div>

Deploy is only needed by Trailhead Labs, so we should separate it out into its own Gruntfile that isn't tracked in the repo.

https://staging-manager.outerspatial.com/5185/maps/paper/100

Looks like the Leaflet snap plugin may be broken. Potentially because of the migration to 1.0.

Controls: