CanvasRenderer using incorrect asset from spritesheet about tilemap

[ivanpopelyshev]

oh right because Tilemap doesnt have tileAnim by default... :) Just register it, it costs nothing anyway

[ShukantPal]

I'll look into this today. Was gone on vacation.

[ShukantPal]

https://github.com/pixijs/tilemap/releases/tag/v3.2.1

[ivanpopelyshev]

OK, found bug, if you want to try fast fix, apply this to your @pixi/tilemap file in node_modules or where is it 7938df8

also, you wont need to register canvas renderer after this

[jake]

thanks for the prompt response! that does seem to work, but it seems i still need the registerPlugin call for it to work.

if I remove the PIXI.CanvasRenderer.registerPlugin('tilemap', CanvasTileRenderer); then I get a separate error I got prior as well (tileAnim is undefined):

when I inspect the breakpoint there, it seems my tileAnim is actually null. presumably since both this.tileAnim is null and renderer.plugins.tilemap.tileAnim is undefined.

perhaps registerPlugin is still just required? or maybe the tileAnim fallback a few lines up is missing something?

[jake]

great! btw, it took me a bit of sleuthing to realize I needed to register that. seems like its only documented in the actual tilemap/src/CanvasTileRenderer.ts source. a section in the readme on "enabling canvas support" might be worthwhile. I'd write it but honestly I'm not sure I have a full enough grasp on how it works just yet :)

[ivanpopelyshev]

Anyway, its fixed in latest release, but still needs to register plugin
thank you for the report!

[jake]

sounds good. when will it be published to npm?

[ivanpopelyshev]

it wasnt? Oh, I just have to call @ShukantPal again. I cant build new version for some reason :(

[jake]

correct, on github and npm the latest version is still 3.2.0

[jake]

@ShukantPal sorry to be annoying, but any update on publishing a new build to npm? hoping to get this integrated in my game upstream.

thanks!