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

Fix: Editmode unable to save issue #53

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

dingoh
Copy link

@dingoh dingoh commented Aug 5, 2024

Don't hook :hideFrame when going into EditMode.

(I am the same person who posted this- https://www.wowace.com/projects/shadowed-unit-frames/issues/2102)

Don't hook :hideFrame when going into EditMode.
dingoh

This comment was marked as resolved.

ShadowedUnitFrames.lua Outdated Show resolved Hide resolved
hideBlizzardFrames()
Update ShadowedUnitFrames.lua
Replace deprecated function UnitDebuff to C_UnitAuras.GetDebuffDataByIndex
Replace deprecated function UnitDebuff to C_UnitAuras.GetDebuffDataByIndex
@Nevcairiel
Copy link
Owner

I'm not able to observe any issues with EditMode snapping or saving in current WoW with current SUF. Can you explain how to reproduce the issue?

@dingoh
Copy link
Author

dingoh commented Aug 16, 2024

I'm not able to observe any issues with EditMode snapping or saving in current WoW with current SUF. Can you explain how to reproduce the issue?

The issue occurs when I check any 'Hide Blizz frame' and enter to Edit mode.

Detail of my settings:
Unit Enable : Player, Target, Target of target, Focus, Target of focus, Boss
Blizz Disable : Target, Boss, Focus

Details of the issue:
When you move any frame in Editmode, the error occurs(regarding SUF), and then you can not save current editing setttings at all.

*This issue has been quite long time for me and some of my friends like since DF Season 3

@nulian
Copy link

nulian commented Aug 16, 2024

Confirm the same for me partially hidden the bliz frames use some parts then it breaks for me without this change.

@dingoh
Copy link
Author

dingoh commented Aug 17, 2024

@Nevcairiel

Please watch this https://www.youtube.com/watch?v=UpHkO9CobMU

I had forgotten to mention the keystone of the error earlier. That was 'Snap to Elements' option in the Edit mode. You can find that in this video.

Use unpack function for packed return data (because of replacing depreated function; UnitDebuff )
Use unpack function for packed return data
Set previous parents of frames before enter Editmode.
Copy link
Author

@dingoh dingoh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This(set old_parent for every frames in ShadowedUnitFrames.lua) is essential fix.
I missed it when I upload code from my local repo to my github repo.

I set the parent before enter Blizz's Editmode.
It is necessary for every frame has their parents to save Edit mode when you enable 'Snap to elements' while SUF hides some blizz frames.

(And sorry for my poor skills to use git. I am so far from programming stuffs.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants