fluid-work IRC Logs-2013-10-10

[08:09:52 CDT(-0500)] <jhung> Justin_o: FYI, I submitted 2 new pull requests.

[08:14:21 CDT(-0500)] <Justin_o> jhung: thanks.. i'll take a look

[09:11:48 CDT(-0500)] <Justin_o> jhung: are there any dependencies between the pull requests you've filed for the prefsEditor

[09:16:54 CDT(-0500)] <jhung> Justin_o: I guess GPII-195 and GPII-205 should go first then GPII-193.

[09:17:19 CDT(-0500)] <jhung> Justin_o: GPII-193 may be conflicted after those two go in, so let me know if that needs fixing.

[09:17:36 CDT(-0500)] <Justin_o> jhung: okay.. i'm testing out your GPII-193 now and the # is showing up in the url still

[09:17:45 CDT(-0500)] <Justin_o> i'll switch to the others and come back to GPII-193

[09:25:33 CDT(-0500)] <Justin_o> jhung: merged in the GPII-195 pull request.. will look at GPII-205 now

[09:25:46 CDT(-0500)] <jhung> great thanks Justin_o

[09:25:53 CDT(-0500)] <jhung> I'm looking at GPII-193 now.

[09:26:11 CDT(-0500)] <Justin_o> jhung: sorry.. GPII-208 pull request is the one i'll look at, not 205..

[09:32:46 CDT(-0500)] <Justin_o> jhung: GPII-208 has been merged

[09:34:32 CDT(-0500)] <jhung> great thanks Justin_o

[09:35:25 CDT(-0500)] <Justin_o> michelled: we have a task to share out the nightly builds from the prefsEditors repo.. They are now in place at http://build.gpii.net/prefsEditors. Did you want to share that out?

[09:36:02 CDT(-0500)] <michelled> Justin_o: the call you are on is probably the right place to do that

[09:36:18 CDT(-0500)] <michelled> and also to encourage everyone to make pull requests so their work can be built nightly

[09:36:31 CDT(-0500)] <michelled> if there's time, can you bring it up?

[09:36:51 CDT(-0500)] <Justin_o> michelled: cool not sure i'll get a chance to talk, but i'll post something in the chat.

[12:03:04 CDT(-0500)] <avtar> Bosmon: ping

[12:03:16 CDT(-0500)] <Bosmon> Hi there AVVTTAR

[12:03:45 CDT(-0500)] <avtar> are we supposed to be on this call using skype?

[12:03:50 CDT(-0500)] <avtar> or something else?

[12:04:10 CDT(-0500)] <avtar> are we still having a call?

[12:04:19 CDT(-0500)] <avtar> i have questions

[12:07:02 CDT(-0500)] <Justin_o> Bosmon: looks like you commented on the same part of the GPII-153 pull request as I did but with a different solution.. your's is simpler though

[12:07:35 CDT(-0500)] <Bosmon> What are we going to do about this jquery combobox plugin he has dumped there?

[12:07:39 CDT(-0500)] <Justin_o> Bosmon: also made those changes you recommended in the FLUID-5131 branch will be talking to cindyli about the aux schema soon.. would you like to be involved in the conversation?

[12:07:53 CDT(-0500)] <Bosmon> Justin_o - in a bit, yes

[12:08:00 CDT(-0500)] <Bosmon> On a call with AVVTTAR and others for an hour

[12:08:49 CDT(-0500)] <Justin_o> Bosmon: good question.. i did recommend him to rewrite that as an Infusion Plugin.. i suppose you can request that again Seeing as you or Yura will be merging it in, i guess you can let him know if this will prevent it from being merged in

[12:08:57 CDT(-0500)] <Justin_o> Bosmon: sure, let us know when you are ready.

[12:09:14 CDT(-0500)] <Bosmon> Well, he could either rewrite it, or just wrap it

[12:09:21 CDT(-0500)] <Bosmon> Or else submit it as a genuine jQuery plugin

[12:09:47 CDT(-0500)] <Justin_o> what does the latter mean

[12:11:44 CDT(-0500)] <Justin_o> Bosmon: ^

[12:17:21 CDT(-0500)] <Justin_o> Bosmon: i'm going to sit with cindyli to chat about the aux schema stuff, can you ping her when you are ready to chat with us about it.

[12:17:24 CDT(-0500)] <Bosmon> Justin_o - that is, he should submit it to jQuery! And manage it properly

[12:17:34 CDT(-0500)] <Justin_o> Bosmon: ah i see

[12:17:39 CDT(-0500)] <Bosmon> I'm puzzled about his "lookupMsg" activity

[12:17:45 CDT(-0500)] <Bosmon> And this business about looked.template

[12:17:54 CDT(-0500)] <Bosmon> Do we have messages that look up to things with a member named template?

[12:18:18 CDT(-0500)] <Justin_o> Bosmon: the lookupMsg follows what was done in the Exploration Tool.. i think i expored just using the resolver directly, but can't seem to find those changes anywhere

[12:18:36 CDT(-0500)] <Bosmon> It's a bit of a mess

[12:18:37 CDT(-0500)] <Justin_o> i'm not sure what the looked.template is

[12:18:42 CDT(-0500)] <Bosmon> We should be able to resolve these messages simply as values

[12:18:50 CDT(-0500)] <Bosmon> Without an indirect function

[12:19:02 CDT(-0500)] <Justin_o> Bosmon: really.. how is that?

[12:19:10 CDT(-0500)] <Justin_o> we don't have to go through the resolver

