Skip to content

Commit cc2ea04

Browse files
author
Hannah
authored
Merge branch 'master' into stats-bug
2 parents 8b9e0ec + c76001e commit cc2ea04

7 files changed

+16
-1
lines changed

CHANGES.md

+2
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@ Change Log
2121
##### Fixes :wrench:
2222
* Fixed a bug causing custom TilingScheme classes to not be able to use a GeographicProjection. [#6524](https://github.com/AnalyticalGraphicsInc/cesium/pull/6524)
2323
* Fixed incorrect 3D Tiles statistics when a tile fails during processing. [#6558](https://github.com/AnalyticalGraphicsInc/cesium/pull/6558)
24+
* Fixed race condition causing intermittent crash when changing geometry show value [#3061](https://github.com/AnalyticalGraphicsInc/cesium/issues/3061)
25+
* `ProviderViewModel`s with no category are displayed in an untitled group in `BaseLayerPicker` instead of being labeled as `'Other'` [#6574](https://github.com/AnalyticalGraphicsInc/cesium/pull/6574)
2426

2527
### 1.45 - 2018-05-01
2628

Source/DataSources/StaticGeometryColorBatch.js

+1
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,7 @@ define([
101101
if (defined(unsubscribe)) {
102102
unsubscribe();
103103
this.subscriptions.remove(id);
104+
this.showsUpdated.remove(id);
104105
}
105106
}
106107
};

Source/DataSources/StaticGeometryPerMaterialBatch.js

+1
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,7 @@ define([
9696
if (defined(unsubscribe)) {
9797
unsubscribe();
9898
this.subscriptions.remove(id);
99+
this.showsUpdated.remove(id);
99100
}
100101
}
101102
return this.createPrimitive;

Source/DataSources/StaticGroundGeometryColorBatch.js

+1
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@ define([
6969
if (defined(unsubscribe)) {
7070
unsubscribe();
7171
this.subscriptions.remove(id);
72+
this.showsUpdated.remove(id);
7273
}
7374
}
7475
};

Source/DataSources/StaticOutlineGeometryBatch.js

+1
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,7 @@ define([
7171
if (defined(unsubscribe)) {
7272
unsubscribe();
7373
this.subscriptions.remove(id);
74+
this.showsUpdated.remove(id);
7475
}
7576
}
7677
};

Source/Widgets/BaseLayerPicker/ProviderViewModel.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ define([
7373
*/
7474
this.iconUrl = options.iconUrl;
7575

76-
this._category = defaultValue(options.category, 'Other');
76+
this._category = defaultValue(options.category, '');
7777

7878
knockout.track(this, ['name', 'tooltip', 'iconUrl']);
7979
}

Source/Widgets/BaseLayerPicker/createDefaultImageryProviderViewModels.js

+9
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ define([
6565
name: 'Mapbox Satellite',
6666
tooltip: 'Mapbox satellite imagery https://www.mapbox.com/maps/',
6767
iconUrl: buildModuleUrl('Widgets/Images/ImageryProviders/mapboxSatellite.png'),
68+
category : 'Other',
6869
creationFunction: function() {
6970
return new MapboxImageryProvider({
7071
mapId: 'mapbox.satellite'
@@ -76,6 +77,7 @@ define([
7677
name: 'Mapbox Streets',
7778
tooltip: 'Mapbox streets imagery https://www.mapbox.com/maps/',
7879
iconUrl: buildModuleUrl('Widgets/Images/ImageryProviders/mapboxTerrain.png'),
80+
category : 'Other',
7981
creationFunction: function() {
8082
return new MapboxImageryProvider({
8183
mapId: 'mapbox.streets'
@@ -87,6 +89,7 @@ define([
8789
name: 'Mapbox Streets Classic',
8890
tooltip: 'Mapbox streets basic imagery https://www.mapbox.com/maps/',
8991
iconUrl: buildModuleUrl('Widgets/Images/ImageryProviders/mapboxStreets.png'),
92+
category : 'Other',
9093
creationFunction: function() {
9194
return new MapboxImageryProvider({
9295
mapId: 'mapbox.streets-basic'
@@ -105,6 +108,7 @@ imagery for Antarctica. The map features 0.3m resolution imagery in the continen
105108
parts of Western Europe from DigitalGlobe. In other parts of the world, 1 meter resolution imagery is available from GeoEye IKONOS, \
106109
i-cubed Nationwide Prime, Getmapping, AeroGRID, IGN Spain, and IGP Portugal. Additionally, imagery at different resolutions has been \
107110
contributed by the GIS User Community.\nhttp://www.esri.com',
111+
category : 'Other',
108112
creationFunction : function() {
109113
return new ArcGisMapServerImageryProvider({
110114
url : 'https://services.arcgisonline.com/ArcGIS/rest/services/World_Imagery/MapServer',
@@ -121,6 +125,7 @@ This worldwide street map presents highway-level data for the world. Street-leve
121125
Canada; Japan; most countries in Europe; Australia and New Zealand; India; parts of South America including Argentina, Brazil, \
122126
Chile, Colombia, and Venezuela; Ghana; and parts of southern Africa including Botswana, Lesotho, Namibia, South Africa, and Swaziland.\n\
123127
http://www.esri.com',
128+
category : 'Other',
124129
creationFunction : function() {
125130
return new ArcGisMapServerImageryProvider({
126131
url : 'https://services.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer',
@@ -136,6 +141,7 @@ http://www.esri.com',
136141
This web map contains the National Geographic World Map service. This map service is designed to be used as a general reference map \
137142
for informational and educational purposes as well as a basemap by GIS professionals and other users for creating web maps and web \
138143
mapping applications.\nhttp://www.esri.com',
144+
category : 'Other',
139145
creationFunction : function() {
140146
return new ArcGisMapServerImageryProvider({
141147
url : 'https://services.arcgisonline.com/ArcGIS/rest/services/NatGeo_World_Map/MapServer/',
@@ -149,6 +155,7 @@ mapping applications.\nhttp://www.esri.com',
149155
iconUrl : buildModuleUrl('Widgets/Images/ImageryProviders/openStreetMap.png'),
150156
tooltip : 'OpenStreetMap (OSM) is a collaborative project to create a free editable map \
151157
of the world.\nhttp://www.openstreetmap.org',
158+
category : 'Other',
152159
creationFunction : function() {
153160
return createOpenStreetMapImageryProvider({
154161
url : 'https://a.tile.openstreetmap.org/'
@@ -161,6 +168,7 @@ of the world.\nhttp://www.openstreetmap.org',
161168
iconUrl : buildModuleUrl('Widgets/Images/ImageryProviders/stamenWatercolor.png'),
162169
tooltip : 'Reminiscent of hand drawn maps, Stamen watercolor maps apply raster effect \
163170
area washes and organic edges over a paper texture to add warm pop to any map.\nhttp://maps.stamen.com',
171+
category : 'Other',
164172
creationFunction : function() {
165173
return createOpenStreetMapImageryProvider({
166174
url : 'https://stamen-tiles.a.ssl.fastly.net/watercolor/',
@@ -173,6 +181,7 @@ area washes and organic edges over a paper texture to add warm pop to any map.\n
173181
name : 'Stamen Toner',
174182
iconUrl : buildModuleUrl('Widgets/Images/ImageryProviders/stamenToner.png'),
175183
tooltip : 'A high contrast black and white map.\nhttp://maps.stamen.com',
184+
category : 'Other',
176185
creationFunction : function() {
177186
return createOpenStreetMapImageryProvider({
178187
url : 'https://stamen-tiles.a.ssl.fastly.net/toner/',

0 commit comments

Comments
 (0)