Issue with altering frame point

[Mayron]

Hi everyone,

I have an issue where if I use this to mvoe a frames position:

local point, relativeFrame, relativePoint, ofsx, ofsy = mui_double:GetPoint()   
mui_double:SetPoint(value, relativeFrame, relativePoint, ofsx, ofsy)

then when I run it a second time without reloading the UI then it has no effect on the frame.

Has anyone else encountered this issue before and know a fix for it? I did not know you cannot alter a frames point only once and a reload is needed to fix it as that is the issue.

[Malsomnus]

Please include more code. To start with, the code that defines the value variable.

[Phanx]

Without seeing the rest of your code -- come on, you've been around here long enough to know better -- I can only guess that your SetPoint call is failing because you aren't calling ClearAllPoints first.

[Mayron]

Originally Posted by Malsomnus Please include more code. To start with, the code that defines the value variable.

Ok, sure. I think this is the only relevant code however I will upload the file anyway if that helps:

Panels = { -- This is from the MayronUI.db.profile table
    UnitFrameBG = {
        Width = 752,
        Height = 180,
        ofsx = '0',
        ofsy = '-110',
        point = 'BOTTOM',
        relativeFrame = 'mui_actionBars',
        relativePoint = "TOP",
        frameStrata = 'BACKGROUND',
        frameLevel = 5,
    },
 
 
local p = MayronUI.db.profile.Panels
mui_single = CreateFrame("Frame", "mui_single", mui_menuBar)
mui_single:SetPoint(p.UnitFrameBG.point, p.UnitFrameBG.relativeFrame, p.UnitFrameBG.relativePoint, p.UnitFrameBG.ofsx, p.UnitFrameBG.ofsy)
-- There are more methods being used for this object but did not include them 
 
point = { -- This option is for altering the frames point in a drop down list
    order = 5,
    type = "select",
    name = "Frame Point",
    get = function()
        return MayronUI.db.profile.Panels.UnitFrameBG.point
    end,
    set = function(info, value)
        MayronUI.db.profile.Panels.UnitFrameBG.point = value
        local point, relativeFrame, relativePoint, ofsx, ofsy = mui_single:GetPoint()
            mui_single:SetPoint(value, relativeFrame, relativePoint, ofsx, ofsy)
 
        local point, relativeFrame, relativePoint, ofsx, ofsy = mui_double:GetPoint()   
            mui_double:SetPoint(value, relativeFrame, relativePoint, ofsx, ofsy)
    end,
    values = {
        TOP = "Top",
        TOPLEFT = "Top Left",
        TOPRIGHT = "Top Right",
        BOTTOM = "Bottom",
        BOTTOMLEFT = "Bottom Left",
        BOTTOMRIGHT = "Bottom Right",
        LEFT = "Left",
        RIGHT = "Right",
        CENTER = "Center",
    },
    style = "dropdown",
},

[Mayron]

Originally Posted by Phanx Without seeing the rest of your code -- come on, you've been around here long enough to know better -- I can only guess that your SetPoint call is failing because you aren't calling ClearAllPoints first.

I thought it would be more simple hehe, sorry. And ok I will look into that

[Mayron]

Yet another simple error. Phanx is right its because I am not using ClearAllPoints() first. I thought it was a very simple error which is why I did not post much code before but sorry about that

Thank you!

[Seerah]

To clarify, a frame can have more than one anchor. If you don't call :ClearAllPoints(), you're just adding an additional anchor to your frame.

[Malsomnus]

Originally Posted by Seerah To clarify, a frame can have more than one anchor. If you don't call :ClearAllPoints(), you're just adding an additional anchor to your frame.

Wait, but you CAN use code like the original poster's for moving an existing anchor point, right...?

[Mayron]

Originally Posted by Seerah To clarify, a frame can have more than one anchor. If you don't call :ClearAllPoints(), you're just adding an additional anchor to your frame.

Don't worry I do know this. I dont actually want to remove all, I just wanted to redo one of the anchors with new values but if I alter "BOTTOM" point for example then I cannot alter it a second time which I thought was odd. Its just for selecting a new anchor point with a drop down menu.

[Lombra]

You can reset the same pointing without clearing, yes. Your code, though, as far as I understand, should not be expected to do anything in this case.

local point, relativeFrame, relativePoint, ofsx, ofsy = mui_double:GetPoint()  
            mui_double:SetPoint(value, relativeFrame, relativePoint, ofsx, ofsy)

You take the point value from the dropdown selection, and the rest from the existing point. If the dropdown selection equals the existing point, that means all the arguments passed to SetPoint as the same as the returns from GetPoint, and you're setting the exact same point again.

[Mayron]

Originally Posted by Lombra You can reset the same pointing without clearing, yes. Your code, though, as far as I understand, should not be expected to do anything in this case. local point, relativeFrame, relativePoint, ofsx, ofsy = mui_double:GetPoint() mui_double:SetPoint(value, relativeFrame, relativePoint, ofsx, ofsy) You take the point value from the dropdown selection, and the rest from the existing point. If the dropdown selection equals the existing point, that means all the arguments passed to SetPoint as the same as the returns from GetPoint, and you're setting the exact same point again.

That is what I assumed as well but it just would not work which makes no sense to me. I will retry but for some reason when I do this method, the second time I do it, nothing changes.

[Phanx]

If you set, for example "BOTTOMRIGHT to the CENTER of the UIParent" and then "BOTTOM to the CENTER of the UIParent", technically you have set both of those points, and you should be able to retrieve them via GetPoints(1) and GetPoints (2), but practically you won't see a differerence, because how can two different parts of your frame both be exactly in the center of the UIParent? That's why you need to use ClearAllPoints.

[Mayron]

Originally Posted by Phanx If you set, for example "BOTTOMRIGHT to the CENTER of the UIParent" and then "BOTTOM to the CENTER of the UIParent", technically you have set both of those points, and you should be able to retrieve them via GetPoints(1) and GetPoints (2), but practically you won't see a differerence, because how can two different parts of your frame both be exactly in the center of the UIParent? That's why you need to use ClearAllPoints.

Yes thats true. I fixed the issue in the end just by altering how the setting worked in other ways and controlled the points better while taking into consideration all the tips I was given so thank you and everyone else

[pelf]

Originally Posted by Phanx [...] because how can two different parts of your frame both be exactly in the center of the UIParent?

The client should be folding the frame's bottom middle and bottom right together and then attaching that to UIParent's center.