[12:19:26 CDT(-0500)]

[12:19:45 CDT(-0500)] <Justin_o> Bosmon: for now there are no string templates in there, but i guess there could be

[12:19:49 CDT(-0500)]

[12:20:36 CDT(-0500)] <Justin_o> maybe you should look through how it's done in the Exploration Tool, so we can fix it there first

[12:21:31 CDT(-0500)] <Justin_o> but it seems reasonable in strings to just assign the value out of the messsage bundle directly, since it shouldn't be resolved at that point.

[12:22:23 CDT(-0500)] <Justin_o> cindyli: you might want to see this ^ we should probably fix up any cases in infusion and the exploration tool

[12:24:11 CDT(-0500)] <cindyli> makes sense. thanks, guys

[12:31:05 CDT(-0500)]

[12:31:43 CDT(-0500)]

[12:31:58 CDT(-0500)] <cindyli> how can we accommodate both?

[12:32:22 CDT(-0500)] <Bosmon> We could construct a fresh message bundle out of expressions like the latter

[12:34:29 CDT(-0500)] <cindyli> Bosmon: do you mean to have 2 message bundles, one is the message resolver, the other contains string mappings

[12:59:37 CDT(-0500)] <cindyli> Bosmon: this is the sketch of the aux schema that Justin_o and I came up to express composite panel: http://pastie.org/8384340

[12:59:49 CDT(-0500)] <cindyli> can you review it and feedback please

[13:02:48 CDT(-0500)] <Bosmon> Hi Justin_o

[13:03:01 CDT(-0500)] <Bosmon> I guess the "" fields are just placeholders,e tc.

[13:04:42 CDT(-0500)] <Bosmon> I'm puzzled that the format seems to provide that the same panel might appear both grouped and ungrouped at the same time

[13:19:41 CDT(-0500)] <Justin_o> Bosmon: yes.. we are trying to provide that

[13:20:17 CDT(-0500)] <Bosmon> Justin_o - can you think of a design justification for that?

[13:20:24 CDT(-0500)] <Justin_o> Bosmon: it's probably debatable if it's an actual issue, but we've been trying to support allowing the same panel be used in multiple groups and a by itself if need be

[13:20:49 CDT(-0500)] <Justin_o> Bosmon: probably along the lines of having multiple adjusters for the same preset

[13:20:52 CDT(-0500)] <Bosmon> Have you seen any designs where the same control appears in multiple places?

[13:21:12 CDT(-0500)] <Justin_o> Bosmon: no not yet, but we wanted this to flexible in case that ever did arise

[13:21:40 CDT(-0500)] <Bosmon> Justin_o - my feeling is that this is one of those things that could be called a "white elephant generalisation"

[13:21:40 CDT(-0500)] <Justin_o> Bosmon: if it is unreasonable, we can drop it... that's not a problem..

[13:21:58 CDT(-0500)] <Justin_o> Bosmon: okay.. we can remove that one..

[13:22:18 CDT(-0500)] <Justin_o> Bosmon: also the "" are just placeholders.. how does the rest of it look though

[13:50:51 CDT(-0500)] <anastasiac> Justin_o and/or cindyli: Could you tell me, with the new restructuring, what steps I now have to take to generate the generated CSS files for the prefs framework? is it an ant task, and if so what's the task name?

[13:51:34 CDT(-0500)] <cindyli> anastasiac: it's a task named "generatePrefsEditorThemes"

[13:51:41 CDT(-0500)] <Bosmon> Justin_o - it looks generally reasonable, plus or minus a few typos

[13:51:46 CDT(-0500)] <anastasiac> cidnyli, is it "ant generatePrefsEditorThemes" ?

[13:51:59 CDT(-0500)] <anastasiac> cindyli: ^

[13:52:41 CDT(-0500)] <cindyli> anastasiac: ant generatePrefsEditorThemes -lib lib/rhino

[13:54:11 CDT(-0500)] <anastasiac> cindyli and Justin_o: Should the StarterGrades.js file be required if I'm working with the schema version of the starter set?

[13:55:30 CDT(-0500)] <cindyli> anastasiac: shoouldn't need

[13:55:43 CDT(-0500)] <cindyli> let us know if it's wrong

[13:55:45 CDT(-0500)] <anastasiac> hm. The full-page demos fail if the file is not included

[13:55:52 CDT(-0500)] <cindyli> ha

[14:05:30 CDT(-0500)] <anastasiac> Justin_o: http://issues.fluidproject.org/browse/FLUID-5178

[14:06:51 CDT(-0500)] <anastasiac> Justin_o, I've added that JIRA to the DT iteration page

[14:07:15 CDT(-0500)] <anastasiac> I wonder if we should rename that page to Exploration Tool...

[14:18:27 CDT(-0500)] <Justin_o> anastasiac: i guess we could.. i can't remember the reason we didn't change all those when we renamed the component.. maybe jessm or jhung know

[14:25:02 CDT(-0500)] <Justin_o> Bosmon: thanks.. we're going to move forward with it and fix the typos

[14:35:54 CDT(-0500)] <Justin_o> michelled: could you please review https://github.com/fluid-project/infusion/pull/416

[14:36:08 CDT(-0500)] <Justin_o> anastasiac: submitted a pull request for FLUID-5178 thanks for spotting that issue

[14:36:19 CDT(-0500)] <anastasiac> np, Justin_o

[15:19:04 CDT(-0500)] <anastasiac> Justin_o and cindyli, can the "new" uiOptions component be invoked in full-page mode?