**** BEGIN LOGGING AT Tue Jan 06 02:59:59 2015
Jan 06 04:26:50 <Roy__> ...pation.
Jan 06 16:03:44 <GodGinrai> hey dmanderson, do you guys use any linters for JS?
Jan 06 16:45:00 <Roy__> GodGinrai: We use jshint
Jan 06 16:45:17 <Roy__> In fact, the grunt task with Bootplate includes a jshint task.
Jan 06 16:45:25 <GodGinrai> ah, thanks
Jan 06 16:46:07 <Roy__> It you used generator-enyo, it's ready to go.  If not, just do a 'npm install' on the bootplate directory and then install the grunt cli if you don't have it.
Jan 06 16:46:31 <GodGinrai> generator-enyo?
Jan 06 16:49:28 <Roy__> You know, our new Yeoman task for scaffolding Enyo apps?  Announced with 2.5.1?
Jan 06 16:50:52 <GodGinrai> ah
Jan 06 16:51:18 <GodGinrai> I heard about that
Jan 06 16:51:19 <GodGinrai> but honestly, I don't even really understand what the hell yeoman is
Jan 06 16:51:19 <GodGinrai> so I haven't looked into it
Jan 06 17:36:10 <Roy__> It's a scaffolding tool.
Jan 06 17:36:21 <Roy__> You really only need it when making a new Enyo project.
Jan 06 17:36:32 <Roy__> Although, it includes tools to update your Enyo version.
Jan 06 17:36:39 <GodGinrai> so it's another bootplate kind of thing?
Jan 06 17:36:51 <Roy__> Not exactly, it installs bootplate.
Jan 06 17:37:11 <GodGinrai> I see
Jan 06 17:38:22 <Roy__> It builds off a common JavaScript tool (Yeoman).  And, by doing that, we can do things like create a full Cordova project, or update Enyo+Libs to a specific version.
Jan 06 17:38:37 <Roy__> Or even add new libs on the fly.
Jan 06 17:38:48 <GodGinrai> I see
Jan 06 18:07:02 <Semanticfire_2> Roy__, did you see my comments ?
Jan 06 18:08:36 <dmanderson> morning enyoers
Jan 06 18:08:47 <Semanticfire_2> hi dmanderson
Jan 06 18:08:55 <dmanderson> Howdy Semanticfire_2!
Jan 06 18:10:47 <Semanticfire_2> Roy__, I couldn't scafold a project around a minimalistic source tree from a repository
Jan 06 18:12:37 <GodGinrai> morning dmanderson
Jan 06 18:12:59 <dmanderson> \o
Jan 06 18:28:58 <Roy__> Semanticfire_2: What do you mean?
Jan 06 18:36:44 <Semanticfire_2> Roy__, http://forums.enyojs.com/discussion/2289/yeoman-scafolding#latest
Jan 06 18:39:07 <Roy__> I will reply there when I get a chance.   Basically, you want a 'not even bootplate' option?
Jan 06 18:39:28 <Roy__> OR, are you saying you want those to be submodules?
Jan 06 18:39:46 <Roy__> Or something else altogether?
Jan 06 18:40:01 <Semanticfire_2> what I want is to be able to construct a bootplate around a minimalistic source code repo with enyo application code
Jan 06 18:40:25 <Semanticfire_2> so I check in source/app.js
Jan 06 18:40:49 <Semanticfire_2> have some grunt / bower / npm config file which I can run from a checkout of that code
Jan 06 18:41:08 <Semanticfire_2> and then end up with a tree from which I can test / run / build my enyo app
Jan 06 18:41:26 <Semanticfire_2> and all the stuff in lib / enyo etc goed in .gitignore / .svnignore
Jan 06 18:42:02 <Semanticfire_2> so a relativly simple app has like a dozen of file for source code in the repo plus some files which allow me to scaffold all the enyo stuff around it
Jan 06 18:42:23 <Semanticfire_2> the link in the message basically does that
Jan 06 18:42:23 <Semanticfire_2> you need npm + bower + grunt installed
Jan 06 18:42:42 <Semanticfire_2> checkout the code, do a 'grunt setup' and there is your enyo enviroment around the code from the repo
Jan 06 18:44:52 <dmanderson> That’s why you forked my grunt-enyo-builder right? because it just tosses the enyo stuff in an ignored directory like node-modules
Jan 06 18:45:06 <trex005> Hello Enyos, I was wondering if there is a known issue with scrims not disappearing in 2.5.1 when you hide() a popup.  None of my scrims are disappearing, but I can't seem to replicate it in jsfiddle.
Jan 06 18:45:07 <Semanticfire_2> yup
Jan 06 18:45:18 <dmanderson> Only used at build time, and not actually part of the project repo
Jan 06 18:45:33 <Semanticfire_2> exactly
Jan 06 18:45:43 <Semanticfire_2> all the other options end up cluttering my project repo
Jan 06 18:46:34 <dmanderson> trex005 I thought there was a z-index problem fixed for 2.5.1 after multiple popup scrims were inserted into the DOM - I’d have to go back and look.
Jan 06 18:51:48 <trex005> Well, I'm trying a bunch of stuff with multiple popups in jsfiddle and that's not replicating it either. :(
Jan 06 18:52:16 <trex005> This worked when I was on 2.4.0, but when I updated to 2.5.1 it just stopped working.
Jan 06 19:03:24 <trex005> I attempted calling showHideScrim(false); directly too, no luck.
Jan 06 19:04:01 <GodGinrai> trex005: you say that it works in the JSFiddle but not in your own code?
Jan 06 19:04:09 <trex005> Correct.
Jan 06 19:04:34 <GodGinrai> are you using the minified 2.5.1?
Jan 06 19:04:48 <trex005> No, I'm using the bootplate 2.5.1
Jan 06 19:05:43 <GodGinrai> hrm
Jan 06 19:06:01 <GodGinrai> trex005: I would set a breakpoint at hide() and walk through it, then
Jan 06 19:06:18 <GodGinrai> if you are using the bootplate 2.5.1, the code should be unminified, and therefore legible
Jan 06 19:19:07 <trex005> It seems like the popup fires and event when it is closed, is the scrim supposed to be listening for this?
Jan 06 19:20:06 <trex005> I'm not quite following the code on line 83, of ObserverSupport.js
Jan 06 19:28:44 <Roy__> trex005:  can you check out 2.5.1.1 and see if the problem still exists?
Jan 06 19:29:15 <trex005> Sure, I'm not using git, so it'll take me a few minutes.
Jan 06 19:30:56 <trex005> release-2.5.1-pre.5 ?
Jan 06 19:31:31 <Roy__> no, 2.5.1.1 is the tag
Jan 06 19:32:00 <trex005> Found it, thank you
Jan 06 19:39:46 <trex005> Roy__: yes, still exists in 2.5.1.1
Jan 06 19:40:56 <trex005> in Scrim.js there is a function hideAtZIndex that when called zStack.length is 3, so it never calls hide().  If I manually call hide() it works.
Jan 06 19:50:00 <trex005> So, after I call PopupName.hide(); I can call PopupName.getScrim().hide(); as a workaround, but that seems like a pretty lousy method :)
Jan 06 20:04:24 <Roy__> trex005: Agreed.  Let me see if this was fixed at some point.
Jan 06 20:04:44 <Roy__> Or if it still exists in master.
Jan 06 20:07:24 <Roy__> trex005: Are you calling showAtZIndex() or whatever the function is called?
Jan 06 20:08:00 <trex005> No, it is automatically called when I call PopupName.hide();
Jan 06 20:08:50 <trex005> sorry, not the show version, the hide version.
Jan 06 20:11:05 <trex005> I am not manually calling showAtZIndex anywhere either.
Jan 06 20:13:57 <Roy__> Are you able to create a reproduction case?
Jan 06 20:14:13 <trex005> I've tried on jsfiddle to no avail
Jan 06 20:15:05 <trex005> If you'd like access to my app, I can remove the getScrim().hide() codes and get you a link
Jan 06 20:16:20 <Roy__> trex005: Is this on Firefox?
Jan 06 20:16:53 <trex005> Chrome, I can test it on FF though
Jan 06 20:17:04 <Roy__> no, there was a bug in Firefox at one point related to scrims.
Jan 06 20:17:11 <Roy__> I wanted to rule that out.
Jan 06 20:18:03 <trex005> Well, I tested it anyway, same bug on FF
Jan 06 20:18:53 <Roy__> trex005: Yes, let me take a look.
Jan 06 20:19:17 <Roy__> Maybe I can figure out how to fix it before we push out 2.5.1.1.  I'll double-check the sampler.
Jan 06 20:19:37 <trex005> https://game.imagefaceoff.com/enyotesters When you first go to the app, it'll ask you "How you want to be known" that is the first issue of it happening.
Jan 06 20:22:30 <Roy__> I didn't see that 'how do you want to be known?
Jan 06 20:22:36 <Roy__> lemme try again.
Jan 06 20:22:51 <Roy__> It dismissed when I switched tabs I guess?
Jan 06 20:23:10 <trex005> That's okay, there are other popups you can trigger.  on the left hand side there is an angel with a +, you can click on that and close it
Jan 06 20:24:07 <trex005> Odd that you didn't get that popup though, I'll have to see if I can reproduce that :)
Jan 06 20:26:25 <sugardave> I got the popup and the scrim stuck around.  You could always work around it by binding the popup and scrim's showing properties
Jan 06 20:28:54 <trex005> If you use the add angel button, you can add the breakpoint on line 319 of FullScreen.js
Jan 06 20:29:23 <trex005> I don't mind using the getScrim().hide() workaround for now
Jan 06 20:30:08 <sugardave> I propose you make a kind that is trex005.ScrimmedPopup or something
Jan 06 20:30:26 <hunternet93> I'm new to Enyo, I've defined a Component that defines two Events and attempts to connect them to a WebSocket's onerror and onmessage callbacks in the component's create function (code: https://gist.github.com/hunternet93/7746d1cbafe37e6d3c1c). It seems that this.do<event>() isn't created until after create() is called? What should I do instead?
Jan 06 20:31:25 <trex005> I can do that.  I wanted to make something like that anyway, I just didn't know how to nest the components properly.
Jan 06 20:32:21 <Roy__> trex005: Enyo thinks you have a popup open.
Jan 06 20:32:41 <Roy__> I wonder if there's a hidden popup somewhere.
Jan 06 20:32:48 <GodGinrai> hunternet93: use camelCase.  onError, doError
Jan 06 20:33:31 <trex005> or if somehow I'm triggering whatever method counts the popups twice
Jan 06 20:33:38 <hunternet93> GodGinrai: hmm, I still get the same error, onError isn't a function
Jan 06 20:34:10 <GodGinrai> hunternet93: yea, that was just a suggestion to make your code more readable (and match Enyo style)
Jan 06 20:34:42 <hunternet93> GodGinrai: Oh. well, looks like it's a namespace issue, i'm a javascript noob.
Jan 06 20:34:43 <sugardave> enyo kind events MUST be onUpperCase
Jan 06 20:35:33 <GodGinrai> hunternet93: yea, you needed to bind this, right?
Jan 06 20:35:34 <Roy__> hunternet93: This during the socket is not 'this' the chat connection.
Jan 06 20:35:36 <Roy__> You need to bind
Jan 06 20:35:42 <Roy__> 'this' during...
Jan 06 20:36:00 <GodGinrai> hunternet93: use enyo.bind
Jan 06 20:36:15 <GodGinrai> or now that I think of it, components now have their own bindSafely method, don't they?
Jan 06 20:36:26 <sugardave> yes
Jan 06 20:36:30 <hunternet93> Roy__: I think i got it now, actually. the anonymous function was unneccessary.
Jan 06 20:38:25 <trex005> Roy__: stupid question, when I override a function in my new kind, how do I call the overridden function?
Jan 06 20:38:29 <Roy__> Indeed.
Jan 06 20:38:44 <Roy__> trex005: this.inherited(arguments)
Jan 06 20:38:51 <trex005> thank you.
Jan 06 20:39:45 <Roy__> trex005: Can you temporarily add a 'this.log(this.name, this.showing)' to onyx.Popup's showingChanged?
Jan 06 20:39:59 <Roy__> I'm curious what popup it thinks is showing.
Jan 06 20:40:06 <trex005> yes,
Jan 06 20:40:43 <trex005> done.
Jan 06 20:40:48 <Roy__> I think it's a bug in your code plus a possible bug in in enyo.
Jan 06 20:40:59 <sugardave> say it isn't so!
Jan 06 20:41:29 <trex005> sorry sugardave, I break everything.
Jan 06 20:41:44 <GodGinrai> even the fourth wall
Jan 06 20:42:03 <sugardave> you join a long line of illustrious thing-breakers that have gone before you
Jan 06 20:43:04 <trex005> I had to look up what the fourth wall was.
Jan 06 20:43:28 <Roy__> Hmm, not seeing any other popups. :/
Jan 06 20:43:29 <GodGinrai> XD
Jan 06 20:48:31 <Roy__> trex005: I think I figured it out and it may be a slight mess.
Jan 06 20:48:40 <trex005> Uhhh ohh, my code?
Jan 06 20:48:42 <Roy__> nope
Jan 06 20:49:04 <Roy__> Delete line 50 from onyx/Popup.js (this.showHideScrime)
Jan 06 20:49:47 <trex005> done
Jan 06 20:50:21 <Roy__> And at the start of showHideScrim in onyx, add a 'this.inherited(arguments);'  Not positive yet whether that's a good idea or not. :)
Jan 06 20:50:26 <trex005> well, that seems to be working
Jan 06 20:51:08 <trex005> done
Jan 06 20:52:34 <Roy__> OK, let me file a bug and work on a correct fix for this.
Jan 06 20:52:44 <Roy__> We're going to hold off on 2.5.1.1 until the fix is in, so to speak
Jan 06 20:52:53 <trex005> Okay, if I leave it as is for now, will it blow my app up?
Jan 06 20:54:09 <Roy__> no, should be fine.
Jan 06 20:54:26 <GodGinrai> aside from the unintentional rubber duckies
Jan 06 20:54:29 <GodGinrai> but don't worry
Jan 06 20:54:32 <GodGinrai> they're harmless
Jan 06 20:54:33 <GodGinrai> :P
Jan 06 20:59:39 <Roy__> trex005: I figured out your problem
Jan 06 20:59:54 <Roy__> enyo.version
Jan 06 20:59:55 <Roy__> Object {enyo: "2.5.1.1", layout: "2.4.0-pre.1", onyx: "2.4.0-pre.1"}
Jan 06 21:00:00 <Roy__> You didn't update onyx or layout.
Jan 06 21:01:09 <Roy__> Now, we will pelt you with rocks.
Jan 06 21:01:40 <Semanticfire> :)
Jan 06 21:01:48 <Semanticfire> Roy__, did you get what I mean now ?
Jan 06 21:04:20 <trex005> Roy, okay, I'm an idiot.
Jan 06 21:04:38 <trex005> So, will it work if I update those?  Was that the root of my problem?
Jan 06 21:05:27 <Roy__> It should work just fine.
Jan 06 21:05:43 <Roy__> yes, popup relocated from onyx to enyo
Jan 06 21:06:01 <Roy__> Or scrim did.  Something.
Jan 06 21:07:26 <dmanderson> Whoa I have to declair enyojs scroll back bankruptcy
Jan 06 21:08:04 <trex005> I guess I really should learn how to use git so I can update more easily.
Jan 06 21:08:28 <trex005> Sorry, I know how to use git, I don't know how to make my sub folders other repos
Jan 06 21:15:54 <trex005> I've updated layout and onyx, but now onyx.Popup doesn't seem to be able to locate the scrim
Jan 06 21:24:43 <dmanderson> git submodule add (other repo)
Jan 06 21:27:42 <trex005> Roy__: it seems enyo\source\views\ImageDetails.js line 93 should be changed to Scrim instead of onyx.Scrim, but then the popups don't have the correct styling.
Jan 06 21:28:26 <trex005> Maybe I didn't update something again!
Jan 06 21:46:30 <trex005> If I want to pass a local function as a string to a kind I created, (kinda like you pass ontap) how to I handle that within the kind?
Jan 06 21:48:11 <trex005> So, for example, I want my popup kind to have an onclose:'FunctionName'
Jan 06 21:53:38 <dmanderson> I assume you mean something like {kind:’myKind’, ontap: ‘handleOnTap’}
Jan 06 22:00:34 <dmanderson> example trex005 http://jsfiddle.net/toxigenicpoem/5k84prLe/
Jan 06 22:08:32 <trex005> Thank you
Jan 06 22:23:51 <trex005> dmanderson: this example, to my understanding, if you press cancel, should not alert('you closed'); but it does, can you help me understand, or show me a way that I can return false to prevent an action? http://jsfiddle.net/trex005/pyex33kq/
Jan 06 22:25:20 <dmanderson> here you go! confirm('Ok to close, cancel to not');
Jan 06 22:25:23 <dmanderson> lol oops
Jan 06 22:25:27 <dmanderson> http://jsfiddle.net/toxigenicpoem/pyex33kq/1/
Jan 06 22:25:30 <dmanderson> copy paste failure
Jan 06 22:51:58 <trex005> dmanderson: is there a way to return false from the function I'm passing?
Jan 06 22:52:30 <dmanderson> you can pass it as a param into the event if you want
Jan 06 22:53:25 <trex005> shouldn't doClose() return whatever the function it is calling returns?
Jan 06 22:57:14 <dmanderson> doClose() is propigating an event
Jan 06 22:57:19 <dmanderson> So if you pass doClose(someval)
Jan 06 22:57:28 <dmanderson> that will get passed into the function it will be bound to
Jan 06 22:58:42 <dmanderson> example : http://jsfiddle.net/toxigenicpoem/pyex33kq/2/
Jan 06 22:59:24 <trex005> Okay, I can pass values from my kind to the called function that way, but I'm trying to do it the other way around.  I want myFunc to be able to return false and prevent an action in func
Jan 06 23:00:27 <trex005> kind of like bubbling, if it returns false, the calling function does not take an action.
Jan 06 23:00:29 <dmanderson> if the depth of the components isn’t too deep, you can waterfall an event
Jan 06 23:00:50 <dmanderson> otherwise you would just put a if statement in to either call or not call the method directly on the child.
Jan 06 23:01:34 <dmanderson> Going up in a DOM is more efficent then going down - so there isn’t really a ‘this way is the best way to do it’
Jan 06 23:01:45 <dmanderson> You also have Observers and Bindings you can rely on as well
Jan 06 23:01:59 <trex005> Let me explain what I'm trying to do, and maybe there is a better way.
Jan 06 23:03:00 <trex005> I have a popup class, which has a close button on it.  When the close button is clicked, it calls a function for form validation, and if validation fails, it will prevent the popup class from calling .hide()
Jan 06 23:04:03 <dmanderson> ok following so far :)
Jan 06 23:04:29 <sugardave> this is where setting showing directly is helpful...if the form validation returns true/false you can say popup.set("showing", formValid());
Jan 06 23:04:36 <sugardave> or whatever your validator function is
Jan 06 23:04:55 <trex005> So, I'm passing the form validation function to the generic popup class.  If that form validation returns false, I want to not self close
Jan 06 23:04:56 <sugardave> or !formValid() whichever one you need
Jan 06 23:05:51 <trex005> right, but I need to call !formValid() from within my kind
Jan 06 23:06:20 <trex005> which is passed to it via onClose='formValid'
Jan 06 23:08:00 <sugardave> so, onClose is fired after .close() is already done, so you have a logic issue there
Jan 06 23:08:26 <sugardave> better to overwrite your popup class's close method and check formvalid there
Jan 06 23:08:36 <sugardave> imo
Jan 06 23:09:15 <trex005> Could I use a different name for passing the function?  validate='formValid' ?
Jan 06 23:09:29 <sugardave> and the close override could basically do:  this.set("showing", formValid());
Jan 06 23:09:53 <dmanderson> more like http://jsfiddle.net/toxigenicpoem/pyex33kq/3/ ?
Jan 06 23:12:40 <trex005> is there any way to put the confirm within myFunc, so it is outside of the generic testctrl?
Jan 06 23:13:54 <trex005> that is the key here, the return false needs to be in the validation function which is passed to the Generic popup control
Jan 06 23:18:23 <trex005> I guess I'm not explaining myself very well.
**** BEGIN LOGGING AT Tue Jan 06 23:22:13 2015
Jan 06 23:23:15 <trex005> Is there any other method of passing a function in that I can execute and get a return value from?
Jan 06 23:23:35 <dmanderson> here is how I did it, if you wanted to derive your own onBeforeHide event https://github.com/enyojs/enyo/pull/898/files
Jan 06 23:23:45 <dmanderson> then you could cut off the popup from hiding.
Jan 06 23:24:08 <dmanderson> The problem like sugardave mentioned is that the event is thrown after the popup is hidden.
Jan 06 23:24:18 <dmanderson> There isn’t actually a way to use the current popup in it’s state as is.
Jan 06 23:25:02 <trex005> Well, I actually have a wrapper kind on the popup, does that change anything?  So my wrapper kind has a sub component which I .hide() on.
Jan 06 23:25:40 <dmanderson> you could always pass logic there, and determine if you should .hide or .show
Jan 06 23:25:47 <trex005> So .hide() is never automatically executed, I have to call it myself
Jan 06 23:27:41 <trex005> in your patch, I don't see where you use onBeforeHide, I only see you declaring it.
Jan 06 23:40:03 <dmanderson> it would be an exposed event like onHide
Jan 06 23:47:57 <trex005> okay, here is the complete code of what I am trying to do :
Jan 06 23:47:58 <trex005> http://jsfiddle.net/trex005/pyex33kq/5/
Jan 06 23:48:36 <trex005> function validator needs to be able to return false to block ScrimmedPopup from closing
Jan 06 23:50:33 <trex005> whoops, there were some bugs : http://jsfiddle.net/trex005/pyex33kq/7/
Jan 07 01:05:19 <dmanderson> sorry trex005 was side tracked, looking now
Jan 07 01:09:48 <dmanderson> trex005 http://jsfiddle.net/toxigenicpoem/pyex33kq/8/
Jan 07 01:35:25 <dmanderson> welcome hunternet93
Jan 07 01:38:27 <trex005> dmanderson: You are awesome! :)
Jan 07 01:39:30 <trex005> Definitely a clever way to go about it.
Jan 07 01:40:42 <dmanderson> np, glad to help!
**** ENDING LOGGING AT Wed Jan 07 02:59:58 2015