fluid-work IRC Logs-2013-04-19
[08:02:17 CDT(-0500)] <arashs> ]
[09:57:02 CDT(-0500)] <michelled> hey cindyli, how's it going?
[09:57:19 CDT(-0500)] <cindyli> michelled: how's what going?
[09:57:51 CDT(-0500)] <cindyli> i sent a pull request this morning to rename "enactors" in the base grade, wonder if you can review it when you have a chance?
[09:58:03 CDT(-0500)] <michelled> cindyli: sure
[09:58:10 CDT(-0500)] <michelled> I was curious about the other pull request
[09:58:18 CDT(-0500)] <cindyli> currently working on the feedbacks from Bosmon's review on 4927
[09:58:22 CDT(-0500)] <michelled> I know Bosmon has been making a lot of review recommendations
[09:58:30 CDT(-0500)] <cindyli> ya, working on it
[09:58:35 CDT(-0500)] <michelled> how's it going?
[09:58:45 CDT(-0500)] <cindyli> pretty good
[09:58:55 CDT(-0500)] <cindyli> no stoppers so far
[09:59:00 CDT(-0500)] <cindyli>
[09:59:28 CDT(-0500)] <michelled> cool
[11:06:01 CDT(-0500)] <michelled> cindyli: now that we've got rid of 'enactors' I wonder if we should actually rename the javascript file itself
[11:06:05 CDT(-0500)] <michelled> what do you think?
[11:06:59 CDT(-0500)] <cindyli> michelled: i thought about that but since that js file is a collection of several enactors. might make sense to use enactors as the file name?
[11:07:36 CDT(-0500)] <michelled> it doesn't seem to fit that pattern we have in the repo. for example, we have Store.js that has both the cookie store and the temp store in it
[11:07:55 CDT(-0500)] <cindyli> ok, i see your point. i will rename the file
[11:23:32 CDT(-0500)] <lastlegion> hello all, I want to work on a music dev environment as a gsoc project, can someone point me to the appropriate mentor, Cheers!
[11:26:20 CDT(-0500)] <Justin_o> lastlegion: colinclark is the mentor for that one, but he isn't in the channel at the momemnt
[11:26:44 CDT(-0500)] <lastlegion> thanks man
[11:27:04 CDT(-0500)] <Justin_o> arashs: if you want you can follow look at my svg test page as I go https://dl.dropboxusercontent.com/u/548929/fontIconTests/svtTest.html not much there at the moment though, but it will update with each of my saves
[11:27:11 CDT(-0500)] <Justin_o> lastlegion: np
[11:27:46 CDT(-0500)] <arashs> thanks Justin_o
[12:02:42 CDT(-0500)] <arashs> Justin_o: are you gonna try the coloured SVGs?
[12:26:29 CDT(-0500)] <Justin_o> arashs: yep.. i still have to put those on and see how hard it is to manipulate the styles
[12:27:00 CDT(-0500)] <arashs> cool,
[12:45:08 CDT(-0500)] <Justin_o> arashs: realized there was a typo in the file name, here's the new link to the svg test page https://dl.dropboxusercontent.com/u/548929/fontIconTests/svgTest.html
[12:45:56 CDT(-0500)] <arashs> nice,
[12:46:10 CDT(-0500)] <arashs> the stacking looks good Justin_o
[12:46:34 CDT(-0500)] <Justin_o> arashs: thanks
[12:46:58 CDT(-0500)] <arashs> I wonder if we could change opacity of the top layer...
[12:55:37 CDT(-0500)] <Justin_o> arashs: if you wanted to do that after the svg was created, we could probably do it with the markup version through css
[12:57:35 CDT(-0500)] <arashs> I think it would be great if we could, as long as it is not a lot of work for you
[13:09:35 CDT(-0500)] <Justin_o> arashs: done
[13:11:24 CDT(-0500)] <arashs> great Justin_o let me know if you would want to try it with other icons
[13:11:40 CDT(-0500)] <Justin_o> arashs: will do.. going to try some other things first
[13:11:49 CDT(-0500)] <arashs> cool
[14:04:31 CDT(-0500)] <Bosmon> Justin_o - are you there?
[14:05:19 CDT(-0500)] <Justin_o> Bosmon: hello
[14:07:17 CDT(-0500)] <Bosmon> I was thinking of starting to implement our dedicated framework functionality for FLUID-4878
[14:07:22 CDT(-0500)] <Bosmon> What do you think of the following syntax: http://pastie.org/7665641
[14:07:25 CDT(-0500)] <Bosmon> Also cindyli
[14:09:13 CDT(-0500)] <Justin_o> Bosmon: looking at it now
[14:09:51 CDT(-0500)] <cindyli> Bosmon: what is "this"? always a dom object, could it be other stuff?
[14:10:38 CDT(-0500)] <Bosmon> It can be any "this"
[14:11:12 CDT(-0500)] <Justin_o> Bosmon: would it be possible and would it make sense, to have "method" be "funcName" so that it looked more like regular invokers?
[14:12:03 CDT(-0500)] <Bosmon> Justin_o - I think it would undesirable
[14:12:08 CDT(-0500)] <Bosmon> Since the expected values are completely different
[14:12:21 CDT(-0500)] <Bosmon> "method" must always be a single string representing a direct method on the object
[14:12:36 CDT(-0500)] <Justin_o> Bosmon: ah i see now
[14:12:38 CDT(-0500)] <Bosmon> Whereas "funcName" can be a general EL path representing a method's global name
[14:13:07 CDT(-0500)] <Bosmon> So for example I have this in my current Uploader branch, which doesn't work:
[14:13:10 CDT(-0500)] <Bosmon> enableBrowseButton: {
[14:13:10 CDT(-0500)] <Bosmon> func: " .swfUpload.setButtonDisabled",
[14:13:10 CDT(-0500)] <Bosmon> args: false
[14:13:10 CDT(-0500)] <Bosmon> },
[14:13:20 CDT(-0500)] <Bosmon> As it turns out, swfUpload also expects to be a "this"
[14:13:26 CDT(-0500)] <Bosmon> And we would rewrite this as follows:
[14:13:35 CDT(-0500)] <Bosmon> enableBrowseButton: {
[14:13:36 CDT(-0500)] <Bosmon> "this": " .swfUpload".setButtonDisabled",
[14:13:36 CDT(-0500)] <Bosmon> args: false
[14:13:36 CDT(-0500)] <Bosmon> },
[14:13:38 CDT(-0500)] <Bosmon> damn
[14:13:50 CDT(-0500)] <Bosmon> If only I could delete that from the record
[14:14:24 CDT(-0500)] <Bosmon> With Skype you can actually call back your last message now....
[14:14:27 CDT(-0500)] <Bosmon> Anyway, I meant to write this:
[14:14:30 CDT(-0500)] <Bosmon> enableBrowseButton: {
[14:14:30 CDT(-0500)] <Bosmon> "this": " .swfUpload"
[14:14:30 CDT(-0500)] <Bosmon> method: "setButtonDisabled",
[14:14:30 CDT(-0500)] <Bosmon> args: false
[14:14:30 CDT(-0500)] <Bosmon> },
[14:15:19 CDT(-0500)] <cindyli> michelled: the pull request for renaming enactors - https://github.com/fluid-project/infusion/pull/316 - is ready for more reviews
[14:15:32 CDT(-0500)] <michelled> thx cindyli
[14:15:35 CDT(-0500)] <cindyli> np
[14:15:41 CDT(-0500)] <Justin_o> Bosmon: just to double check, does "this" always refer to the parent of the method?
[14:15:58 CDT(-0500)] <Bosmon> Justin_o - yes
[14:16:17 CDT(-0500)] <Bosmon> It's no different to the arguments for your "fluid.bind", only differently packaged
[14:19:19 CDT(-0500)] <Justin_o> Bosmon: cool
[14:19:24 CDT(-0500)] <Justin_o> Bosmon: could we do something like this http://pastie.org/7665848
[14:20:01 CDT(-0500)] <Justin_o> oops.. lost some of my comment in there.. how do i edit that?
[14:20:29 CDT(-0500)] <Justin_o> oh it just scrolled off screen
[14:20:48 CDT(-0500)] <Bosmon> Justin_o - isn't that just the first example in my pastie?
[14:22:16 CDT(-0500)] <Justin_o> Bosmon: hmm maybe i'm missing something .. i was suggesting to remove the "this" property and source it instead from the elpath to the method that you'd want to call
[14:22:31 CDT(-0500)] <Bosmon> Oh, right
[14:22:58 CDT(-0500)] <Bosmon> yes, that would be more difficult to implement
[14:23:03 CDT(-0500)] <Bosmon> Don't you think it might also be more unclear?
[14:23:35 CDT(-0500)] <Justin_o> Bosmon: maybe just exploring ideas… i guess it is nice that the "this" is explicity
[14:23:40 CDT(-0500)] <Justin_o> "explicit"
[14:26:59 CDT(-0500)] <cindyli> Bosmon: hope I'm not distracting you from "this" conversation. I have a question on demands block.
[14:27:07 CDT(-0500)] <Bosmon> Hi cindyli, go ahead!
[14:27:31 CDT(-0500)] <Justin_o> Bosmon: i think that's probably all i'd have to really comment… your first suggestion is pretty clear and explicit and i think the syntax would be usable. And of course, I'd love to have this type of declarative way of calling things in the framework
[14:27:36 CDT(-0500)] <cindyli> working the last piece of your suggestions to extract the common options share by all default settings panels to a demands block
[14:27:42 CDT(-0500)] <cindyli> http://pastie.org/7663807
[14:28:27 CDT(-0500)] <Bosmon> cindyli - ah, that's funny, isn't it
[14:28:30 CDT(-0500)] <cindyli> as you would see in the code, if the demands is issued against a particular panel like "textSizer", it would just work fine
[14:28:32 CDT(-0500)] <Bosmon> Yes, currently we have no such function
[14:28:49 CDT(-0500)] <Bosmon> Please add a JIRA requesting it and I will think about it
[14:28:56 CDT(-0500)] <Bosmon> In the meantime you will have to put it in the GRADE...
[14:29:15 CDT(-0500)] <cindyli> it doesn't make sense to put in the grade
[14:29:23 CDT(-0500)] <Bosmon> You can put it in a DIFFERENT grade
[14:29:26 CDT(-0500)] <cindyli> if you meant the base grade settingsPanel
[14:29:38 CDT(-0500)] <cindyli> ah i see
[14:29:40 CDT(-0500)] <cindyli> trying
[14:29:48 CDT(-0500)] <Bosmon> You can make a new grade, perhaps "defaultSettingsPanel"
[14:30:10 CDT(-0500)] <Bosmon> Although people will have to look hard to distinguish it from "defaultSettingsPanels"
[14:30:16 CDT(-0500)] <cindyli> haha, right
[14:30:33 CDT(-0500)] <cindyli> so similar
[14:30:40 CDT(-0500)] <system64> Hello colinclark!
[14:30:46 CDT(-0500)] <Bosmon> I am sorry your CATTs ran away!
[14:31:02 CDT(-0500)] <cindyli> i feel sorry for myself too
[14:33:21 CDT(-0500)] <Justin_o> Bosmon, colinclark, arashs: are you guys happy with the svg testing http://wiki.fluidproject.org/display/fluid/Research+the+viability+of+using+icon+fonts+in+UI+Options
[14:33:42 CDT(-0500)] <Justin_o> fluid-everyoen: ^
[14:35:41 CDT(-0500)] <arashs> nice Justin_o , I'm guessing we can do the opacity effect with the coloured logos too
[14:35:46 CDT(-0500)] <arashs> right?
[14:36:06 CDT(-0500)] <Bosmon> Justin_o - that looks like a really valuable set of testing, and a great resources to have it on the page there
[14:36:06 CDT(-0500)] <arashs> in markup
[14:38:52 CDT(-0500)] <Bosmon> Justin_o - do .ttf files work fine on Android?
[14:42:05 CDT(-0500)] <arashs> Justin_o: the complex SVG margup doesn't fully appear on my iPhone
[14:43:40 CDT(-0500)] <cindyli> Bosmon:
[14:43:47 CDT(-0500)] <cindyli> https://github.com/fluid-project/infusion/pull/308 is ready for more reviews
[14:46:46 CDT(-0500)] <Justin_o> Bosmon: don't have an android to test on personally, and the test page for that has fall backs.. so not exactly sure
[14:47:01 CDT(-0500)] <Justin_o> arashs: i think it's a safari issues as it it doesn't even show up properly on my desktop
[14:47:57 CDT(-0500)] <arashs> good point Justin_o
[14:49:02 CDT(-0500)] <arashs> the chrome app has the same issue
[14:50:22 CDT(-0500)] <Justin_o> arashs: right, must be a webkit thing then