xiconfjs/Gladdy-TBC
1
0
Fork 0
Code Issues 36 Pull Requests 2 Packages Projects Releases 30 Wiki Activity

Fix support for ElvUI with TotemPlates #81

Closed
Flamanis wants to merge 1 commits from main into main
pull from: main
merge into: xiconfjs:main
Conversation 4 Commits 1 Files Changed 1 +14 -2
Flamanis commented 2023-09-27 22:33:03 +02:00 (Migrated from github.com)
Copy Link

ElvUI does not force show their nameplates anymore as it can cause blocked actions. So TotemPlates would perma hide any ElvUI nameplate that was attached to a totem until a reload. This causes erroneous bug reports in ElvUI support channels from Gladdy users.

This forces a call to show when the TotemPlate is removed.

ElvUI does not force show their nameplates anymore as it can cause blocked actions. So TotemPlates would perma hide any ElvUI nameplate that was attached to a totem until a reload. This causes erroneous bug reports in ElvUI support channels from Gladdy users. This forces a call to show when the TotemPlate is removed.
XiconQoo commented 2023-09-29 20:51:21 +02:00 (Migrated from github.com)
Copy Link

See c122e24054 for latest fix. I modified your commit a bit using the function ToggleAddon. Thanks for this PR. Gonna close it now

See https://github.com/XiconQoo/Gladdy-TBC/commit/c122e2405498bf1753aa464f25a1fe789ed07cef for latest fix. I modified your commit a bit using the function `ToggleAddon`. Thanks for this PR. Gonna close it now
Flamanis commented 2023-09-29 21:23:34 +02:00 (Migrated from github.com)
Copy Link

Awesome! I didn't put too much effort into this fix as I don't personally pvp, but there were quite a few reports in elvui about it so I cobbled something together and had some users test. Glad you did something you find better :)

Thanks for the update.

Awesome! I didn't put too much effort into this fix as I don't personally pvp, but there were quite a few reports in elvui about it so I cobbled something together and had some users test. Glad you did something you find better :) Thanks for the update.
Flamanis commented 2023-09-30 18:17:37 +02:00 (Migrated from github.com)
Copy Link

Your change now causes issues with Plater users. It keeps the nameplate on screen after totemplates is over.

Your change now causes issues with Plater users. It keeps the nameplate on screen after totemplates is over.
XiconQoo commented 2023-09-30 20:50:25 +02:00 (Migrated from github.com)
Copy Link

Thanks again, just updated it.

Thanks again, just updated it.

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working
documentation
Improvements or additions to documentation
duplicate
This issue or pull request already exists
enhancement
New feature or request
feature request
good first issue
Good for newcomers
help wanted
Extra attention is needed
invalid
This doesn't seem right
question
Further information is requested
wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiconfjs/Gladdy-TBC#81
No description provided.
Delete Branch "main"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?