added improved texture compression, refactored texturecoord flow (#688)

Author: jfboeve

src/core/CoreNode.ts

@@ -728,7 +728,6 @@ export class CoreNode extends EventEmitter {
     valid: false,
   };
   public textureCoords?: TextureCoords;
-  public updateTextureCoords: boolean = false;
   public updateShaderUniforms: boolean = false;
   public isRenderable = false;
   public renderState: CoreNodeRenderState = CoreNodeRenderState.Init;
@@ -901,6 +900,13 @@ export class CoreNode extends EventEmitter {
       } satisfies NodeTextureLoadedPayload);
     }
 
+    if (
+      this.stage.calculateTextureCoord === true &&
+      this.props.textureOptions !== null
+    ) {
+      this.textureCoords = this.stage.renderer.getTextureCoords!(this);
+    }
+
     // Trigger a local update if the texture is loaded and the resizeMode is 'contain'
     if (this.props.textureOptions?.resizeMode?.type === 'contain') {
       this.setUpdateType(UpdateType.Local);
@@ -1261,11 +1267,6 @@ export class CoreNode extends EventEmitter {
       this.sortChildren();
     }
 
-    if (this.updateTextureCoords === true) {
-      this.updateTextureCoords = false;
-      this.textureCoords = this.stage.renderer.getTextureCoords!(this);
-    }
-
     // If we're out of bounds, apply the render state now
     // this is done so nodes can finish their entire update loop before
     // being marked as out of bounds
@@ -1497,14 +1498,6 @@ export class CoreNode extends EventEmitter {
         isRenderable,
       } satisfies NodeRenderablePayload);
     }
-
-    if (
-      isRenderable === true &&
-      this.stage.calculateTextureCoord === true &&
-      this.textureCoords === undefined
-    ) {
-      this.updateTextureCoords = true;
-    }
   }
 
   /**
@@ -1704,6 +1697,8 @@ export class CoreNode extends EventEmitter {
     const t = this.globalTransform!;
     const coords = this.renderCoords;
     const texture = p.texture || this.stage.defaultTexture;
+    const textureCoords =
+      this.textureCoords || this.stage.renderer.defaultTextureCoords;
 
     // There is a race condition where the texture can be null
     // with RTT nodes. Adding this defensively to avoid errors.
@@ -1720,7 +1715,7 @@ export class CoreNode extends EventEmitter {
       colorBr: this.premultipliedColorBr,
       texture,
       textureOptions: p.textureOptions,
-      textureCoords: this.textureCoords,
+      textureCoords: textureCoords,
       shader: p.shader as CoreShaderNode<any>,
       alpha: this.worldAlpha,
       clippingRect: this.clippingRect,
@@ -1797,7 +1792,6 @@ export class CoreNode extends EventEmitter {
 
   set w(value: number) {
     if (this.props.w !== value) {
-      this.updateTextureCoords = true;
       this.props.w = value;
       this.setUpdateType(UpdateType.Local);
 
@@ -1819,7 +1813,6 @@ export class CoreNode extends EventEmitter {
 
   set h(value: number) {
     if (this.props.h !== value) {
-      this.updateTextureCoords = true;
       this.props.h = value;
       this.setUpdateType(UpdateType.Local);
 
@@ -2430,6 +2423,9 @@ export class CoreNode extends EventEmitter {
 
   set textureOptions(value: TextureOptions) {
     this.props.textureOptions = value;
+    if (this.stage.calculateTextureCoord === true && value !== null) {
+      this.textureCoords = this.stage.renderer.getTextureCoords!(this);
+    }
   }
 
   get textureOptions(): TextureOptions {

src/core/lib/WebGlContextWrapper.ts

@@ -73,6 +73,7 @@ export class WebGlContextWrapper {
   public readonly TEXTURE_WRAP_S;
   public readonly TEXTURE_WRAP_T;
   public readonly LINEAR;
+  public readonly LINEAR_MIPMAP_LINEAR;
   public readonly CLAMP_TO_EDGE;
   public readonly RGB;
   public readonly RGBA;
@@ -164,6 +165,7 @@ export class WebGlContextWrapper {
     this.TEXTURE_WRAP_S = gl.TEXTURE_WRAP_S;
     this.TEXTURE_WRAP_T = gl.TEXTURE_WRAP_T;
     this.LINEAR = gl.LINEAR;
+    this.LINEAR_MIPMAP_LINEAR = gl.LINEAR_MIPMAP_LINEAR;
     this.CLAMP_TO_EDGE = gl.CLAMP_TO_EDGE;
     this.RGB = gl.RGB;
     this.RGBA = gl.RGBA;