Release v1.3.0

[orangemug]

New release ready for testing at https://maputnik.github.io/releases/v1.3.0-beta

If anyone has time to give it a try, please report anything broken that wasn't broken in the previous version, or respond 'It looks good to me!'. CC @gregorywolanski @pathmapper @keum

I'll release the non-beta once people have given it a test.

Changes

• Added Survey (Survey #328)
• Updated osm-liberty repository for change of ownership (Update repository for OSM Liberty #333)
• Add loading modal for styles (Loading dialog #330)
• Added url option to display tile boundaries (Added option to display tile boundaries #315)
• Added keyboard shortcuts + modal (Keyboard shortcuts #313)
• Color blindness emulation via url option (Color blindness emulation #312)
• Added skip-menu link for keyboard users (Added skip-menu link for keyboard users #309)
• Added keyboard accessible layer options (Keyboard accessible layer options #306)
• Disable spellcheck on <input>'s (Disable spellcheck on <input/>'s #304)
• Added support for encoding to raster-dem source, enabling terrarium tiles (Added support for encoding to raster-dem source #302)
• Added prefers-reduced-motion support (Added prefers-reduced-motion support #296)
• Made color swatch larger to make it easier for visually imparied users (Make color swatch larger so its easier to see #297)
• Added accessible modal via react-aria-modal (Added accessible modal via react-aria-modal #300)
• Made buttons actual <button>s so they are keyboard accessible (Made buttons keyboard accessible #298)

Contributions from @gregorywolanski @pathmapper @orangemug

[gregwolanski]

Awesome work on the accessibility, @orangemug!

First thing from the browser console:

app.b31056cb1cf2efb150c4.js:19 Fetch API cannot load mapbox://mapbox.mapbox-streets-v7. URL scheme must be "http" or "https" for CORS request.

[orangemug]

@gregorywolanski I think that's an existing bug in https://github.com/maputnik/editor/blob/4b5536b282cb97059fcbb181e2af01af37a2bc2f/src/components/App.jsx#L352 only for styles with mapbox data sources.

Can you confirm that this was also an issue with the previous release? If so raise a new ticket and we'll fix for the next release

[orangemug]

@gregorywolanski I noticed one bug with the survey

Comments (at the end of the survey) are a single line input should probably be multiline

Also is it possible (and would you consider) putting a message on survey completion saying

If you appreciate Maputnik please consider donating

[gregwolanski]

Can you confirm that this was also an issue with the previous release?

I will do my best to take a look at this before the end of the week.

At glance, I don’t see this same message in the console in the previous release.

Comments (at the end of the survey) are a single line input should probably be multiline

Good point! Fixed.

Also is it possible (and would you consider) putting a message on survey completion […]

I can’t change this part of the survey in the free version of Typeform.

[orangemug]

@gregorywolanski you're correct, the actual error is (printed as a warning a few lines above)

Failed to normalizeSourceURL: TypeError: Cannot read property 'normalizeSourceURL' of undefined

It looks like mapboxUtil.normalizeSourceURL no longer exists, see

https://github.com/maputnik/editor/blob/8b0aa194cfde3cb9d36be78a9013c8512ee9950c/src/components/App.jsx#L368

[orangemug]

Fixed mapboxUtil.normalizeSourceURL issue in #341. Nice spot @gregorywolanski

[gregwolanski]

@pathmapper @keum Any feedback? :)

[pathmapper]

Looks good to me, thanks!

[orangemug]

Released v1.3.0 all the places

• Editor - https://maputnik.github.io/editor
• Blog post - https://maputnik.github.io/blog/2018/07/12/release-v1.3.0

[keum]

@orangemug sorry...didn't get a chance to test it out yet...will take a look tomorrow and will reply. thanks for everyone for the work you guys have put together. Also quick remark on documentation relating to issue #337 I could stab at contributing few things for sure. There staff here who also loves Maputnik, so we'll collaborate to the effort. thanks!

[orangemug]

sorry...didn't get a chance to test it out yet...will take a look tomorrow and will reply.

No worries @keum, any feedback just create a new ticket now. I prioritised pushing the release since a few people had already given it a check over.

Also quick remark on documentation relating to issue #337 I could stab at contributing few things for sure

Awesome! Lets talk documentation over in #337 I've got a few ideas as well