sArena Updated 2 by sammers

sArena Updated 2 by sammers

211k Downloads

Absorbs are not being shown properly

Sammers21 opened this issue · 11 comments

commented

They are not being shown at all atm.

commented

@Sammers21 Do you have an example of what it should show? Just to get an idea of what is not working and how it should be displayed when working

commented

@devai42coding ait thank you. Would be nice

commented

It looks like the whole structure is too densed into a small thing
image

It should look more like:
image

@devai42coding here is the vod where i play with the new version https://www.twitch.tv/videos/2205909170

commented

@Sammers21 You mean the texture is different on sarena and party frames? Yes unfortunately implementing the texture (two textures) actually that is implemented on the party frames was causing issues, so I went with the shield-fill texture just to get the absorb implementation in a working state.

commented

@devai42coding is it possible to use the same texture as on party frames? Its been looking the same before it broke

commented

@Sammers21 It is possible yes - Will take a bit to implement, but should be easier to test now since new patch is live. I will give it a look when possible.

commented

image
@devai42coding, this is the example of the absorb texture

commented

Hi @Sammers21

I have had a look at the API changes and it indeed look like they removed the used templates back then.

However, it looks like they have added the templates back on Beta.

These were the templates on 10.2.0 where it worked before the breaking changes in 10.2.5:

https://github.com/Gethe/wow-ui-source/blob/10.2.0/Interface/FrameXML/UnitFrame.xml

These are the templates on live:

https://github.com/Gethe/wow-ui-source/blob/live/Interface/AddOns/Blizzard_UnitFrame/UnitFrame.xml

These are the templates on beta:

https://github.com/Gethe/wow-ui-source/blob/beta/Interface/AddOns/Blizzard_UnitFrame/UnitFrame.xml

This shows the templates that were used with sArena back then are readded back.

I have reverted the changes you did back then to solve the XML issues below:

devai42coding@a57ae0f

I have not been able to test this yet at all other than implementing the changes and logging in on beta to see there are no BugSack errors. I Think we should try to test this to see if it works again since the removed templates are back.

commented

I just got to test it on Beta. Unfortunately it did not work - There are no lua errors, but it seems the functionality somewhere must have changed.

commented

@Sammers21 Is this better?
image

image

PR created in case it's what you wanted: #11

commented

@Sammers21 - Created a PR for absorb implementation that have worked for me in the last 3 lobbies. Might still need some work, but it’s a start. PR also fixes other issues #10