/
fluid-work IRC Logs-2013-09-24

fluid-work IRC Logs-2013-09-24

Owned by Fluid IRC Log Bot
Last updated: Sept 24, 2013Version comment

[08:29:14 CDT(-0500)] <Justin_o> cindyli: thanks to Bosmon's suggestion of a dynamic grade, the distribute options are now working dynamically for FLUID-5131. The next step will be to combine templates. Bosmon suggested to use the renderer to combine the templates and dump the output to the template resourceText to be used by the actual rendering. I'll continue on to this today.

[08:30:03 CDT(-0500)] <cindyli> nice. thanks for the update, Justin_o

[09:40:55 CDT(-0500)] <colinclark> cindyli, jhung: You guys were awesome

[09:40:56 CDT(-0500)] <colinclark> thanks for such great demos

[09:44:24 CDT(-0500)] <jhung> colinclark: my pleasure.

[10:20:43 CDT(-0500)] <Justin_o> anastasiac: supposed the documentation for angularjs is auto generated from the code. Not sure how though, and I'm guessing that's only for the API. Here's an example http://docs.angularjs.org/api/ng.directive:input

[10:21:14 CDT(-0500)] <anastasiac> thanks for the pointer, Justin_o

[14:43:55 CDT(-0500)] <Bosmon> Hey, Justin_o

[14:44:04 CDT(-0500)] <Bosmon> How is stuff with the combined panels?

[14:46:46 CDT(-0500)] <Justin_o> Bosmon: hello.. it's coming along.. the dynamic grade for setting up the distribute options worked.. and here is the code i have for combining the templates.. i didn't use the jointID though https://github.com/jobara/infusion/blob/FLUID-5131/src/components/uiOptions/js/Panels.js#L182-L215

[14:50:15 CDT(-0500)] <Bosmon> Justin_o - oh yes, that's fine

[14:50:23 CDT(-0500)] <Bosmon> I somehow "double-counted" in my mind yesterday (smile)

[14:50:37 CDT(-0500)] <Bosmon> Clearly once you use the UIVerbatim approach, you don't ALSO need to use the UIJointContainer approach .......

[14:50:45 CDT(-0500)] <Bosmon> Good job

[14:51:02 CDT(-0500)] <Bosmon> cindyli - thanks for your framework reports

[14:51:17 CDT(-0500)] <Bosmon> FLUID-5151 is actually a known issue, but I'll make sure to deal with it in the current round of changeApplier work

[14:51:36 CDT(-0500)] <Bosmon> A workaround for now is to wrap your listeners in a further function before registering them

[14:52:36 CDT(-0500)] <Bosmon> FLUID-5155 seems peculiar - could you add some more detail on what you mean by "doesn't get resolved correctly"?

[14:52:46 CDT(-0500)] <Justin_o> Bosmon: thanks (smile)

[14:53:39 CDT(-0500)] <cindyli> Bosmon: sure, i will add more for 5155. in terms of 5151, what do you mean by "wrap your listeners in a further function before registering them"?

[14:54:17 CDT(-0500)]

<Bosmon> cindyli - if you simply supply a level of wrapping such as function ()

