Help with my xml coding

[zireko]

This is my xml code so far. What I'm trying to do is have a box with a list of quests in it and later in my lua I want to make each quest clickable so that the text well turn to a dark gray and if they click it again it'll go back to the original color. I currently have just two quest in it. How I want it to look is like this example below.

A Pinch of Sugar
Cast Adrift

My question is if my code has errors and if so what do I need to do to fix them so far?

<GuiXml>
    <Controls>
        <TopLevelControl name="QuestMaster" mouseEnabled="true" movable="true">
            <Dimensions x="300" y="400" />
            <Anchor point="CENTER" />
 
            <OnUpdate>
                QuestMasterUpdate()
            </OnUpdate>
 
            <OnMouseDown>
                QuestMasterReset()
            </OnMouseDown>
 
            <Controls>
                <Backdrop name="$(parent)BG" inherits="ZO_ThinBackdrop" />
                <Label name="$(parent)" font="ZoFontAnnounceSmall" color="CFDCBD" wrapMode="ELLIPSIS" verticalAlignment="CENTER" text="A Pinch of Sugar">
                <Label name="$(parent)" font="ZoFontWindowSmall" color="CFDCBD" wrapMode="ELLIPSIS" verticalAlignment="CENTER" text="Cast Adrift">
                    <AnchorFill />
                </Label>
            </Controls>
        </TopLevelControl>
    </Controls>
</GuiXml>

[Ouijan]

I can't test it right now (Server is down) but I'm pretty sure u need to close your first <label> tag and add an <AnchorFill /> to it To display them they way you want you will have to look into anchors and offsets you could either just offset them each a certain distance OR anchor the top of one to the bottom of the one above it which is a bit trickier

As for the functions what do you plan to do with QuestMasterUpdate() and QuestMasterReset()?

Edit: also you need to give your <Label>s different names currently both of them will be named "QuestMaster"

[zireko]

Here is my updated code so far. And as for the QuestMasterUpdate & QuestMasterReset I'm not sure yet I want to make the text change colors to dark gray and if they click on it again turn back to the original color. I don't know exactly if I need to pull the update and reset out or if they can be used for the text color.

<GuiXml>
    <Controls>
        <TopLevelControl name="QuestMaster" mouseEnabled="true" movable="true">
            <Dimensions x="300" y="400" />
            <Anchor point="CENTER" />
 
            <OnUpdate>
                QuestMasterUpdate()
            </OnUpdate>
 
            <OnMouseDown>
                QuestMasterReset()
            </OnMouseDown>
 
            <Controls>
                <Backdrop name="$(parent)BG" inherits="ZO_ThinBackdrop" />
                <Label name="$(parent)" font="ZoFontAnnounceSmall" color="CFDCBD" wrapMode="ELLIPSIS" verticalAlignment="CENTER" text="A Pinch of Sugar">
                    <AnchorFill />
                </Label>
                <Label name="$(parent)" font="ZoFontWindowSmall" color="CFDCBD" wrapMode="ELLIPSIS" verticalAlignment="CENTER" text="Cast Adrift">
                    <AnchorFill />
                </Label>
            </Controls>
        </TopLevelControl>
    </Controls>
</GuiXml>

[Total-Khaos]

Originally Posted by Ouijan I can't test it right now (Server is down) but I'm pretty sure u need to close your first <label> tag and add an <AnchorFill /> to it To display them they way you want you will have to look into anchors and offsets you could either just offset them each a certain distance OR anchor the top of one to the bottom of the one above it which is a bit trickier As for the functions what do you plan to do with QuestMasterUpdate() and QuestMasterReset()? Edit: also you need to give your <Label>s different names currently both of them will be named "QuestMaster"

You saw the same things I did.

[Ouijan]

Originally Posted by Total-Khaos You saw the same things I did.

tehe

@Zireko
Cool man getting there! you need to give EVERYTHING on the screen a different name otherwise the game will have issues drawing it (learnt from experience xD) so where you have each <Label name="$(parent)"> it is essentially saying <Label name="QuestMaster"> because the parent frame of each label is your TopLevelControl which is named "QuestMaster". Currently your backdrop will be named it "QuestMasterBG" which is good, so stick something after the "$(parent)" in your <Label>s

The name of each frame is its identifier, you cant have two frames with the same name or the ui freaks out and doesn't know which frame to do what to :P

For the functions you only really need to have one function in the <OnMouseDown> (or <OnMouseUp> your choice) to toggle between two colours when clicked at the moment you don't need an OnUpdate

[zireko]

Ok I think I understand. So I've updated the code again. How does it look now?

<GuiXml>
    <Controls>
        <TopLevelControl name="QuestMaster" mouseEnabled="true" movable="true">
            <Dimensions x="300" y="400" />
            <Anchor point="CENTER" />        
 
            <OnMouseDown>
                QuestMasterReset()
            </OnMouseDown>
 
            <Controls>
                <Backdrop name="$(parent)BG" inherits="ZO_ThinBackdrop" />
                <Label name="$(parent)quest1" font="ZoFontAnnounceSmall" color="CFDCBD" wrapMode="ELLIPSIS" verticalAlignment="CENTER" text="A Pinch of Sugar">
                    <AnchorFill />
                </Label>
                <Label name="$(parent)quest2" font="ZoFontWindowSmall" color="CFDCBD" wrapMode="ELLIPSIS" verticalAlignment="CENTER" text="Cast Adrift">
                    <AnchorFill />
                </Label>
            </Controls>
        </TopLevelControl>
    </Controls>
</GuiXml>

[zireko]

This is the little lua I have currently I know it's geared for the counter from the tutorial. But I have taken out the Update part because I do not need it. What I'm asking here is how do I change this code so that instead of reset it would change the color to gray text instead.

local counter = 1
  
function QuestMasterReset()
    counter = 0
end