3dstreet icon indicating copy to clipboard operation
3dstreet copied to clipboard

error to undo material texture src change

Open kfarr opened this issue 1 year ago β€’ 5 comments

[object HTMLImageEleme...] instead of #actual-id after undo of texture change

https://github.com/user-attachments/assets/49db072d-0c72-43bd-b2a8-b65709dde6da

kfarr avatar Sep 05 '24 04:09 kfarr

Probably related to the entity serialization for asset properties. We did a fix related to that in aframe master https://github.com/aframevr/aframe/pull/5544 Let's see if we can make a workaround just to fix it without updating to aframe master.

vincentfretin avatar Sep 05 '24 08:09 vincentfretin

If in index.html you use an aframe master build (three r167) <script src="https://cdn.jsdelivr.net/gh/aframevr/aframe@4142ec90a76916d252e375ceb1595971ee264966/dist/aframe-master.min.js"></script> the issue is fixed. Really we should just update aframe, not worth the time to try to fix that in an old version.

vincentfretin avatar Sep 05 '24 09:09 vincentfretin

#842

vincentfretin avatar Sep 05 '24 09:09 vincentfretin

We can do a custom 1.5.0 build with just the change from https://github.com/aframevr/aframe/pull/5544 that should be safer. I looked at the code, that's not possible to monkey patch propertyTypes, this object is not exposed.

vincentfretin avatar Sep 07 '24 12:09 vincentfretin

Reoponed because we reverted the aframe update in #850 and pending a fix for #851 before updating it again.

vincentfretin avatar Sep 11 '24 08:09 vincentfretin