"close" button inside draggable movie clip

I tried it this way and it all works!!!!!

//remove video box...
var closevid_btn:btnclose = new btnclose();

closevid_btn.addEventListener(MouseEvent.CLICK, closeThis);

videobox1_mc.addChild(closevid_btn);
closevid_btn.x = 96;
closevid_btn.y = 296;

function closeThis(e:MouseEvent):void
{
removeChild(videobox1_mc);
trace("The lesson disappears");
videobox1_mc.alapvideo.stop();
trace("The audio stops");
}

still wondering if I can simplify the functions or if I just need to make 14 of them though.

yo myalkut,

try this:
function closeThis(e:MouseEvent):void
{

// instead of -----> dispatchEvent(new Event("closeMe", true));

removeChild(videobox1_mc);

}

Quote:

---

Originally Posted by ariesf829

yo myalkut,

try this:
function closeThis(e:MouseEvent):void
{

// instead of -----> dispatchEvent(new Event("closeMe", true));

removeChild(videobox1_mc);

}

---

This guy is right too. My code was meant to be done inside an instance of a class that has a parent. However, if you ever end up coding without the timeline at some point and you need to delete and instance of a class from within, my code is what you will be using.

Quote:

---

Originally Posted by myalkut

I tried it this way and it all works!!!!!

//remove video box...
var closevid_btn:btnclose = new btnclose();

closevid_btn.addEventListener(MouseEvent.CLICK, closeThis);

videobox1_mc.addChild(closevid_btn);
closevid_btn.x = 96;
closevid_btn.y = 296;

function closeThis(e:MouseEvent):void
{
removeChild(videobox1_mc);
trace("The lesson disappears");
videobox1_mc.alapvideo.stop();
trace("The audio stops");
}

still wondering if I can simplify the functions or if I just need to make 14 of them though.

---

To simplify it:

function closeThis(e:MouseEvent):void
{
removeChild(e.target.parent);
e.target.alapvideo.stop(); // That will work only if all video instances within your mcboxvideos are called alapvideo.
}

OK, that makes sense about the dispatch event code being used inside an instance of a class. I thought that might be the case as I looked into it more (I've been doing the Lynda.com action script video classes).

that makes sense to simplify the function too, but will it not cause errors to call all the different videos' instances the same name? guess I'll try it and find out

I just talked to another friend who does Flash and he said not good to use removeChild, but instead best practice to use
videobox1_mc.visible = false;
What do you think?

He says do it like this:
ques1.addEventListener(MouseEvent.CLICK, showBox);
function showBox(event:MouseEvent)
{
if (this.contains(videobox1_mc) )
{
videobox1_mc.visible = true
}
else {
addChild(videobox1_mc);
}
trace("The lesson appears");
videobox1_mc.x = 908;
videobox1_mc.y = 549;
videobox1_mc.gotoAndPlay(2); //so it fades in
}

Quote:

---

Originally Posted by myalkut

I just talked to another friend who does Flash and he said not good to use removeChild, but instead best practice to use
videobox1_mc.visible = false;
What do you think?

---

Your friend is a retard.

If you make something 'invisible' it is still in memory and being processed by 'flash'. Where as removing the movie clip and assigning a null value to the variable pointing to that movieclip will let the garbage collector get rid of the unused memory.

Beathoven is right about your friend.

right on, glad I asked