Help with my xml coding

zireko · 04/15/14, 08:09 AM

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?

Lua Code:

<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 · 04/15/14, 09:04 AM

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 · 04/15/14, 09:23 AM

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.

Lua Code:

<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 · 04/15/14, 09:23 AM

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 · 04/15/14, 10:10 AM

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 · 04/15/14, 10:21 AM

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

Lua Code:

<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 · 04/15/14, 10:40 AM

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.

Lua Code:

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

Garkin · 04/15/14, 10:56 AM

Format is:
labelname:SetColor(red, green, blue, alpha)

Lua Code:

QuestMasterquest1:SetColor(128, 128, 128, 1)

For colors see http://www.colorpicker.com/.
Alpha is number between 0 and 1. Where 0 means transparent and 1 fully visible.

Ouijan · 04/15/14, 10:59 AM

I'm not sure but if you go looking in the controls API there is likely to be a SetColor() or something

and you will need to change the <OnMouseDown> to be inside each <label> At the moment it will run when you click the frame not each label individually. You will also need to give your function the name of which label you have clicked on so it knows which one to change, have a look at sending parameters to functions in lua for that

It's also a good plan to pre think things before you jump into coding. When I click this what exactly do I want to happen behind the scenes it saves a lot of time later

Most of what your looking for can be found in a quick google search

you'll learn a lot quicker and feel like you've accomplished more if ya go digging

zireko · 04/15/14, 01:46 PM

This is my xml and lua. I will show a few images of what I'm getting.

XML

Lua Code:

<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>

LUA

Lua Code:

QuestMasterquest1:SetColor(128, 128, 128, 1)

zireko · 04/15/14, 02:10 PM

Originally Posted by zireko

This is my xml and lua. I will show a few images of what I'm getting.

Also when I click on text it does not change I used the code the way I thought it might go. But something feels like its missing and I don't know what it is.

GetBackYouPansy · 04/15/14, 02:16 PM

Originally Posted by Ouijan

You will also need to give your function the name of which label you have clicked on so it knows which one to change, have a look at sending parameters to functions in lua for that

The On* event script blocks defined in XML have a "local self" so you don't need to pass names around, you can pass the thing itself

See the ScriptArguments at http://wiki.esoui.com/UI_XML#OnUpdate for example, or an example in the wild here: link.

Garkin · 04/15/14, 03:32 PM

Please check this addon: http://www.esoui.com/downloads/info2...AddonTest.html

This addon is well commented tutorial for beginners. Just download it and read through the code.

zireko · 04/15/14, 04:17 PM

Originally Posted by Garkin

Please check this addon: http://www.esoui.com/downloads/info2...AddonTest.html

This addon is well commented tutorial for beginners. Just download it and read through the code.

Okies thank you. I just wish some people would make a few tutorials making a few different addons for eso. I tend to learn a lot better when I hear someone explaining things to me. This is as close as I can get currently so I'm reading through the insAddon coding a few times over and trying to comprehend it. Thanks again for the info and all the help.

Banana Attack

lol sorry had to.

Ouijan · 04/15/14, 06:27 PM

Originally Posted by GetBackYouPansy

The On* event script blocks defined in XML have a "local self" so you don't need to pass names around, you can pass the thing itself

See the ScriptArguments at http://wiki.esoui.com/UI_XML#OnUpdate for example, or an example in the wild here: link.

Thanks GetBack, I'll have to re-investigate this I thought I had troubles before with the self referencing my addon table rather each individual frame

From memory I was calling AddonName:Function(...) in an OnMoveStop and it would return my AddonName table. :P so I defined a FrameName:Function(...) as a work around. I might post my own thread if I get stuck

Edit// just had a look at your example and I get ya now xD