Added flipX and flipY to loadSpriteFromXML()

[THEkatinamicrowave]

why was this not already implemented? Nevermind I'm pretty sure it's because parseBool isn't a thing

[THEkatinamicrowave]

NOTE: this has NOT been playtested yet

[NeeEoo]

Your repo is outdated, and removed features, please fix it before we merge

[THEkatinamicrowave]

Your repo is outdated, and removed features, please fix it before we merge

noted. fixing stuff rn

[THEkatinamicrowave]

is dat gooood :>

[NeeEoo]

Id move it down to above

if (node.has.updateHitbox && node.att.updateHitbox == "true") spr.updateHitbox();

and id make it like

if (node.has.flipX && node.att.flipX == "true") spr.flipX = true;
if (node.has.flipY && node.att.flipY == "true") spr.flipY = true;

[SenDoesStuff]

Make sure this works with characters as well. Iirc characters have a different useage of flipX from most sprites in PlayState

[THEkatinamicrowave]

Make sure this works with characters as well. Iirc characters have a different useage of flipX from most sprites in PlayState

pretty sure that characters are added and handled differently than sprites. Sprites are FunkinSprite objects; characters are just extents of the Character class. I will double check though in case I'm dead wrong (feel free to laugh at and kill me)

[SenDoesStuff]

Make sure this works with characters as well. Iirc characters have a different useage of flipX from most sprites in PlayState

pretty sure that characters are added and handled differently than sprites. Sprites are FunkinSprite objects; characters are just extents of the Character class. I will double check though in case I'm dead wrong (feel free to laugh at and kill me)

You’re correct, but the nodes for character positioning in stage xmls uses the same flipX from XMLUtils

[THEkatinamicrowave]

I'm compiling it rn; if it works it works, if it doesn't I'll fix it
I'll update as soon as I can and realize something's wrong

[THEkatinamicrowave]

sorry it's taking so long to proofread; I had to install hscript-improved and ran update.bat which is taking forever

[NeeEoo]

I also said it put it above updateHitbox

[THEkatinamicrowave]

k last night my computer was being fucky wucky but now I can actually test this and see if it works

[NexIsDumb]

eh no need to check lowercase aswell, i mean look at updateHitbox
but tyy, imma prob merge it in the future

[NexIsDumb]

aight peak gameplay