windowResized() not firing

[stephanbogner]

Expected Behavior

p5.windowResized = () => {} should be triggered when the window is resized

Actual Behavior

Doesn't fire. Should it be supported or is this by design/limitation?

Steps to Reproduce the Problem

Note: I am using NextJS but I assume this is not an issue specific to @p5-wrapper/next

import * as React from "react"
import { NextReactP5Wrapper } from "@p5-wrapper/next"

function sketch(p5) {
    p5.setup = () => p5.createCanvas(600, 400, p5.WEBGL)

    p5.draw = () => {
        p5.background(250)
        p5.normalMaterial()
        p5.push()
        p5.rotateZ(p5.frameCount * 0.01)
        p5.rotateX(p5.frameCount * 0.01)
        p5.rotateY(p5.frameCount * 0.01)
        p5.plane(100)
        p5.pop()
    }

    p5.windowResized = () => {
        console.log('resized')
    }
}

export default function({ width, height }) {
  return <NextReactP5Wrapper sketch={sketch} />
}

Specifications

On Mac using Brave
"@p5-wrapper/next": "^0.2.0"
"@p5-wrapper/react": "^4.2.0"

[stephanbogner]

For now I am using the workaround document.addEventListener('resize', p5.windowResized) inside p5.setup

[jamesrweb]

There's no limitation on p5 in this package. This package simply wraps the p5 value constructed via instance mode.

You should check upstream if you can't see the function being correctly exposed. If it is correctly exposed and you say that your "workaround" works though, I would assume it's a next issue OR that the instance mode event handler works differently when compared with global mode.

[jamesrweb]

Closing issue due to inactivity and reasons outlined above.

[stephanbogner]

@jamesrweb Sorry, I didn't have time yet to check if it works without Nextjs. Thanks for clarifying and explaining 👍