fluid-work IRC Logs-2011-05-17

[09:06:40 CDT(-0500)] <colinclark> anastasiac: I'm reading the documentation on this page: http://wiki.fluidproject.org/display/docs/fluid.initRendererComponent
[09:06:48 CDT(-0500)] <colinclark> What does the In/Out column mean?
[09:06:52 CDT(-0500)] <colinclark> Which way is in and which way is out?
[09:12:15 CDT(-0500)] <anastasiac> that's a good question, colinclark, and perhaps Bosmon can confirm my understanding, since he's the one who wrote that: As I understand it, in/out refers to read/write, in a sense: in means it's an input, used but not modified/returned; out means it's something that the function will modify, and that can be examined to see the new values
[09:12:42 CDT(-0500)] <colinclark> ok
[09:13:12 CDT(-0500)] <colinclark> cindyli: I managed to fix the IE bug in the Uploader Image Gallery demo last night
[09:13:29 CDT(-0500)] <cindyli> ya, saw ur comments on jira, colinclark
[09:13:40 CDT(-0500)] <cindyli> what's next then?
[09:13:44 CDT(-0500)] <colinclark> I hope it's okay that I took the time to turn it into a series of components
[09:13:58 CDT(-0500)] <colinclark> I was thinking that I'd like to do one more little refactoring, entirely for curiosity's sake
[09:14:21 CDT(-0500)] <colinclark> If you think about it, the "settings" area of the Image Gallery is probably best as a rendererComponent
[09:14:32 CDT(-0500)] <colinclark> but more interestingly, its model is actually the options for the Uploader
[09:14:47 CDT(-0500)] <colinclark> We've never done anything like this before, and I'm awfully curious to try it out
[09:14:52 CDT(-0500)] <colinclark> So I'll spend a few minutes doing that
[09:14:59 CDT(-0500)] <colinclark> and then the next steps, I think, are:
[09:15:08 CDT(-0500)] <colinclark> 1. You review the code and tell me all the things I should improve
[09:15:18 CDT(-0500)] <Justin_o> anastasiac: is this still needed for 1.4 http://issues.fluidproject.org/browse/FLUID-4145
[09:15:24 CDT(-0500)] <Justin_o> or will it be for some future release
[09:15:27 CDT(-0500)] <colinclark> 2. Justin_o takes a peek and sees if he's satisfied
[09:15:45 CDT(-0500)] <colinclark> 3. If Justin_o is satisfies, he forks it into a project repo and redeploys
[09:15:59 CDT(-0500)] <cindyli> ic
[09:16:02 CDT(-0500)] <colinclark> Oh, and I have to fix the big I introduced by running the install script in the midst of a whole lot of changes
[09:16:11 CDT(-0500)] <colinclark> I'd like your advice, cindyli...
[09:16:23 CDT(-0500)] <cindyli> sure if i have (wink)
[09:16:25 CDT(-0500)] <colinclark> I was thinking that we should probably just turn Uploader.html into a PHP script...
[09:16:42 CDT(-0500)] <colinclark> Loading the [INFUSION_PATH] from a properties file
[09:16:44 CDT(-0500)] <cindyli> why? is it necessary
[09:17:01 CDT(-0500)] <colinclark> Well, the install.sh script sort of ate itself (smile)
[09:17:06 CDT(-0500)] <anastasiac> colinclark, re 4145: it's in progress for 1.4 (e.g. the FSS examples I've put into the wiki already) and will continue well past 1.4. I think we should reset the fix-for to 1.5
[09:17:24 CDT(-0500)] <anastasiac> sorry, that's Justin_o, not colinclark. ^
[09:17:33 CDT(-0500)] <colinclark> The problem is that I accidentally ran it on my development repo while I was in the midst of making a whole lot of changes
[09:17:44 CDT(-0500)] <Justin_o> anastasiac: thanks
[09:17:47 CDT(-0500)] <colinclark> and that caused all the [INFUSION_PATH] tokens to get removed
[09:18:03 CDT(-0500)] <colinclark> It seems to me that modifying code like this is a somewhat awkward solution
[09:18:14 CDT(-0500)] <colinclark> I see exactly why you did it, and it's a sort of complex problem
[09:18:48 CDT(-0500)] <cindyli> um.. ok. i see ur point
[09:18:57 CDT(-0500)] <colinclark> To be clear, it was my own dumb mistake
[09:19:12 CDT(-0500)] <colinclark> but modifying code like this I guess increases the cost of dumb mistakes...
[09:19:16 CDT(-0500)] <colinclark> which I make a lot of (tongue)
[09:19:23 CDT(-0500)] <cindyli> totally agree
[09:19:45 CDT(-0500)] <colinclark> Since we already depend on PHP for the server part of it, I was thinking we might want to use PHP to parameterize Infusion's path into the page's ,head>, as well as using it to drop the path into the static environment where the JS code can access it via IoC
[09:19:49 CDT(-0500)] <colinclark> Does this make sense to you, cindyli?
[09:20:00 CDT(-0500)] <cindyli> i do agree it increases the cost of maintaining working copy of the script and the commit copy
[09:21:00 CDT(-0500)] <colinclark> If Kettle was working, we'd get all of this pretty much for free
[09:21:02 CDT(-0500)] <cindyli> sure. should I make the change
[09:21:07 CDT(-0500)] <colinclark> I can try it
[09:21:16 CDT(-0500)] <colinclark> and then you can fix it up if I make a mistake (smile)
[09:21:46 CDT(-0500)] <colinclark> Sorry for having caused the problem in the first place
[09:22:06 CDT(-0500)] <cindyli> not a problem at all. the current structure is causing the problem
[09:22:30 CDT(-0500)] <colinclark> So, it'll be crazy to have a component use another component's options as its model
[09:22:37 CDT(-0500)] <colinclark> I don't know for sure if it will work
[09:22:42 CDT(-0500)] <cindyli> (smile)
[09:22:58 CDT(-0500)] <colinclark> but if it does, it'll be cool
[09:23:25 CDT(-0500)] <cindyli> absolutely
[09:23:37 CDT(-0500)] <colinclark> I can imagine it might serve as a prototype for a "play around with all the configuration options" version of the demo portal
[09:23:54 CDT(-0500)] <colinclark> jessm: I don't know where that fits into your taxonomy of underwear and top hats
[09:24:01 CDT(-0500)] <colinclark> (smile)
[09:24:31 CDT(-0500)] <colinclark> cindyli has essentially created a UI that lets you tweak some of Uploader's options and try it out
[09:24:41 CDT(-0500)] <jessm> sweet
[09:24:52 CDT(-0500)] <colinclark> right now it's just for QA testing
[09:24:54 CDT(-0500)] <jessm> like a dress-up-doll of uploader (wink)
[09:25:00 CDT(-0500)] <colinclark> (smile)
[09:25:17 CDT(-0500)] <colinclark> I can imagine, in the future, we might have this feature in the demo portal
[09:25:26 CDT(-0500)] <jessm> it sounds perfect for a demo
[09:25:32 CDT(-0500)] <colinclark> or even a kind of "help me configure this component" authoring tool feature
[09:25:57 CDT(-0500)] <colinclark> Going along the lines of our long term trajectory, where we've got things like an FSS builder
[09:26:06 CDT(-0500)] <colinclark> where you choose how many columns, and their particular layout
[09:26:07 CDT(-0500)] <cindyli> sounds promising (smile)
[09:26:22 CDT(-0500)] <colinclark> similarly, we might have a little authoring tool for helping you customize your components
[09:26:24 CDT(-0500)] <colinclark> super cool
[09:48:33 CDT(-0500)] <Justin_o> mlam, harriswong: guess what just got released, but we won't be upgrading to
[09:48:45 CDT(-0500)] <harriswong> ....................................................................
[09:48:50 CDT(-0500)] <mlam> yay.
[09:49:10 CDT(-0500)] <Justin_o> http://blog.jqueryui.com/2011/05/jquery-ui-1-8-13/
[09:49:11 CDT(-0500)] <harriswong> i will take a wild guess, jquery 3.0
[09:49:16 CDT(-0500)] <Justin_o> harriswong: lol
[09:50:09 CDT(-0500)] <heidi> cindyli how's the uio control-splitting going? not sure if that happened or yr busy with other stuff
[09:50:39 CDT(-0500)] <cindyli> heidi: working on it, almost done
[09:50:47 CDT(-0500)] <heidi> sweeet
[09:50:55 CDT(-0500)] <cindyli> the ui options refactoring is done, now fixing the tests
[09:51:03 CDT(-0500)] <harriswong> Justin_o: do we use this? Fixed: $.effects.animateClass
[09:51:18 CDT(-0500)] <heidi> great cindyli, and then fixing UIO controls to work with new layouts after that?
[09:51:24 CDT(-0500)] <cindyli> sure
[09:51:43 CDT(-0500)] <heidi> cool
[09:52:10 CDT(-0500)] <heidi> Justin_o are you UIO'ing? i'm thinking of picking up the 'save automatically when a control changes' task
[09:52:44 CDT(-0500)] <harriswong> Justin_o: Just wondering cause it is broken with jquery 1.6; I just did a quick search on 'animateClass', nothing found.
[09:57:40 CDT(-0500)] <Justin_o> harriswong: hmm.. i don't know for sure, but i don't think so
[09:57:47 CDT(-0500)] <Justin_o> we don't do too much in the way of animation
[09:57:58 CDT(-0500)] <Justin_o> jameswy: are you using this stuff ^ for uio
[09:58:23 CDT(-0500)] <Justin_o> heidi: i have to get all the bug parade stuff in order but will but will be onto UIO after that
[09:58:44 CDT(-0500)] <heidi> Justin_o cool, live preview or something else?
[09:58:46 CDT(-0500)] <harriswong> justin_o: btw, in your step 3,
[09:58:47 CDT(-0500)] <harriswong> "git checkout -b master", how does it know it's from upstream but not origin? curious..
[10:00:17 CDT(-0500)] <Justin_o> harriswong: because of the checkout you did in step 2
[10:00:35 CDT(-0500)] <jameswy> Justin_o: heidi would know better
[10:00:40 CDT(-0500)] <Justin_o> harriswong: you are refering to step 4 right?
[10:00:59 CDT(-0500)] <harriswong> justin_o; ah. right step 4, sorry
[10:01:03 CDT(-0500)] <heidi> what would i know?
[10:01:19 CDT(-0500)] <heidi> oh animateClass - no idea..
[10:01:23 CDT(-0500)] <Justin_o> heidi: if we are using $.effects.animateClass in uio stuff
[10:01:49 CDT(-0500)] <Justin_o> fluid-everyone: have you all had a chance to update your forks and clones based on the e-mail i sent out today
[10:02:02 CDT(-0500)] <michelled> Justin_o: I'm doing it now
[10:02:04 CDT(-0500)] <anastasiac> I was just getting to that right now
[10:02:20 CDT(-0500)] <Justin_o> okay, thanks.. let me know if you have any questions
[10:02:27 CDT(-0500)] <michelled> why do I need to change the default repo on github Justin_o?
[10:02:43 CDT(-0500)] <Justin_o> michelled: if you don't do that you won't be able to delete it
[10:02:54 CDT(-0500)] <Justin_o> github locks the "default" repo
[10:02:57 CDT(-0500)] <michelled> oh ok, thx
[10:03:25 CDT(-0500)] <anastasiac> Justin_o, when I tried step number 2, it put me into 'detached head' state - is that right?
[10:03:36 CDT(-0500)] <Justin_o> anastasiac: yes
[10:03:44 CDT(-0500)] <anastasiac> ok, thanks
[10:03:45 CDT(-0500)] <Justin_o> since that isn't a local branch
[10:08:05 CDT(-0500)] <anastasiac> Justin_o, I've fixed up my master branch. You recommend that we fix any other branches we may have merged from master - how exactly would I do that? the same process in the second half of your email?
[10:09:07 CDT(-0500)] <Justin_o> anastasiac: depends on the issue... for you i noticed that you created a branch off of what was the latest in the master repo
[10:09:29 CDT(-0500)] <Justin_o> probably the easiest way would be to somehow save your changes, created a new branch and apply them to there
[10:10:00 CDT(-0500)] <Justin_o> anastasiac: there are a bunch of ways to do this, you could create a patch and apply it to the new branch.. you could just cherry-pick the single commit
[10:10:08 CDT(-0500)] <Justin_o> or you could cut and paste the changes over
[10:10:38 CDT(-0500)] <anastasiac> Justin_o, I currently have three branches on the go, all of which I've been trying to keep up-to-date with master. I take it I'll need to essentially re-create all of them?
[10:11:06 CDT(-0500)] <Justin_o> anastasiac: have you committed to all of them since you updated them?
[10:11:20 CDT(-0500)] <anastasiac> only one, I believe
[10:11:52 CDT(-0500)] <Justin_o> okay.. so for the ones you haven't, you can reset them to the commit before things went amiss and then remerge with your new master
[10:12:17 CDT(-0500)] <anastasiac> ok
[10:12:22 CDT(-0500)] <Justin_o> anastasiac: that would look like this
[10:12:25 CDT(-0500)] <Justin_o> git reset 275f559d96217e04bdc554dd6b0b0ed222fac016
[10:12:33 CDT(-0500)] <Justin_o> and then do a git merge with your updated masetr
[10:12:42 CDT(-0500)] <anastasiac> got it
[10:12:46 CDT(-0500)] <anastasiac> thanks, Justin_o
[10:12:54 CDT(-0500)] <Justin_o> forgot to mention that you have to do the reset in the branch you want to change
[10:21:10 CDT(-0500)] <Justin_o> colinclark: do you know if FLUID-3709 and FLUID-3708 were finished for Infusion 1.3.1
[10:21:24 CDT(-0500)] <Justin_o> I remeber Bosmon put in a fix that would announce the movements
[10:21:40 CDT(-0500)] <colinclark> lemme check
[10:22:37 CDT(-0500)] <colinclark> For FLUID-3708, definitely yes
[10:22:51 CDT(-0500)] <colinclark> there may be side issues, but at this point those should be separate JIRAs anyway
[10:22:55 CDT(-0500)] <colinclark> FLUID-3709, I can't actually remember
[10:22:59 CDT(-0500)] <colinclark> I'll take a quick peek at the code
[10:24:03 CDT(-0500)] <Justin_o> colinclark: thanks
[10:24:17 CDT(-0500)] <Justin_o> cindyli, colinclark: did this make it into the repo http://issues.fluidproject.org/browse/FLUID-4171 and can it now be closed?
[10:24:32 CDT(-0500)] <cindyli> yes, it can be closed
[10:24:52 CDT(-0500)] <cindyli> Justin_o: we will create another jira for data tag components
[10:25:05 CDT(-0500)] <Justin_o> cindyli: thanks
[10:25:07 CDT(-0500)] <cindyli> np
[10:25:36 CDT(-0500)] <colinclark> I really can't tell with FLUID-3709, Justin_o
[10:25:44 CDT(-0500)] <colinclark> We can ask Bosmon when he is around
[10:26:23 CDT(-0500)] <Justin_o> colinclark: thanks
[10:27:18 CDT(-0500)] <Justin_o> cindyli, colinclark: is FLUID-4207 all merged and ready to be closed
[10:28:05 CDT(-0500)] <cindyli> not yet, the merge only completes part of the work. michelled is working on the rest.
[10:28:23 CDT(-0500)] <colinclark> michelled: Is it the sort of JIRA that is ripe for breaking down into smaller chunks?
[10:28:28 CDT(-0500)] <colinclark> Or is it still best as an umbrella?
[10:28:42 CDT(-0500)] <michelled> colinclark: I think breaking it down makes sense
[10:28:57 CDT(-0500)] <michelled> I'm currently working on a different JIRA then I can turn my attention to breaking that one down
[10:30:25 CDT(-0500)] <colinclark> What's the other JIRA?
[10:30:49 CDT(-0500)] <michelled> 4235 - putting the enhancer into the static environment
[10:43:26 CDT(-0500)] <Justin_o> michelled, anastasiac: what was left for this one http://issues.fluidproject.org/browse/FLUID-4120
[10:43:55 CDT(-0500)] <Justin_o> I wonder if we should just close it and have the rest of the work go for a jira to update the README and release notes for 1.4
[10:46:41 CDT(-0500)] <anastasiac> Justin_o, I agree: close 4120, and start fresh. michelled, what you do think?
[10:47:18 CDT(-0500)] <Justin_o> michelled, yura_: also do you remember why this one was reopenned http://issues.fluidproject.org/browse/FLUID-4039
[10:48:20 CDT(-0500)] <Justin_o> jameswy: is this still going to be a feature of the new ui options design
[10:48:21 CDT(-0500)] <Justin_o> http://issues.fluidproject.org/browse/FLUID-3975
[10:48:29 CDT(-0500)] <Justin_o> basically being able to close it with the esc button
[10:48:43 CDT(-0500)] <jameswy> Justin_o: Yes.
[10:48:55 CDT(-0500)] <yura_> Justin_o: looks like it was reopened to fix the test case, and was not resolved
[10:49:01 CDT(-0500)] <Justin_o> jameswy: thanks..
[10:49:09 CDT(-0500)] <michelled> anastasiac: did you not want to include the minor changes I put in the patch on that bug?
[10:49:17 CDT(-0500)] <Justin_o> yura_: so the test case is all that needs to be fixed.. would you be able to update the jira about that?
[10:50:20 CDT(-0500)] <yura_> Justin_o: i provided the test case and Antranig committed the fix for it
[10:50:39 CDT(-0500)] <anastasiac> oh, shoot - yes, michelled, we should include them. I thought they were in already. sorry
[10:50:41 CDT(-0500)] <Justin_o> yura_: not sure i follow
[10:50:57 CDT(-0500)] <michelled> np
[10:51:40 CDT(-0500)] <yura_> Justin_o: I think he should comment/resolve , or i can mention that the test case passes now
[10:51:45 CDT(-0500)] <yura_> Justin_o: is that what you want?
[10:51:55 CDT(-0500)] <Justin_o> colinclark: can this be closed, or were you planning on converting it to some html5 task http://issues.fluidproject.org/browse/FLUID-2210
[10:52:11 CDT(-0500)] <Justin_o> yura_: okay.. so it can be closed?
[10:52:18 CDT(-0500)] <Justin_o> yes please comment about that and close it off
[10:52:27 CDT(-0500)] <colinclark> Justin_o: Even if we were to convert it to something HTML5, mlam has pretty much done all the work for it. I'll close FLUID-2210 now
[10:52:27 CDT(-0500)] <yura_> ok
[10:52:39 CDT(-0500)] <Justin_o> colinclark: thanks
[10:52:40 CDT(-0500)] <Justin_o> yura_: tanks
[10:52:41 CDT(-0500)] <Justin_o> thanks
[10:53:49 CDT(-0500)] <yura_> Justin_o: just resolved it, or do you wnat me to close it?
[10:57:03 CDT(-0500)] <Justin_o> yura_: you can close it all the way off
[10:57:05 CDT(-0500)] <Justin_o> thanks
[11:04:29 CDT(-0500)] <cindyli> Justin_o: michelled, colinclark, sent a pull request on FLUID-4210, to split up the controls component. Waiting for code review
[11:04:58 CDT(-0500)] <michelled> cool cindyli - I'm in the cspace stand up now but I'll look at it later
[11:05:00 CDT(-0500)] <Justin_o> anastasiac, michelled will those changes be pushed as part of FLUID-4120 or a new jira
[11:05:11 CDT(-0500)] <cindyli> thx, michelled
[11:05:39 CDT(-0500)] <michelled> I don't have a preference Justin_o - I just wanted anastasiac or jessm to put them in
[11:05:59 CDT(-0500)] <anastasiac> why don't I push them for 4120 then close it
[11:06:08 CDT(-0500)] <Justin_o> anastasiac: okay
[11:12:13 CDT(-0500)] <harriswong> Justin_o: i want to get mlam's 3878 branch and push it on top of mine, essentially a overwrite. Should i be running git clone or git pull?
[11:13:39 CDT(-0500)] <Justin_o> harriswong: hmm.. what do you want to replace?
[11:14:33 CDT(-0500)] <harriswong> Justin_o: my github: 3878, which is under pull request
[11:15:05 CDT(-0500)] <harriswong> Justin_o: mlam has updated it on his local instead of my github branch, but the pull request is pointing to mine still
[11:15:35 CDT(-0500)] <harriswong> Justin_o: so i want to get his branch onto mine, and replaces mine with his.
[11:16:25 CDT(-0500)] <harriswong> Justin_o: easier way is i close my pull request and ask mlam to initiate his own 3878 request with his branch
[11:17:24 CDT(-0500)] <Justin_o> harriswong: that might be easier yes (smile)
[11:17:57 CDT(-0500)] <harriswong> Justin_o: so do you think i should just close off my request and simply ask mlam to trigger a new pull request on his 3878
[11:18:10 CDT(-0500)] <Justin_o> harriswong: you could also just merge his changes into your branch
[11:18:33 CDT(-0500)] <Justin_o> to do that the easiest way would be to set up a remote to his github repo and merge his branch into yours
[11:19:18 CDT(-0500)] <Justin_o> harriswong: i don't think it makes a big deal either way, but i guess it would be nice to minimize the number of pull requests.. so just merging his into yours is probably ideal
[11:20:47 CDT(-0500)] <harriswong> Justin_o: ok
[11:26:39 CDT(-0500)] <harriswong> Justin_o: i got infinitely many conflicts, can i delete the branch and just replaces it with his instead of merging
[11:26:58 CDT(-0500)] <Justin_o> harriswong: yes
[11:27:00 CDT(-0500)] <harriswong> thanks
[11:28:02 CDT(-0500)] <colinclark> Hey yura_, have you ever used fluid.fetchResources() with a resourceSpec that has a nodeId instead of an href?
[11:29:05 CDT(-0500)] <yura_> well i've seen the use of something like "templateUrl css-selector"
[11:29:08 CDT(-0500)] <yura_> colinclark: ^
[11:29:11 CDT(-0500)] <yura_> is that what you mean
[11:29:18 CDT(-0500)] <yura_> ?
[11:29:22 CDT(-0500)] <colinclark> no
[11:29:33 CDT(-0500)] <yura_> colinclark: i guess i haven't then (smile)
[11:29:35 CDT(-0500)] <colinclark> A resourceSpec can either have an href or a nodeId property
[11:29:40 CDT(-0500)] <yura_> oh
[11:29:51 CDT(-0500)] <colinclark> The documentation suggests that it might be a useful thing
[11:29:51 CDT(-0500)] <yura_> no, never tried that, what does it do , colinclark?
[11:29:57 CDT(-0500)] <colinclark> but so far I can't quite figure it all out
[11:29:58 CDT(-0500)] <colinclark> http://wiki.fluidproject.org/display/docs/fluid.fetchResources
[11:31:19 CDT(-0500)] <anastasiac> colinclark, my understanding is that if you provide a nodeId instead of an href to a file, fetchResources will treat the node as the source of the template, and return that
[11:31:28 CDT(-0500)] <anastasiac> I've never tried it, though
[11:31:31 CDT(-0500)] <colinclark> That's exactly what I want
[11:31:52 CDT(-0500)] <anastasiac> well, do let us know if it works (wink)
[11:31:55 CDT(-0500)] <yura_> really (smile) ? what is the use case?
[11:31:59 CDT(-0500)] <yura_> colinclark: ?
[11:32:05 CDT(-0500)] <colinclark> The documentation suggests that the node's innerHTML is used as the template, anastasiac
[11:32:14 CDT(-0500)] <colinclark> but that is in fact not how the implementation actually works
[11:32:30 CDT(-0500)] <colinclark> and as a result, I can't quite figure out how I'd actually use it
[11:32:33 CDT(-0500)] <colinclark> yura_: My use case is this...
[11:32:48 CDT(-0500)] <colinclark> I want a plain old autoInitted rendererComponent
[11:32:50 CDT(-0500)] <anastasiac> colinclark, what does the implementation actually do?
[11:32:55 CDT(-0500)] <colinclark> My tree is entirely declarative
[11:33:10 CDT(-0500)] <colinclark> I just don't really want to put my markup into a separate HTML fragment file
[11:33:15 CDT(-0500)] <colinclark> It's fine where it is, sitting in the DOM
[11:33:24 CDT(-0500)] <colinclark> against the raw Renderer API, this is trivial
[11:34:37 CDT(-0500)] <colinclark> anastasiac: https://github.com/fluid-project/infusion/blob/master/src/webapp/framework/core/js/FluidRequests.js#L266-272
[11:34:41 CDT(-0500)] <yura_> colinclark: but why then do you need to deal with the template at all? container (which is a part of the DOM) would be enough for the renderer component
[11:35:00 CDT(-0500)] <colinclark> yura_: Can you elaborate?
[11:36:17 CDT(-0500)] <yura_> i mean all renderer component would need is a declarative tree and container. template/resource is sort of options only if you need from some where else and if it's not in the dom. that's my understanding, at least
[11:36:33 CDT(-0500)] <colinclark> ah, okay
[11:36:41 CDT(-0500)] <colinclark> that's pretty much even more exactly like what I want (smile)
[11:36:50 CDT(-0500)] <colinclark> yura_: let me see what happens
[11:37:15 CDT(-0500)] <Justin_o> here is the current bug parade: http://issues.fluidproject.org/secure/ConfigureReport.jspa?atl_token=hLyns-VDmh&amp;filterid=10128&amp;mapper=components&amp;selectedProjectId=10001&amp;reportKey=com.atlassian.jira.plugin.system.reports%3Asinglelevelgroupby&amp;Next=Next
[11:37:16 CDT(-0500)] <yura_> so fluid.defaults("mycomponent", {gradeNames: ["fluid.rendererComponent", "autoInit", protoTree: {}]}); is all you need + the finalInitFunction, i guess, that renders.
[11:37:30 CDT(-0500)] <Justin_o> anastasiac: could you please look over the bug parade for the documentation issues
[11:37:35 CDT(-0500)] <colinclark> yeah, that's great, yura_
[11:37:37 CDT(-0500)] <colinclark> exactly what I want
[11:37:38 CDT(-0500)] <yura_> colinclark: which reminds me, it would be nice to have an autoRender grade (smile)
[11:37:45 CDT(-0500)] <colinclark> yes, something
[11:37:49 CDT(-0500)] <anastasiac> Justin_o, will do
[11:37:57 CDT(-0500)] <Justin_o> colinclark, mlam: could you please look over the bug parade for the uploader related issues
[11:38:00 CDT(-0500)] <Justin_o> anastasiac: thanks
[11:38:04 CDT(-0500)] <colinclark> i'm sure we all have a one line init function like this, yura_:
[11:38:13 CDT(-0500)]