Unknown macro: { listener.apply(null, arguments;}

, then the framework will no longer believe the listener is the same

[14:54:44 CDT(-0500)] <cindyli> ah, i see. thanks

[14:55:31 CDT(-0500)] <Bosmon> Also, could you explain what you were doing when you encountered 5155?

[14:56:17 CDT(-0500)] <cindyli> in video player, we need to pass in different UIO templates for html5 browsers or non-html5, so it has this chunk of code

[14:57:06 CDT(-0500)] <cindyli> https://github.com/cindyli/videoPlayer/blob/VP-297/demos/VideoPlayer.html#L84-L86

[14:57:25 CDT(-0500)] <cindyli> one sec, let me push up the latest code

[15:01:37 CDT(-0500)] <cindyli> i'm creating a grade component that dynamically figures out which UIO template should be applied:

[15:01:37 CDT(-0500)] <cindyli> https://github.com/cindyli/videoPlayer/blob/VP-297/js/MediaSettings.js#L200-L244

[15:02:26 CDT(-0500)] <Bosmon> Intriguing use of IoC-driven console.log, cindyli (smile)

[15:02:47 CDT(-0500)] <cindyli> ya, temporarily for debugging

[15:02:49 CDT(-0500)] <cindyli> Note that the in the first link, templateLoader is passed down via distributeOptions

[15:03:31 CDT(-0500)]

<cindyli> the 2nd link, line 215,

Unknown macro: {that}

.getTemplateLoaderGrade is not resolved in the instantiated UIO

[15:03:45 CDT(-0500)] <Bosmon> Yes, this is an interesting situation

[15:03:53 CDT(-0500)] <Bosmon> And I guess the one which we used to handle with demands blocks

[15:03:59 CDT(-0500)] <Bosmon> Until they decided that they must be abolished

[15:04:01 CDT(-0500)] <cindyli> exactly

[15:04:35 CDT(-0500)] <Bosmon> I guess we still need at least some "little piece of framework" for this, just so people aren't constantly writing little functions with "if" in them

[15:04:58 CDT(-0500)] <Bosmon> For example, I imagine you could achieve this effect using the progressiveEnhancement system

[15:06:18 CDT(-0500)] <cindyli> ah, i certainly need to learn more about progressiveEnhancement system

[15:06:35 CDT(-0500)] <cindyli> how do we do this with progressiveEnhancement system?

[15:06:52 CDT(-0500)] <Bosmon> cindyli - this allows you to specify a set of "environmental checks" together with an "output grade"

[15:07:02 CDT(-0500)] <Bosmon> Which exactly replicates the effect of your "little function with if statements"

[15:07:40 CDT(-0500)] <Bosmon> So what does your dynamic grade return, if not the right thing?

[15:07:42 CDT(-0500)] <Bosmon> fluid.videoPlayer.getTemplateLoaderGrade

[15:07:59 CDT(-0500)] <Bosmon> It looks as if it should be foolproof, given it doesn't seem to rely on its location in the component tree

[15:08:51 CDT(-0500)] <cindyli> no, it only rely on a tag type in the static env

[15:11:18 CDT(-0500)] <cindyli> Bosmon: i still don't know how I can make use of the "output grade" to come up with the dynamic grade for templateLoader, without a if statement and a demands block

[15:11:42 CDT(-0500)] <Bosmon> cindyli - the progressiveEnhancer does it by itself

[15:13:47 CDT(-0500)] <Bosmon> https://github.com/fluid-project/infusion/blob/master/src/framework/enhancement/js/ProgressiveEnhancement.js#L109-L119

[15:14:00 CDT(-0500)] <Bosmon> The array "checks" that you supply to it contain the tests you want made against the static environment

[15:14:44 CDT(-0500)] <Bosmon> But - can you explain what does happen with your function "getTemplateLoaderGrade", compared with what should happen?

[15:17:24 CDT(-0500)] <colinclark> Bosmon: Dude, we're chatting on IRC!

[15:17:37 CDT(-0500)] <Bosmon> Yes, about some code you wrote 3 years ago (smile)

[15:17:42 CDT(-0500)] <colinclark> oh no

[15:17:46 CDT(-0500)] <colinclark> I'm sorry (sad)

[15:17:48 CDT(-0500)] <Bosmon> Why is it that colinclark's contributions always seem the subtlest : P

[15:18:03 CDT(-0500)] <colinclark> aka the most subtly misfactored

[15:18:09 CDT(-0500)] <Bosmon> No, it's fine... just trying to persuade cindyli she should use your progressiveChecker system

[15:19:22 CDT(-0500)] <cindyli> ya, trying to understand how to use your progressiveChecker

[15:19:56 CDT(-0500)] <colinclark> I always forget myself

[15:20:02 CDT(-0500)] <colinclark> it's pretty intense

[15:20:08 CDT(-0500)] <colinclark> but it works

[15:20:25 CDT(-0500)] <Bosmon> It works, and it is mostly what we will have left once demands blocks are gone

[15:20:26 CDT(-0500)]

<cindyli> Bosmon: what happened is that, the gradeNames list in the final instantiated uio.templateLoader has "

Unknown macro: {that}

.getTemplateLoaderGrade" as a grade, rather than the actual returned grade name.

[15:20:48 CDT(-0500)] <cindyli> i will add this into the jira

[15:20:53 CDT(-0500)] <Bosmon> Interestingly it always did seem to do the lion's share of the resolution work even when we had demands blocks

[15:21:37 CDT(-0500)] <Bosmon> cindyli - ok, I think I can believe that (smile)