fix: Remounting canvas

[cjboy76]

Close #302
Appreciate you for your incredible work.
I found that while remounting the canvas, the perspective camera isn't rendered so that it causes this issue.

Also the condition with firstCamera is not needed because setFirstCamera only pushes camera if there is no cameras in state, perhaps there is some reasons why doing this.
Let me know any further updates.

[netlify]

✅ Deploy Preview for tresjs-docs ready!

Name	Link
🔨 Latest commit	ed0754c
🔍 Latest deploy log	https://app.netlify.com/sites/tresjs-docs/deploys/648de139b2c88b000878d037
😎 Deploy Preview	https://deploy-preview-306--tresjs-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[alvarosabu]

Hi @cjboy76 first of all thanks for taking the time to create this PR. The solution does fix the issue of remounting but unfortunately breaks the HMR.

Although this is our fault because we recently figured out that using Composables like useCamera that access the provider context state shouldn't live inside the custom renderer nodeOpts (Composables need a component setup context which is lost constantly).

I'm gonna refactor the camera logic on #307, hope you don't mind if I close this PR.

[cjboy76]

@alvarosabu No problem, thanks for your reply 👍

[alvarosabu]

@cjboy76 fixed here #308