Merge pull request #8964 from NalinDalal/main

Fix framebuffer texture size clamping using wrong pixel density

Author: davepagurek

src/webgl/p5.Framebuffer.js

@@ -93,7 +93,7 @@ class Framebuffer {
     this.density = settings.density || this.renderer._pixelDensity;
     if (settings.width && settings.height) {
       const dimensions =
-        this.renderer._adjustDimensions(settings.width, settings.height);
+        this.renderer._adjustDimensions(settings.width, settings.height, this.density);
       this.width = dimensions.adjustedWidth;
       this.height = dimensions.adjustedHeight;
       this._autoSized = false;
@@ -175,7 +175,7 @@ class Framebuffer {
   resize(width, height) {
     this._autoSized = false;
     const dimensions =
-      this.renderer._adjustDimensions(width, height);
+      this.renderer._adjustDimensions(width, height, this.density);
     width = dimensions.adjustedWidth;
     height = dimensions.adjustedHeight;
     this.width = width;

src/webgl/p5.RendererGL.js

@@ -441,14 +441,14 @@ class RendererGL extends Renderer3D {
     return gl.getParameter(gl.MAX_TEXTURE_SIZE);
   }
 
-  _adjustDimensions(width, height) {
+  _adjustDimensions(width, height, density = this._pixelDensity) {
     if (!this._maxTextureSize) {
       this._maxTextureSize = this._getMaxTextureSize();
     }
     let maxTextureSize = this._maxTextureSize;
 
     let maxAllowedPixelDimensions = Math.floor(
-      maxTextureSize / this._pixelDensity
+      maxTextureSize / density
     );
     let adjustedWidth = Math.min(width, maxAllowedPixelDimensions);
     let adjustedHeight = Math.min(height, maxAllowedPixelDimensions);