<colinclark> demo.imageGallery.settings.init = function (that)

Unknown macro: { that.refreshView(); }

[11:38:22 CDT(-0500)] <yura_> haha yes
[11:38:24 CDT(-0500)] <Justin_o> michelled, cindyli, heidi : could you please take a look at the various issues related to ui options and other related components
[11:38:29 CDT(-0500)] <colinclark> Justin_o: Yep, I'll try to take a look shortly
[11:38:34 CDT(-0500)] <Justin_o> colinclark: thanks
[11:39:07 CDT(-0500)] <Justin_o> fluid-everyone: right now the bug parade is pretty big, please let me know if anything is missing or if there are parts that you know we won't be getting to http://issues.fluidproject.org/secure/ConfigureReport.jspa?atl_token=hLyns-VDmh&amp;filterid=10128&amp;mapper=components&amp;selectedProjectId=10001&amp;reportKey=com.atlassian.jira.plugin.system.reports%3Asinglelevelgroupby&amp;Next=Next
[11:39:17 CDT(-0500)] <heidi> Justin_o not sure what you mean. look at them for?
[11:39:33 CDT(-0500)] <heidi> ah, i see
[11:39:39 CDT(-0500)] <Justin_o> heidi: please make sure that all the ones that we need to cover are there
[11:39:55 CDT(-0500)] <Justin_o> heidi: thanks
[11:39:59 CDT(-0500)] <heidi> Justin_o as long as what's listed in our UIO roadmap is here, i think it's good
[11:40:05 CDT(-0500)] <heidi> did you use the roadmap ?
[11:40:15 CDT(-0500)] <Justin_o> i haven't cross referenced it
[11:40:34 CDT(-0500)] <colinclark> heidi and cindyli's roadmap is wicked
[11:40:52 CDT(-0500)] <heidi> Justin_o it's our master to-do list with all the jiras
[11:44:37 CDT(-0500)] <colinclark> yura_: you rule, thanks for your help
[11:44:42 CDT(-0500)] <colinclark> seems to work quite nicely
[11:44:48 CDT(-0500)] <yura_> colinclark: (smile) glad
[11:46:20 CDT(-0500)] <Justin_o> heidi: okay i updated but parade based on the roadmap page
[11:46:27 CDT(-0500)] <Justin_o> except i didn't include the integration stuff
[11:46:33 CDT(-0500)] <cindyli> Justin_o: 4224 can be closed
[11:46:33 CDT(-0500)] <Justin_o> that should probably happen post 1.4 anyways
[11:46:47 CDT(-0500)] <heidi> Justin_o excellent thanks. and yeah agreed re: integration
[11:47:09 CDT(-0500)] <cindyli> the issue of "In IE 6 the UI Options unit tests don't even run. " is to be addressed in 4234
[11:47:49 CDT(-0500)] <Justin_o> cindyli: and the tests are passing everywhere else?
[11:48:07 CDT(-0500)] <cindyli> Justin_o: no, that's 4234
[11:48:16 CDT(-0500)] <colinclark> yura_: I have one more question for you
[11:48:22 CDT(-0500)] <yura_> colinclark: sure
[11:48:28 CDT(-0500)] <colinclark> or maybe it's two (smile)
[11:48:59 CDT(-0500)] <colinclark> So, I have two components
[11:49:05 CDT(-0500)] <colinclark> One of which is a UI for editing some stuff
[11:49:15 CDT(-0500)] <Justin_o> cindyli: thanks for closing it
[11:49:18 CDT(-0500)] <colinclark> so it's a ["fluid.rendererComponent", "fluid.modelComponent"]
[11:49:28 CDT(-0500)] <cindyli> np
[11:49:36 CDT(-0500)] <colinclark> So it has a ChangeApplier
[11:49:59 CDT(-0500)] <Bosmon> A renderer component is already a model component...
[11:50:08 CDT(-0500)] <yura_> colinclark: i think fluid.modelComponent is redundant
[11:50:11 CDT(-0500)] <colinclark> It didn't seem to be from my reading of the code, Bosmon
[11:50:19 CDT(-0500)] <colinclark> Justin_o: Speak of the devil ^
[11:50:35 CDT(-0500)] <colinclark> How would I know these arcane things you know, Bosmon and yura_?
[11:50:44 CDT(-0500)] <colinclark> I often just look at the implementation in absence of documentation
[11:50:45 CDT(-0500)] <colinclark> (smile)
[11:50:49 CDT(-0500)] <Bosmon> colinclark - You could know it by reading the grade definitions (smile)
[11:50:57 CDT(-0500)] <colinclark> Well, that's what I did
[11:51:03 CDT(-0500)] <colinclark> let me see if I can track down the source of my confusion
[11:51:47 CDT(-0500)] <Bosmon> And by applying the Holy Hand Grenade....
[11:51:49 CDT(-0500)] <colinclark> Ah, of course I missed the fact that all View Components are Model Components.
[11:51:58 CDT(-0500)] <colinclark> great
[11:52:02 CDT(-0500)] <Bosmon> Countest though not to Two, excepting that thou then procedeth to Three!
[11:52:06 CDT(-0500)] <Bosmon> FOUR, is right out!
[11:52:10 CDT(-0500)] <colinclark> after this little smackdown, I will remove one line of declaration from my component
[11:52:21 CDT(-0500)] <colinclark> If only I could remove its only line of code, instead!
[11:52:21 CDT(-0500)] <yura_> (smile)
[11:52:30 CDT(-0500)] <Bosmon> The one that calls "append"? (tongue)
[11:52:47 CDT(-0500)] <colinclark> That one would be nice, too
[11:52:51 CDT(-0500)] <colinclark> The one that calls refreshView() (tongue)
[11:53:12 CDT(-0500)] <colinclark> At least give me a renderOnInit option or something (tongue)
[11:53:24 CDT(-0500)] <colinclark> maybe there is one?
[11:54:19 CDT(-0500)] <Bosmon> There isn't.... there should be
[11:54:23 CDT(-0500)] <Bosmon> Am I allowed to add one? (tongue)
[11:54:28 CDT(-0500)] <Bosmon> Did Justin_o fix the graph this morning?
[11:54:32 CDT(-0500)] <colinclark> He did
[11:54:37 CDT(-0500)] <Justin_o> Bosmon: hello, yes i did
[11:54:44 CDT(-0500)] <Bosmon> Justin_o - amazing
[11:54:45 CDT(-0500)] <colinclark> so beware that all your branches are probably completely obsolete (wink)
[11:54:58 CDT(-0500)] <Bosmon> That's fine, I was never terribly attached to them (tongue)
[11:55:02 CDT(-0500)] <Justin_o> Bosmon: did you happen to see my e-mail to the fluid-work list about it, there some work that will need to be done to get your branches fixed up
[11:55:04 CDT(-0500)] <colinclark> lol
[11:55:08 CDT(-0500)] <Justin_o> (smile)
[11:56:33 CDT(-0500)] <colinclark> Bosmon or yura_: What advice can you offer me?
[11:56:55 CDT(-0500)] <yura_> colinclark: i would probalby pass modelChanged event as component's own event in this case
[11:57:04 CDT(-0500)] <colinclark> hmm
[11:57:09 CDT(-0500)] <Bosmon> It is part of the work that we have had to push out to 1.5 as FLUID-4195/FLUID-3674
[11:57:11 CDT(-0500)] <colinclark> Can you elaborate a bit?
[11:57:21 CDT(-0500)] <colinclark> Bosmon: okay, that's cool
[11:57:27 CDT(-0500)] <colinclark> I'm all for pushing off work at this point
[11:57:29 CDT(-0500)] <yura_> so is this is a parent - child component relationship (smile) ?
[11:57:35 CDT(-0500)] <colinclark> just looking for an interesting workaround
[11:57:43 CDT(-0500)] <colinclark> yura_: As a matter of fact, it is (smile)
[11:58:00 CDT(-0500)] <yura_> colinclark: and you want a listener within the subcomponent or parent ?
[11:58:05 CDT(-0500)] <Bosmon> colinclark - what is it you realised was hilarious?
[11:58:09 CDT(-0500)] <colinclark> The listener is on the parent
[11:58:12 CDT(-0500)] <colinclark> Bosmon: The spokesperson effect
[11:58:15 CDT(-0500)] <yura_> oh
[11:58:16 CDT(-0500)] <colinclark> the event is on the child
[11:58:18 CDT(-0500)] <Justin_o> Bosmon: please feel free to update the framework/ioc and related jiras on the bug parade, if you are going to remove one, please change the fix for version and remove the "Bug Parade Infusion" comment.. if you are going to add one, do the reverse
[11:58:25 CDT(-0500)] <Bosmon> Justin_o - thank you
[11:58:33 CDT(-0500)] <Bosmon> When does Bug Parade end?
[11:58:43 CDT(-0500)] <Justin_o> Bosmon: next friday
[11:58:51 CDT(-0500)] <Bosmon> colinclark - I'm glad to hear that your thing is a component now - rather than merely a sleazy invoker (tongue)
[11:59:00 CDT(-0500)] <Justin_o> Bosmon: which is the 27th of May
[11:59:00 CDT(-0500)] <colinclark> sleazy (tongue)
[11:59:09 CDT(-0500)] <colinclark> How could you say that about my invoker?
[11:59:27 CDT(-0500)] <colinclark> Bosmon: I'm actually talking about another thing altogether now
[11:59:33 CDT(-0500)] <colinclark> but yes, my "sleazy" invoker is now a component
[11:59:34 CDT(-0500)] <Bosmon> I'm sure it is a very good invoker, in its way (tongue)
[11:59:47 CDT(-0500)] <colinclark> Your are AN INVOKER!
[11:59:52 CDT(-0500)] <colinclark> oh damn typing
[11:59:59 CDT(-0500)] <colinclark> spelling mistakes always ruin jokes
[12:00:05 CDT(-0500)] <Bosmon> (smile)
[12:00:05 CDT(-0500)] <yura_> haha
[12:00:13 CDT(-0500)] <Bosmon> Actually I didn't notice the mistake, until you drew attention to it
[12:00:20 CDT(-0500)] <Bosmon> Sometimes one KNOWS what someone is trying to say (tongue)
[12:00:24 CDT(-0500)] <colinclark> To elaborate, a portion of the parent's options is actually the model for the child
[12:00:29 CDT(-0500)] <Bosmon> Especially, when they say "RRRRAAAOURW"
[12:00:33 CDT(-0500)] <Bosmon> However they spell it
[12:00:45 CDT(-0500)] <colinclark> When the child changes the model, I want the parent to zap its subcomponent (whose options have changed)
[12:01:00 CDT(-0500)] <colinclark> Bosmon: Invokers use a strange sort of grammar
[12:01:05 CDT(-0500)] <colinclark> I think it would be something like this:
[12:01:19 CDT(-0500)]

<colinclark> "

Unknown macro: {arguments}

.2.RRRRAAAOURW"


[12:01:35 CDT(-0500)] <jessm> hey CSS peeps, check this out: http://www.layerstyles.org/builder.html
[12:02:23 CDT(-0500)] <yura_> colinclark: and you want to do it all declaratively ?
[12:02:31 CDT(-0500)] <colinclark> It's be cooler if it was called Laser Styles, jessm
[12:02:40 CDT(-0500)] <colinclark> and didn't have a fake popup window in it
[12:02:43 CDT(-0500)] <colinclark> yura_: but of course!
[12:02:45 CDT(-0500)] <jessm> yes, indeed
[12:02:46 CDT(-0500)] <colinclark> (smile)
[12:03:09 CDT(-0500)] <jessm> the fake popup is lame-o
[12:03:29 CDT(-0500)] <colinclark> but it does give us some ideas of what an Infusion Authoring Tool might be like
[12:04:02 CDT(-0500)] <jessm> no kidding, eh? colinclark
[12:04:07 CDT(-0500)] <Bosmon> I guess it is a similar illustration of the event-driven modality
[12:04:19 CDT(-0500)] <Bosmon> To "When I mention my maiden aunt from Winnipeg, you will jump into the canal"
[12:04:34 CDT(-0500)] <colinclark> lol
[12:04:47 CDT(-0500)] <yura_> colinclark: (smile) i dont know of any way of declaratively providing a listener to changeApplier. But assuming you can have a preInitFunction that will binds your subcomponent's event to applier's changeEvent then you have a hook to provide a listener to that event from the parent component
[12:04:56 CDT(-0500)] <colinclark> eek
[12:05:02 CDT(-0500)] <colinclark> I guess I should just jump into a canal
[12:05:16 CDT(-0500)] <colinclark> yura_: I think I can handle that
[12:05:18 CDT(-0500)] <colinclark> thanks for the advice
[12:05:43 CDT(-0500)] <Bosmon> ChangeAppliers will be fully awesome, some day
[12:05:47 CDT(-0500)] <Bosmon> But they have received no love in 1.4
[12:05:53 CDT(-0500)] <colinclark> Bosmon: A question for you, while you're here and you're on the subject
[12:05:59 CDT(-0500)] <colinclark> we're on the subject
[12:06:31 CDT(-0500)] <colinclark> What's the best way for me to transform my model to suit the requirements of presentation?
[12:06:38 CDT(-0500)] <Bosmon> ha!
[12:06:42 CDT(-0500)] <colinclark> This example is probably the opposite of a real world scenario
[12:06:47 CDT(-0500)] <colinclark> Since it's a pretty geeky UI right now
[12:06:48 CDT(-0500)] <Bosmon> Speaking as the author of the ModelTransformation system?
[12:06:53 CDT(-0500)] <colinclark> lol
[12:06:57 CDT(-0500)] <colinclark> Well, I had hoped you might suggest that
[12:07:03 CDT(-0500)] <colinclark> let me outline my use case, just in case
[12:07:05 CDT(-0500)] <colinclark> case
[12:07:15 CDT(-0500)] <colinclark> So, in one case I have a piece of the model that is an array of strings
[12:07:27 CDT(-0500)] <Bosmon> We even have a special event you can use to do it on
[12:07:33 CDT(-0500)] <Bosmon> although it is currently a little lame
[12:07:34 CDT(-0500)] <colinclark> and because I genuinely don't like my users, I want them to edit that array as a string
[12:07:36 CDT(-0500)] <colinclark> oh, what is it?
[12:07:43 CDT(-0500)] <Bosmon> It only exists on full rendererComponents
[12:07:50 CDT(-0500)] <colinclark> I have a full rendererComponent
[12:07:51 CDT(-0500)] <colinclark> hurray!
[12:08:06 CDT(-0500)] <Bosmon> that.events.prepareModelForRender.fire(that.model, that.applier, that);
[12:08:26 CDT(-0500)] <Justin_o> anastasiac: did you push the change for FLUID-4120 yet? i'm about to kick off bug parade so curious?
[12:08:52 CDT(-0500)] <anastasiac> Justin_o, I'm sorry, no, I haven't yet - still working on cleaning my branches up
[12:08:57 CDT(-0500)] <anastasiac> I'll do it right now
[12:09:03 CDT(-0500)] <cindyli> heidi: let's implement the ur new controls. u have the templates ready?
[12:09:31 CDT(-0500)] <Justin_o> anastasiac: okay.. if you prefer we can just move it to the new jira i have for the release?
[12:09:37 CDT(-0500)] <colinclark> Bosmon: oh, yes, I saw that event when I was debugging through the framework when my code wasn't working
[12:09:41 CDT(-0500)] <colinclark> cool
[12:09:47 CDT(-0500)] <anastasiac> no, Justin_o, I'll do it right now
[12:09:54 CDT(-0500)] <Justin_o> anastasiac: okay..
[12:10:20 CDT(-0500)] <colinclark> Bosmon: And then the best time to transform it back into its original format?
[12:10:29 CDT(-0500)] <Justin_o> anastasiac: if you are using the patch from that jira, please make sure to update the version of jquery ui to 1.8.12
[12:10:30 CDT(-0500)] <colinclark> I guess just in the change listener?
[12:10:45 CDT(-0500)] <anastasiac> Justin_o, will do
[12:10:57 CDT(-0500)] <Bosmon> colinclark - well, possibly... I recall you are also the great proponent of idempotent transforms (tongue)
[12:11:04 CDT(-0500)] <colinclark> lol
[12:11:05 CDT(-0500)] <heidi> cindyli yep best would be to take from jameswy's full page without preview template , let me get the link
[12:11:07 CDT(-0500)] <colinclark> tongues, tongues
[12:11:18 CDT(-0500)] <Bosmon> But yes, the transforming back part is currently not really well thought of
[12:11:27 CDT(-0500)] <Bosmon> This is why the system is a little lame
[12:11:29 CDT(-0500)] <Justin_o> anastasiac: thanks
[12:11:55 CDT(-0500)] <Justin_o> heidi: have you got all your branches back to working order... i think i need to branch off of one of yours for the uio work
[12:12:02 CDT(-0500)] <colinclark> Bosmon and yura_: I really appreciate your help with this
[12:13:55 CDT(-0500)] <heidi> Justin_o i haven't merged the new master with FLUID-3782. is that all i need to do?
[12:14:52 CDT(-0500)] <Justin_o> heidi: when was the last time you updated FLUID-3782 from master?
[12:14:55 CDT(-0500)] <mlam> Justin_o: http://issues.fluidproject.org/browse/FLUID-4241 can be closed now that we have removed the buttons from the demo
[12:15:01 CDT(-0500)] <heidi> Justin_o prob fri
[12:15:18 CDT(-0500)] <Justin_o> okay so it's the same as what's on your github account?
[12:15:24 CDT(-0500)] <Justin_o> heidi: ^
[12:15:30 CDT(-0500)] <heidi> Justin_o ya
[12:15:40 CDT(-0500)] <heidi> er, wait
[12:15:48 CDT(-0500)] <heidi> my master was updated with yr instructions this morning
[12:15:53 CDT(-0500)] <heidi> but i haven't touched that branch
[12:16:00 CDT(-0500)] <Justin_o> heidi: okay, that's fine..
[12:16:13 CDT(-0500)] <Justin_o> i think we were just going to branch off of your branch anyways.. so i think you can just leave it
[12:16:16 CDT(-0500)] <heidi> Justin_o so do i merge the new master with it and push up?
[12:16:20 CDT(-0500)] <heidi> yeah
[12:16:52 CDT(-0500)] <Justin_o> heidi: i think you don't need to do anything.. we can all just update the new branches we create off of it with the correct master, and we should be okay
[12:17:01 CDT(-0500)] <heidi> Justin_o okay that sounds good
[12:18:30 CDT(-0500)] <heidi> so cindyli the branch is at https://github.com/heidiv/infusion/tree/FLUID-3782 and jameswy's layout to use is in components/uiOptions/html/FullNoPreviewUIOptions.html
[12:18:40 CDT(-0500)] <heidi> there's a demo of it in /demos
[12:18:54 CDT(-0500)] <cindyli> ok. thx, heidi
[12:19:07 CDT(-0500)] <heidi> so fetch my github, checkout to heidi/FLUID-3782 and create your branch - right Justin_o?
[12:19:44 CDT(-0500)] <Justin_o> heidi, cindyli : that's right
[12:20:01 CDT(-0500)] <Justin_o> heidi: do you have an idea of which of the current jiras were started as part of FLUID-3782
[12:20:30 CDT(-0500)] <heidi> Justin_o the 3 layouts, the sliding panel
[12:21:07 CDT(-0500)] <Justin_o> heidi: would you be able to comment on all of the relevant jiras about it
[12:21:19 CDT(-0500)] <heidi> k
[12:21:20 CDT(-0500)] <Justin_o> and put up a link to your FLUID-3782 branch
[12:21:24 CDT(-0500)] <heidi> will do
[12:21:24 CDT(-0500)] <Justin_o> heidi: thanks
[12:25:03 CDT(-0500)] <Justin_o> anastasiac: thanks for pushing FLUID-4120
[12:25:09 CDT(-0500)] <anastasiac> np
[12:31:13 CDT(-0500)] <colinclark> Bosmon: one more question?
[12:37:57 CDT(-0500)] <Justin_o> fluid-everyone: we're officially in bug parade, i just sent out an e-mail to the fluid-work list about it
[12:38:28 CDT(-0500)] <colinclark> great
[12:38:34 CDT(-0500)] <Justin_o> fluid-everyone: please make sure that if you have any updates or changes to what issues are in bug parade that you get those in today and let me know about them, thanks
[12:38:37 CDT(-0500)] <colinclark> Justin_o: I still haven't looked at the Uploader issues
[12:38:42 CDT(-0500)] <colinclark> did you just guess at them until I do so?
[12:39:17 CDT(-0500)] <Justin_o> colinclark: yes.. i tried to err on the side of too many
[12:39:21 CDT(-0500)] <colinclark> ok
[12:39:25 CDT(-0500)] <colinclark> sorry about that
[12:39:28 CDT(-0500)] <colinclark> I'll be on it shortly
[12:39:28 CDT(-0500)] <Justin_o> so i'm hoping you can just knock a bunch off
[12:39:32 CDT(-0500)] <Justin_o> colinclark: no problem
[12:40:09 CDT(-0500)] <mlam> Justin_o: I have FLUID-4240 fixed.
[12:41:08 CDT(-0500)] <Justin_o> mlam: great.. please fire off a pull request for that one
[12:42:44 CDT(-0500)] <mlam> ok
[12:43:23 CDT(-0500)] <colinclark> Justin_o or heidi: Is there now a standard class for hiding stuff with FSS?
[12:43:51 CDT(-0500)] <Justin_o> Bosmon: can FLUID-3709 be closed?
[12:44:25 CDT(-0500)] <heidi> colinclark there is - not sure Justin_o, it's in trunk now right? it'd be in fss-layout, at the bottom colinclark
[12:45:03 CDT(-0500)] <Justin_o> colinclark, heidi : yes it's there https://github.com/fluid-project/infusion/blob/master/src/webapp/framework/fss/css/fss-layout.css#L390-405
[12:45:18 CDT(-0500)] <colinclark> thanks!
[12:45:22 CDT(-0500)] <colinclark> this is so much fun
[12:45:26 CDT(-0500)] <colinclark> writing code
[12:45:32 CDT(-0500)] <colinclark> learning about new stuff in the framework and in FSS
[12:45:39 CDT(-0500)] <heidi> (smile)
[12:45:51 CDT(-0500)] <colinclark> ah, we added visibility: hidden to fl-hidden
[12:46:06 CDT(-0500)] <colinclark> For the Window-Eyes screen reader, in part?
[12:47:35 CDT(-0500)] <heidi> colinclark i hope i commented the options clearly. depends if you want hidden from both visual and screen readers or both or...
[12:47:43 CDT(-0500)] <colinclark> right
[12:47:57 CDT(-0500)] <colinclark> your comments are really good
[13:02:49 CDT(-0500)] <mlam> hey Justin_o, you mind if i just take FLUID-4132? It should be a pretty quick fix
[13:03:41 CDT(-0500)] <Justin_o> mlam: i think cindyli already has a pull request for that
[13:03:53 CDT(-0500)] <mlam> oh ok, cool!
[13:06:51 CDT(-0500)] <mlam> Justin_o: Does this one still apply to you? http://issues.fluidproject.org/browse/FLUID-2052 I'll try on ur win7 machine, but seems to work fine in XP/ FF3.6
[13:08:38 CDT(-0500)] <Justin_o> mlam: haven't tried it in a while.. probably won't have a chance to try it before i leave today
[13:08:49 CDT(-0500)] <Justin_o> but it's possible that it's working now
[13:08:56 CDT(-0500)] <mlam> ok, is your win7 machine on?
[13:21:41 CDT(-0500)] <Justin_o> mlam: the one across from harriswong , that one should be one yes
[13:21:52 CDT(-0500)] <mlam> thanks Justin_o
[13:22:11 CDT(-0500)] <Justin_o> mlam: np
[13:32:18 CDT(-0500)] <harriswong> Justin_o: should #1426 be added to the list?
[13:33:01 CDT(-0500)] <colinclark> cindyli and Justin_o: I've pushed up a version of the new Image Gallery that seems reasonably stable
[13:33:02 CDT(-0500)] <colinclark> https://github.com/colinbdclark/image-gallery
[13:33:16 CDT(-0500)] <colinclark> I still need to handle variations in Infusion's paths, but I think it shouldn't take me too long to do so
[13:33:34 CDT(-0500)] <colinclark> I'm going to head off to the dentist for a bit, but if you guys have a chance to take a look and try it out, that would be great
[13:33:41 CDT(-0500)] <colinclark> I'm sorry it's not as compact as it used to be
[13:33:49 CDT(-0500)] <colinclark> there's actually a lot less "code" there, at least (smile)
[13:33:58 CDT(-0500)] <Justin_o> colinclark: i'm heading off to the doctor now, actually, maybe tomorrow?
[13:34:03 CDT(-0500)] <cindyli> great. thx, colinclark. will try
[13:34:08 CDT(-0500)] <colinclark> Justin_o: Sure, no problem
[13:34:10 CDT(-0500)] <colinclark> thanks cindyli
[13:34:14 CDT(-0500)] <cindyli> np
[13:34:25 CDT(-0500)] <colinclark> Please feel free to give me any code review advice you think of
[13:34:36 CDT(-0500)] <colinclark> Bosmon and yura_, I'd appreciate one of you looking at it, too, and sharing some comments
[13:34:46 CDT(-0500)] <colinclark> that would be cool, like a bowtie
[13:45:07 CDT(-0500)] <yura_> Bosmon: if the component has createOnEvent , would anything happen on the second time the event is fired?
[14:37:44 CDT(-0500)] <heidi> hey anastasiac is this page http://wiki.fluidproject.org/display/fluid/Events+for+Component+Developers still up to date? does IOC affect in any way?
[14:38:57 CDT(-0500)] <anastasiac> heidi, that page hasn't been updated in a while; there's lots of new stuff. I'll have a look now to see whether or not there's anything there that's actually no longer correct, but at a quick glance it looks basically sound - just incomplete
[14:39:08 CDT(-0500)] <anastasiac> Events are on my list of docs to update
[14:39:30 CDT(-0500)] <heidi> gotcha. okay cool. but this should give me an okay overview of how to add events, fire events
[14:39:58 CDT(-0500)] <anastasiac> yes, certainly a good start. As I said, I'll have a look now, and let you know if there's anything there you should ignore (wink)
[14:42:00 CDT(-0500)] <heidi> thanks!
[14:48:18 CDT(-0500)] <anastasiac> heidi, for some more advanced info on events, see http://wiki.fluidproject.org/display/docs/Event+injection+and+boiling
[14:48:46 CDT(-0500)] <heidi> will check it out - thanks anastasiac
[14:49:07 CDT(-0500)] <anastasiac> heidi, also: listeners can have priorities now. I don't think that's documented anywhere - I'll check, and try to write something up
[14:49:24 CDT(-0500)] <heidi> that's prob beyond me right now, but good to know
[15:10:24 CDT(-0500)] <cindyli> anastasiac: i'd like to seek ur opinion on a naming issue, michelled and I all agree that u r very good at naming :
[15:10:26 CDT(-0500)] <cindyli> (smile)
[15:10:45 CDT(-0500)] <cindyli> it's about michelled's code review comment @ https://github.com/cindyli/infusion/commit/601e0cb5f3c3c71767636ff109a2527cfed804df#commitcomment-387514
[15:11:26 CDT(-0500)] <cindyli> anastasiac: if u search for "wrapping finalInitControlsComponent",
[15:11:42 CDT(-0500)] <anastasiac> found it...
[15:11:46 CDT(-0500)] <cindyli> u will find the comment that's in debate
[15:11:47 CDT(-0500)] <cindyli> great
[15:12:20 CDT(-0500)] <anastasiac> what's the debate, exactly?
[15:12:21 CDT(-0500)] <cindyli> so, as u've seen that finalInitControlsComponent () is used by 3 subcompoents of fluid.uiOptions for finalInit
[15:12:57 CDT(-0500)] <anastasiac> ok, see that
[15:13:01 CDT(-0500)] <cindyli> one suggestion that michelled has is to wrap it as fluid.uiOptions.finalInitControlsComponent and use it in default blocks
[15:13:40 CDT(-0500)] <michelled> not wrap it so much as rename it
[15:13:44 CDT(-0500)] <anastasiac> right
[15:13:51 CDT(-0500)] <michelled> then we could get rid of the three wrappers it currently has
[15:14:25 CDT(-0500)] <anastasiac> that makes sense to me, so far
[15:14:25 CDT(-0500)] <cindyli> however, i'm a bit worry that since this func is for initing subcomponents of fluid.uiOptions, naming it that way may give an impression that this func is for fluid.uiOptions. but it isn't indeed
[15:14:54 CDT(-0500)] <anastasiac> the name "fluid.uiOptions.finalInitControlsComponent
[15:15:08 CDT(-0500)] <anastasiac> hm
[15:15:11 CDT(-0500)] <anastasiac> let me check
[15:16:00 CDT(-0500)] <anastasiac> I see your point... it does say "finalInitControlsComponent", which does distinguish it from "finalInit", but it is a directly member of uiOptions... hm...
[15:16:24 CDT(-0500)] <anastasiac> what do you think of naming it "fluid.uiOptions.controlsFinalInit", cindyli and michelled?
[15:16:42 CDT(-0500)] <michelled> I think that's fine
[15:16:48 CDT(-0500)] <anastasiac> does it seem more clear?
[15:17:04 CDT(-0500)] <cindyli> good. making change
[15:17:12 CDT(-0500)] <michelled> yes, more clear and shorter
[15:18:07 CDT(-0500)] <cindyli> thx, anastasiac, michelled
[15:18:12 CDT(-0500)] <anastasiac> np
[15:24:59 CDT(-0500)] <cindyli> michelled: i've pushed in the changes on 4210
[15:25:42 CDT(-0500)] <michelled> thanks - did you reissue the pull request?
[15:25:54 CDT(-0500)] <cindyli> michelled: do i need?
[15:26:09 CDT(-0500)] <cindyli> let me check if it's made into the old pull request
[15:26:14 CDT(-0500)] <michelled> cindyli: I'm actually not sure
[15:26:40 CDT(-0500)] <cindyli> i think it will as i've done for 4171. double checking
[15:27:21 CDT(-0500)] <cindyli> yes, it made in, michelled
[15:28:42 CDT(-0500)] <michelled> ok, thx cindyli
[15:28:44 CDT(-0500)] <heidi> cindyli michelled ... newbie q: does that.events.afterTransferComplete.addListener(transferCompleteHandlerFunc); mean afterTransferComplete gets fired when transferCompleteHandlerFunc finishes?
[15:29:29 CDT(-0500)] <cindyli> i think the func gets fired when transferCompleteHandlerFunc is triggered
[15:29:31 CDT(-0500)] <michelled> heidi: that line is adding a listener to the afterTransferComplete event
[15:29:41 CDT(-0500)] <cindyli> sorry, when afterTransferComplete is triggered
[15:30:14 CDT(-0500)] <cindyli> which means when this line is run: that.events.afterTransferComplete.fire();
[15:30:53 CDT(-0500)] <heidi> confused... what's the significance of sending it transferCompleteHandlerFunc
[15:31:14 CDT(-0500)] <heidi> so when that function is called somewhere
[15:31:27 CDT(-0500)] <heidi> afterTransferComplete does something.... hmph.
[15:32:07 CDT(-0500)] <michelled> when the afterTransferComplete event is fired the listener (transferCompleteHandlerFunc) gets run
[15:32:11 CDT(-0500)] <michelled> does that make sense?
[15:32:17 CDT(-0500)] <heidi> ah, yes that makes sense
[15:32:17 CDT(-0500)] <cindyli> no, heidi, i think it's the other way around. u don't call that function explicitly
[15:32:37 CDT(-0500)] <heidi> thanks you two
[15:32:41 CDT(-0500)] <michelled> np
[15:32:45 CDT(-0500)] <cindyli> np
[15:33:21 CDT(-0500)] <heidi> listener = function ready to do something when event is fired
[15:35:14 CDT(-0500)] <heidi> and events are fired by literally calling .fire() on it
[18:37:20 CDT(-0500)] <DrewWills> can anyone point me in the right direction with the pager/renderer: need to render a table based on JSON from the server (working), but then change the data based on user bahavior and have those changes reflected in the table...
[18:37:57 CDT(-0500)] <DrewWills> i can change the table myself manually... and I can change the JSON data structure... but when I page away & back the old info appears on the table