fluid-work IRC Logs-2011-07-11

[08:19:34 CDT(-0500)] <huslage> mornin
[08:53:42 CDT(-0500)] <Justin_o> huslage: good morning... i think we got some spam sent to our design-commit list
[08:54:03 CDT(-0500)] <huslage> yay
[08:54:06 CDT(-0500)] <huslage> SPAM RULES
[08:54:32 CDT(-0500)] <Justin_o> huslage: i don't like spam either virtual or real (smile)
[09:10:56 CDT(-0500)] <colinclark> Bosmon4: Does this work?
[10:25:58 CDT(-0500)] <anastasiac> fluid-everyone: Toronto IDRC folks are at the jQuery hackathon right now, so we won't be joining stand-up
[15:00:24 CDT(-0500)] <Bosmon4> Hi, cindyli - don't forget to add that JIRA number to the code comment before you commit
[15:01:03 CDT(-0500)] <cindyli> ah, Bosmon4, i forgot to add in my previous commit?
[15:01:47 CDT(-0500)] <Bosmon4> cindyli - I meant the code comment that I added just before that dreadful function I wrote to hack around the framework problem
[15:02:11 CDT(-0500)] <cindyli> ok, icic. sure, will do.
[15:03:46 CDT(-0500)] <cindyli> thanks a lot for finding the workaround and making it work, Bosmon4
[15:04:14 CDT(-0500)] <Bosmon4> colinclark, Justin_o, michelled - we found an awkward issue due to the exact timing of the workflow of "initRendererComponent"
[15:04:32 CDT(-0500)]

<Bosmon4> In the Friday refactoring of UIOptions, we tried to add a direct IoC binding of an event onto

Unknown macro: {component}


[15:04:49 CDT(-0500)] <Bosmon4> But unfortunately the method "is not ready yet" at the time the component options expand
[15:04:51 CDT(-0500)] <colinclark> ok
[15:05:05 CDT(-0500)] <Bosmon4> This is a perfect sample of the kinds of issues that are caused by i) lack of gingerness, ii) use of "methods" (tongue)
[15:05:37 CDT(-0500)] <Justin_o> Bosmon4: what is "gingerness", it always makes me think of cookies
[15:05:43 CDT(-0500)] <Justin_o> the kind you eat, that is
[15:05:47 CDT(-0500)] <colinclark> yawn
[15:05:49 CDT(-0500)] <colinclark> can we ship yet?
[15:06:13 CDT(-0500)] <colinclark> save the campaign against methods for 1.5 and ship?
[15:06:15 CDT(-0500)] <colinclark> (wink)
[15:10:16 CDT(-0500)] <michelled> Justin_o: he means ginger as in careful - like a tiptoeing cat (smile)
[15:10:31 CDT(-0500)] <colinclark> you are a cat!
[15:11:01 CDT(-0500)] <Bosmon4> Justin_o - there's a headline JIRA for this issue now at http://issues.fluidproject.org/browse/FLUID-4330
[15:11:26 CDT(-0500)] <Bosmon4> The issue we found this afternoon is written up at http://issues.fluidproject.org/browse/FLUID-4334
[15:11:36 CDT(-0500)] <Bosmon4> We have added a hack in UIOptions to work around the issue for 1.4
[15:12:07 CDT(-0500)] <colinclark> What's the nature of the hack, Bosmon4?
[15:12:28 CDT(-0500)] <Bosmon4> colinclark - it's pretty nasty - we add a kind of "placeholder function" in a preInitFunction
[15:12:34 CDT(-0500)] <Bosmon4> Named "refreshView"
[15:12:52 CDT(-0500)] <colinclark> oh wow
[15:12:54 CDT(-0500)] <Bosmon4> It is visible just for the tiny sliver of time between preInitFunction and the event binding, where it is picked up
[15:13:00 CDT(-0500)] <colinclark> yes, i'm just looking at the FLUID-4334 issue
[15:13:03 CDT(-0500)] <Bosmon4> And shortly afterwards replaced by the real one
[15:13:11 CDT(-0500)] <Bosmon4> It looks something like this:
[15:13:21 CDT(-0500)] <Bosmon4> fluid.uiOptions.lateRefreshViewBinder = function(that) {
[15:13:28 CDT(-0500)] <Bosmon4> that.refreshView = function() {
[15:13:35 CDT(-0500)] <Bosmon4> that.renderer.refreshView();
[15:13:37 CDT(-0500)] <Bosmon4> }
[15:13:38 CDT(-0500)] <Bosmon4> }
[15:14:02 CDT(-0500)] <colinclark> Only in JavaScript (smile)
[15:14:08 CDT(-0500)] <Bosmon4> I hope you are impressed by what awful code we have come up with in order to make sure we can try to make our ship date (smile)
[15:16:40 CDT(-0500)] <colinclark> I'm very impressed
[15:16:46 CDT(-0500)] <colinclark> nice job, cindyli and Bosmon4
[15:17:31 CDT(-0500)] <colinclark> interesting that anastasiac didn't like my colloquialism in the new Infusion README
[15:17:43 CDT(-0500)] <colinclark> It even received its own commit
[15:17:43 CDT(-0500)] <colinclark> https://github.com/acheetham/infusion/commit/cf215237658d8101bc5a66906d0cdeab957ef173
[15:19:32 CDT(-0500)] <anastasiac> colinclark, feel free to change it back
[15:20:25 CDT(-0500)] <colinclark> anastasiac: You did miss a few changes, though
[15:20:38 CDT(-0500)] <colinclark> I'll need to dig up the pastie
[15:20:55 CDT(-0500)] <colinclark> But there was a comment about "advisors" in the last paragraph, which I removed quite intentionally
[15:21:56 CDT(-0500)] <anastasiac> sorry about that
[15:22:00 CDT(-0500)] <colinclark> no worries
[15:22:05 CDT(-0500)] <colinclark> I'm just reviewing it
[15:22:06 CDT(-0500)] <colinclark> http://pastie.org/2179693
[15:22:16 CDT(-0500)] <colinclark> You can see I've changed that first sentence
[15:22:27 CDT(-0500)] <colinclark> "The Fluid community is an international group of designers and developers who focus on a common mission..."
[15:22:41 CDT(-0500)] <colinclark> I think there's a key distinction I was trying to get at with our community
[15:23:01 CDT(-0500)] <colinclark> that we've gone beyond the sort of accessibility advocacy that people often see in open source as "the accessibility cops"
[15:23:13 CDT(-0500)] <colinclark> that, in fact, Fluid is a community of people who make things
[15:23:30 CDT(-0500)] <colinclark> not just those who advise
[15:24:02 CDT(-0500)] <colinclark> Does anyone else have an opinion about the colloquialism in the commit log I posted above?
[15:24:13 CDT(-0500)] <colinclark> Somehow "interfaces" strikes me as missing something,
[15:24:30 CDT(-0500)] <colinclark> in that Infusion addresses the full depth of an application's architecture
[15:24:49 CDT(-0500)] <anastasiac> colinclark, I'm not too happy with 'interfaces' - 'applications' would be best, but a bit redundant in the sentence
[15:24:55 CDT(-0500)] <colinclark> yes
[15:25:25 CDT(-0500)] <Bosmon4> I'm all for using new words where the existing words are overloaded/boring (tongue)
[15:25:47 CDT(-0500)] <colinclark> you think? (tongue)
[15:25:50 CDT(-0500)] <colinclark> "stuff"
[15:25:52 CDT(-0500)] <colinclark> "code"
[15:26:02 CDT(-0500)] <colinclark> "NotCode(tm)"