diff --git a/src/core/renderer/tiles/TilesRendererBase.d.ts b/src/core/renderer/tiles/TilesRendererBase.d.ts index c64c47789..d709ef66f 100644 --- a/src/core/renderer/tiles/TilesRendererBase.d.ts +++ b/src/core/renderer/tiles/TilesRendererBase.d.ts @@ -6,6 +6,7 @@ import { Tileset } from './Tileset.js'; // Events dispatched by TilesRendererBase, available across all renderer implementations. export interface TilesRendererBaseEventMap { 'needs-update': {}; + /** @deprecated */ 'load-content': {}; 'load-tileset': { tileset : Tileset, /* @deprecated Use tileset instead */ tileSet? : Tileset, url : string }; /* @deprecated Use 'load-tileset' instead */ diff --git a/src/core/renderer/tiles/TilesRendererBase.js b/src/core/renderer/tiles/TilesRendererBase.js index 9adf05aab..d76a547b1 100644 --- a/src/core/renderer/tiles/TilesRendererBase.js +++ b/src/core/renderer/tiles/TilesRendererBase.js @@ -253,6 +253,7 @@ const unifiedPriorityCallback = ( a, b ) => { /** * Fired when any tile content (model or external tileset) finishes loading. + * @deprecated * @event TilesRendererBase#load-content */ @@ -835,7 +836,7 @@ export class TilesRendererBase { this.rootLoadingState = LOADED; this.rootTileset = root; this.dispatchEvent( { type: 'needs-update' } ); - this.dispatchEvent( { type: 'load-content' } ); + this.dispatchEvent( { type: 'load-content' } ); // @deprecated this.dispatchEvent( { type: 'load-tileset', tileset: root, @@ -1800,7 +1801,7 @@ export class TilesRendererBase { // dispatch an event indicating that this model has completed and that a new // call to "update" is needed. this.dispatchEvent( { type: 'needs-update' } ); - this.dispatchEvent( { type: 'load-content' } ); + this.dispatchEvent( { type: 'load-content' } ); // @deprecated if ( isExternalTileset ) { this.dispatchEvent( { diff --git a/src/three/plugins/images/ImageOverlayPlugin.js b/src/three/plugins/images/ImageOverlayPlugin.js index 0b82eea47..2eed3d7f2 100644 --- a/src/three/plugins/images/ImageOverlayPlugin.js +++ b/src/three/plugins/images/ImageOverlayPlugin.js @@ -204,7 +204,7 @@ export class ImageOverlayPlugin { this.resetVirtualChildren( ! this.enableTileSplitting ); tiles.recalculateBytesUsed(); - tiles.dispatchEvent( { type: 'needs-rerender' } ); + tiles.dispatchEvent( { type: 'needs-render' } ); } @@ -1191,7 +1191,7 @@ export class ImageOverlayPlugin { } info.failed = true; - tiles.dispatchEvent( { type: 'load-error', tile, overlay, error: err } ); + tiles.dispatchEvent( { type: 'load-error', tile, overlay, error: err, url: null } ); return null; } ); diff --git a/src/three/renderer/tiles/TilesRenderer.js b/src/three/renderer/tiles/TilesRenderer.js index af8e722d9..0ad61caa5 100644 --- a/src/three/renderer/tiles/TilesRenderer.js +++ b/src/three/renderer/tiles/TilesRenderer.js @@ -146,6 +146,10 @@ export class TilesRenderer extends TilesRendererBase { console.warn( 'TilesRenderer: "load-tile-set" event has been deprecated. Use "load-tileset" instead.' ); type = 'load-tileset'; + } else if ( type === 'load-content' ) { + + console.warn( 'TilesRenderer: "load-content" event has been deprecated. Use "load-model" or "load-tileset" instead.' ); + } EventDispatcher.prototype.addEventListener.call( this, type, listener ); @@ -159,6 +163,10 @@ export class TilesRenderer extends TilesRendererBase { console.warn( 'TilesRenderer: "load-tile-set" event has been deprecated. Use "load-tileset" instead.' ); type = 'load-tileset'; + } else if ( type === 'load-content' ) { + + console.warn( 'TilesRenderer: "load-content" event has been deprecated. Use "load-model" or "load-tileset" instead.' ); + } return EventDispatcher.prototype.hasEventListener.call( this, type, listener ); @@ -172,6 +180,10 @@ export class TilesRenderer extends TilesRendererBase { console.warn( 'TilesRenderer: "load-tile-set" event has been deprecated. Use "load-tileset" instead.' ); type = 'load-tileset'; + } else if ( type === 'load-content' ) { + + console.warn( 'TilesRenderer: "load-content" event has been deprecated. Use "load-model" or "load-tileset" instead.' ); + } EventDispatcher.prototype.removeEventListener.call( this, type, listener ); diff --git a/test/three/TilesRenderer.test.ts b/test/three/TilesRenderer.test.ts index b6300067e..b16c84088 100644 --- a/test/three/TilesRenderer.test.ts +++ b/test/three/TilesRenderer.test.ts @@ -33,7 +33,6 @@ function typecheck( renderer: TilesRenderer ) { renderer.addEventListener( 'load-tileset', loadTileset ); renderer.addEventListener( 'tiles-load-start', emptyEvent ); renderer.addEventListener( 'tiles-load-end', emptyEvent ); - renderer.addEventListener( 'load-content', emptyEvent ); renderer.addEventListener( 'load-model', loadModel ); renderer.addEventListener( 'dispose-model', disposeModel ); renderer.addEventListener( 'tile-visibility-change', tileVisibilityChange );