Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UnitFrames fader does not respect correct distance to target #1259

Closed
3 of 4 tasks
WiredUK opened this issue Jul 25, 2024 · 8 comments
Closed
3 of 4 tasks

UnitFrames fader does not respect correct distance to target #1259

WiredUK opened this issue Jul 25, 2024 · 8 comments

Comments

@WiredUK
Copy link

WiredUK commented Jul 25, 2024

Troubleshooting Steps

Note the target unitframe is faded out despite me being 25-30 yards from the target. I can take a step closer and it will fade back in again. I have installed RangeDisplay addon to show the range, shown just above the target unitframe in the screenshot, hope that helps.

Not sure if relevant, but I have the new Phantom Reach priest talent to increase range by 15% so I am within range at ~45 feet.

image

Describe the expected behavior and what actually happened?

When the range setting for fader on the target UnitFrame, the range is not accurate as to when it fades out. Note the distance doesn't seem to be consistent either. Very limited testing seems to indicate the physical size of the mob might play a factor, the larger the mob, the closer I need to be to unfade the frame, but that could be a red herring.

image

Reproducing the issue.

No response

Lua Errors

No response

Verification Steps

  • I have read the Changelog and verified ElvUI is up to date by checking /estatus
  • I have verified my issue doesn't exist in the Default UI or the Issue Tracker.
  • I have not read any of these steps, please close my issue when you see it.
  • I have done the Troubleshooting Steps and included the requested screenshots, which show the issue and status panel.
@WiredUK WiredUK changed the title UnitFrams fader does not respect correct distance to target UnitFrames fader does not respect correct distance to target Jul 25, 2024
@kodewdle
Copy link
Member

post talent page

@WiredUK
Copy link
Author

WiredUK commented Jul 25, 2024

@kodewdle This is my talent build:

CIQAJSbRac/22NxZbHdYKOQzcMwYGDAAAAAAAAAAAAwMsNGzYWmNmZmZ2mZmF4B2YmZmxsxAjxMmFz2UzMYBmBAAAAAA

image

@HerrNirmas
Copy link

Same for Hunter Range seem 10m to short

@WiredUK
Copy link
Author

WiredUK commented Jul 27, 2024

@HerrNirmas Yes, same on my hunter too. Also confirmed on a warlock and a MoP Remix druid.

@WiredUK
Copy link
Author

WiredUK commented Jul 28, 2024

I think I have figured out the problem. In ElvUI_Libraries/Core/LibRangeCheck-3.0/LibRangeCheck on line 86 we have this line:

local GetSpellBookItemName = _G.GetSpellBookItemName or function(index, bookType)

If I change it to be this, the range check works:

local GetSpellBookItemName = function(index, bookType)

So it seems as if something else is setting up _G.GetSpellBookItemName to do something different.

Also, the official repo for LibRangeCheck seems to have some very different code in there now, so perhaps this is as simple as updating to the latest version.

EDIT: I can confirm that replacing the local lib with the version from the original, minor version 21, (and renaming to "LibRangeCheck-3.0-ElvUI" of course) also fixes this problem too.

@kodewdle
Copy link
Member

kodewdle commented Jul 31, 2024

Thanks @WiredUK we actually have a fork of thiers, there was an update to theirs again (WeakAuras/LibRangeCheck-3.0@7c73357) which we accounted for (ed65ca4) and fixed a bug in dev (a888856). please retry and report back :)

dev version: elvui-main.zip

@WiredUK
Copy link
Author

WiredUK commented Aug 2, 2024

dev version: elvui-main.zip

Works great, thanks @kodewdle

@kodewdle
Copy link
Member

kodewdle commented Aug 7, 2024

fixed in 13.73

@kodewdle kodewdle closed this as completed Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants