fluid-work IRC Logs-2013-05-31

[05:03:44 CDT(-0500)] * Topic is 'This channel is logged – for details see: http://wiki.fluidproject.org/display/fluid/IRC+Channel' set by jessm on 07:30:00 CST(-0600)

[10:46:43 CDT(-0500)] <michelled> cindyli: is now a good time to meet?

[10:47:14 CDT(-0500)] <cindyli> yes, michelled

[10:47:24 CDT(-0500)] <michelled> anastasiac: are you available too?

[10:47:32 CDT(-0500)] <anastasiac> sure

[10:48:22 CDT(-0500)] <michelled> skype?

[10:48:37 CDT(-0500)] <cindyli> fine

[13:47:47 CDT(-0500)] <Bosmon> hi there cindyli

[13:47:49 CDT(-0500)] <Bosmon> How are you getting on?

[13:48:14 CDT(-0500)] <cindyli> hi, Bosmon, still struggling

[13:48:44 CDT(-0500)] <Bosmon> Tell me about your problems

[13:49:34 CDT(-0500)] <cindyli> thanks, one sec to dig out the code on github

[13:50:42 CDT(-0500)] <cindyli> my goal is to replace this line with IoCSS: https://github.com/cindyli/infusion/blob/FLUID-5010/src/webapp/components/uiOptions/js/UIOptions.js#L86

[13:51:38 CDT(-0500)] <cindyli> The use of this mapping is @ https://github.com/cindyli/infusion/blob/FLUID-5010/src/webapp/components/uiOptions/js/FatPanelUIOptions.js#L108-131

[13:52:18 CDT(-0500)] <cindyli> my IoCSS rewritting:

[13:52:28 CDT(-0500)] <cindyli> distributeOptions: [{

[13:52:28 CDT(-0500)]

[13:52:28 CDT(-0500)] <cindyli> removeSource: true,

[13:52:28 CDT(-0500)]

[13:52:28 CDT(-0500)] <cindyli> }],

[13:52:57 CDT(-0500)] <Bosmon> Seems reasonable

[13:53:22 CDT(-0500)] <cindyli> 2. moved the uiOptions source tree in the fat panel out of "derivedDefaults"

[13:53:43 CDT(-0500)] <Bosmon> btw I think you are unnecessarily using the ">" combiner, since there's no real need for the components to be direct descendents

[13:53:47 CDT(-0500)] <Bosmon> You should just use whitespace

[13:54:04 CDT(-0500)] <cindyli> well, whitespace does not work

[13:54:11 CDT(-0500)] <Bosmon> !!

[13:54:17 CDT(-0500)] <Bosmon> If it doesn't, that is a bug : P

[13:54:21 CDT(-0500)] <cindyli> and i found ">" is not necessarily the direct descendents

[13:54:30 CDT(-0500)] <Bosmon> urgh

[13:54:33 CDT(-0500)] <cindyli> i can use it to address a deeper sub component

[13:54:36 CDT(-0500)] <Bosmon> The implementation must somehow be inverted

[13:56:05 CDT(-0500)] <cindyli> ok, i can create one or two test cases for this

[13:56:15 CDT(-0500)] <Bosmon> That would be marvellous

[13:56:23 CDT(-0500)] <Bosmon> Are you running into particular problems with 1. and 2.?

[13:56:31 CDT(-0500)] <Bosmon> I imagine 1. will fail until you deal with 2.

[13:57:13 CDT(-0500)] <cindyli> 1 you mean my IoCSS rewritting for uiOptions?

[13:57:25 CDT(-0500)] <cindyli> or whitespace

[13:58:38 CDT(-0500)] <cindyli> assuming we are back to the topic of rewritting uiOptions mapping

[13:58:48 CDT(-0500)] <Bosmon> back to the topic, yes

[13:58:56 CDT(-0500)] <Bosmon> I think we can deal with the whitespace as a separate bug

[13:59:04 CDT(-0500)] <cindyli> cool

[13:59:37 CDT(-0500)] <cindyli> the problem I'm running into is at instantiating fluid.uiOptions, it complaints a missing container

[13:59:47 CDT(-0500)] <cindyli> which means, this line is not resolved: https://github.com/cindyli/infusion/blob/FLUID-5010/src/webapp/components/uiOptions/js/FatPanelUIOptions.js#L111

[14:00:26 CDT(-0500)] <Bosmon> It may be just because it is still part of the ridiculous "derivedDefaults" system

[14:00:55 CDT(-0500)] <cindyli> no, at that point, i've moved everything outside of "derivedDefaults"

[14:01:08 CDT(-0500)]

[14:01:15 CDT(-0500)] <Bosmon> Yes

[14:01:45 CDT(-0500)] <cindyli> my first thought was that this assignment is inside a finalInit which is not recognizable on IoC tree

[14:02:12 CDT(-0500)] <Bosmon> Well, it depends WHERE you have moved it outside "derivedDefaults"

[14:02:20 CDT(-0500)] <cindyli> as per our talk yesterday, i rewrote the whole finalInit into a listener of onCreate

[14:02:29 CDT(-0500)] <Bosmon> If it is not attached to a component which has overall "createOnEvent" it's true it will not be seen

[14:02:43 CDT(-0500)] <Bosmon> This configuration must not be evaluated until the event "templateAndIframeReady" has fired

[14:02:50 CDT(-0500)] <cindyli> yes

[14:02:53 CDT(-0500)] <Bosmon> In the past this was ensured by ridiculous manual code

[14:03:08 CDT(-0500)] <Bosmon> Whereas now it should be ensured "naturally" by having it attached to a component with the correct createOnEvent

[14:03:37 CDT(-0500)] <Bosmon> So when you say, "I've moved everything outside of "derivedDefaults"", can we look at how you have done this?

[14:03:41 CDT(-0500)] <Bosmon> Did you check that in yet?

[14:03:59 CDT(-0500)] <cindyli> not yet, can check in now

[14:04:02 CDT(-0500)] <Bosmon> cool

[14:07:43 CDT(-0500)] <cindyli> it's in, Bosmon - https://github.com/cindyli/infusion/tree/FLUID-5010

[14:10:20 CDT(-0500)] <Bosmon> cindy: so I see that the UIOptions subcomponent has lost its "createOnEvent" annotation

[14:10:37 CDT(-0500)] <Bosmon> Which will explain why its container won't resolve

[14:12:44 CDT(-0500)] <cindyli> Bosmon: it's not lost, it was moved into options block: https://github.com/cindyli/infusion/blob/FLUID-5010/src/webapp/components/uiOptions/js/FatPanelUIOptions.js#L112

[14:12:57 CDT(-0500)] <Bosmon> cindyli - that's the wrong place!

[14:13:04 CDT(-0500)] <Bosmon> It can't appear inside the component options, it will be ignored

[14:13:07 CDT(-0500)] <cindyli> man, i wrote a test case for that

[14:13:32 CDT(-0500)] <cindyli> createOnEvent can be passed down only if it's inside options block