Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updated basemaps widget combining agol and custom basemaps #522

Merged
merged 5 commits into from
Apr 2, 2016

Conversation

tmcgee
Copy link
Member

@tmcgee tmcgee commented Mar 20, 2016

No description provided.

maintained support for existing agol and custom configurations
removed BasemapGallery widget custom basemaps allowing for more custom map types
Basemap and BasemapLayer dijits no longer needed for custom basemaps
some of the examples required v 3.16 or higher of the ESRI JavaScript API
@tmcgee tmcgee added this to the v1.4.0 milestone Mar 20, 2016
@tmcgee
Copy link
Member Author

tmcgee commented Mar 22, 2016

As I experiment with this further, it appears that ESRI has limited the esri/basemaps to their Tiled Map Service, OpenStreetMap and Vector Tile layers. Others like WebTiled, WMS, Dynamic Map Service and Image Map Service layers are not supported.

I will keep this PR open for any additional discussion and potential revision but it should not be committed as is.

Custom basemaps are also supported in agol mode for those types
    of basemaps which are supported by `esri/basemaps`
Added additional examples of custom basemaps using WebMapTiled layers
@tmcgee
Copy link
Member Author

tmcgee commented Mar 26, 2016

I have restored the custom mode so we have the original functionality PLUS the ability to include custom basemaps of the supported types to esri/basemaps.

@DavidSpriggs Ready for review and merge.

@tmcgee tmcgee removed the proposal label Mar 26, 2016
@DavidSpriggs
Copy link
Member

@tmcgee Merge prior into this PR.

# Conflicts:
#	viewer/js/config/basemaps.js
@tmcgee
Copy link
Member Author

tmcgee commented Apr 1, 2016

@DavidSpriggs done.

@DavidSpriggs DavidSpriggs merged commit 679dec0 into develop Apr 2, 2016
@DavidSpriggs DavidSpriggs deleted the feature/updated-basemaps-widget branch April 2, 2016 03:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants