[05:14:49 CDT(-0500)] <chrispetsos> Hi Justin_o, i know this must be very early there, but any time you are here ping me please, in order to continue our conversation on the new infusion lib

[07:44:26 CDT(-0500)] <Justin_o> chrispetsos: hello

[07:44:50 CDT(-0500)] <chrispetsos> justin_o: hi

[07:45:34 CDT(-0500)] <chrispetsos> Justin_o: do you have some time? have you seen my last email?

[07:46:05 CDT(-0500)] <Justin_o> chrispetsos: yes.. i'm just taking a look at your code now.. so you are getting an error about the iframe still?

[07:46:52 CDT(-0500)] <chrispetsos> Justin_o: yes, if i remove the frame markup and that.jQuery error if i add it

[07:48:10 CDT(-0500)] <Justin_o> chrispetsos: interesting.. looking at the code i would have thought that what you did would have changed the prefsEditor.. let me try to run it.

[07:48:35 CDT(-0500)] <chrispetsos> Justin_o: OK

[07:54:17 CDT(-0500)] <Justin_o> chrispetsos: it has to do with the renaming.. you need to change uioType to prefsEditorType

[07:54:34 CDT(-0500)] <Justin_o> in your demo.js file

[07:55:34 CDT(-0500)] <Justin_o> i'll reply to your e-mail about that as well.

[07:56:58 CDT(-0500)] <chrispetsos> Justin_o: yes that did it, thanks. I'll get back to you if i need something more...

[07:57:23 CDT(-0500)] <Justin_o> chrispetsos: sounds good

[08:51:21 CDT(-0500)] <chrispetsos> Justin_o: OK, managed to create the first adjuster with the new Infusion lib, thanks! And going on...

[08:52:17 CDT(-0500)] <Justin_o> chrispetsos: great, oh by the way, we have a new way to do the strings, but we haven't updated the docs yet.. i'll try to get to that as soon as i can..

[09:05:45 CDT(-0500)] <colinclark> jessm: ux meeting is in an hour, is that right?

[09:05:59 CDT(-0500)] <jessm> colinclark: i believe so

[09:06:02 CDT(-0500)] <colinclark> I guess everything will go back to normal next week

[09:06:16 CDT(-0500)] <jessm> yep

[09:29:49 CDT(-0500)] <jessm> macfans: http://jamesfriend.com.au/pce-js/

[10:17:29 CDT(-0500)] <Justin_o> anastasiac: i have a fix for FLUID-5201 in my branch https://github.com/jobara/infusion/tree/FLUID-5201

[10:17:43 CDT(-0500)] <Justin_o> i'm going to try to fix up the other things as well and send a single pull request for all

[10:18:54 CDT(-0500)] <anastasiac> Justin_o, do you think you know how to fix the other issues?

[10:30:58 CDT(-0500)] <Justin_o> anastasiac: i'm not sure yet.. i'm going to try to investigate them some more..

[11:13:29 CDT(-0500)] <amilchev> hi Justin_o, I'm not sure if you understood my question in the list - [Ux] Combining two or more textfieldSteppers inside one panel

[11:14:24 CDT(-0500)] <Justin_o> amilchev: would you like to talk about it here a bit more

[11:14:34 CDT(-0500)] <amilchev> Justin_o: exactly

[11:15:15 CDT(-0500)] <amilchev> Justin_o: I pushed it in a new branch, you could run this file https://github.com/radmanovi4/prefsEditors/blob/gpii-271/demos/pcp/index.html

[11:15:37 CDT(-0500)] <amilchev> Justin_o: and see how the two textfieldSteppers share the same value

[11:16:59 CDT(-0500)] <amilchev> Justin_o: this is really work in process, ignore anything that seems strange to you I'm just trying to have 2 textfieldSteppers working in the same panel (with no success).

[11:18:41 CDT(-0500)] <amilchev> Justin_o: you could type "hook.model" in the console to check the current model - the upper textfieldStepper's value is stored in "value", the second one - "value2"

[11:19:47 CDT(-0500)] <amilchev> tell me what you get whenever you can, it's not that urgent

[11:22:42 CDT(-0500)] <Justin_o> amilchev: sorry, was looking through the code.. where is the preference map?

[11:24:03 CDT(-0500)] <amilchev> Justin_o: https://github.com/radmanovi4/prefsEditors/blob/gpii-271/src/shared/adjusters/pcp/js/speakTextPanels.js#L39-L74

[11:26:13 CDT(-0500)] <amilchev> Justin_o: here are the decorators: https://github.com/radmanovi4/prefsEditors/blob/gpii-271/src/shared/adjusters/pcp/js/speakTextCollectivePanel.js#L22-L49

[11:30:35 CDT(-0500)] <Justin_o> i'm going to guess that this is the problem https://github.com/radmanovi4/prefsEditors/blob/gpii-271/src/shared/adjusters/pcp/js/textfieldStepper.js#L149

[11:34:10 CDT(-0500)] <amilchev> yeah, sounds like it is...

[11:34:14 CDT(-0500)] <Justin_o> actually maybe it's line 145 that is the problem

[11:35:24 CDT(-0500)] <Justin_o> i guess they work together.. so you are sharing the model, which is updated externally by the other, and thus triggers model to update on the next one.

[11:35:28 CDT(-0500)] <Justin_o> at least it looks like that..

[11:36:35 CDT(-0500)] <Justin_o> amilchev: by the way, for now you should actually just make these completely separate panels and not worry about combining them into a larger panel.. I'll have the composite panel ready for use soon.. although anastasiac uncovered some bugs in it that i'm trying to weed out at the moment

[11:36:58 CDT(-0500)] <Justin_o> amilchev: but you should fix up this issue too.. i would suggest making use of the modelRelay system for now

[11:37:40 CDT(-0500)] <amilchev> Justin_o: isn't modelRelay useful when we need to coordinate two components' models?

[11:38:52 CDT(-0500)] <Justin_o> amilchev: yes.. but you are sort of doing that now, by sharing a model directly, as is on line 145

[11:39:11 CDT(-0500)] <Justin_o> amilchev: you should just keep the models separate and use the model relay to keep them in sync

[11:39:46 CDT(-0500)] <amilchev> Justin_o: ohh, I don't want them synchronised in any way

[11:40:03 CDT(-0500)] <amilchev> Justin_o: they have nothing to do with each other

[11:40:37 CDT(-0500)] <amilchev> Justin_o: I need them completely independant..

[11:40:54 CDT(-0500)] <Justin_o> amilchev: right not between the two textfieldSteppers for sure.. but you want the model of a textfieldStepper to be inSync with it's parent..

[11:41:27 CDT(-0500)] <amilchev> Justin_o: for that, yes undoubtedly..

[11:41:56 CDT(-0500)] <Justin_o> amilchev: so you'll need the model relay for that

[11:42:05 CDT(-0500)] <Justin_o> amilchev: right now by sharing the model like this https://github.com/radmanovi4/prefsEditors/blob/gpii-271/src/shared/adjusters/pcp/js/textfieldStepper.js#L145

[11:42:29 CDT(-0500)] <Justin_o> you are actually giving both of the textfieldSteppers the same model

[11:43:14 CDT(-0500)] <Justin_o> internally the textfieldStepper only works on the model path "value"

[11:49:53 CDT(-0500)] <amilchev> Justin_o: I think this was a temporary solution Evgeni created..

[11:50:38 CDT(-0500)] <amilchev> Justin_o: thanks, I'll stick to creating separate panels for each adjuster

[11:54:35 CDT(-0500)] <Justin_o> amilchev: okay.. let me know if you need anything else

[11:55:03 CDT(-0500)] <amilchev> Justin_o: thanks

[13:13:40 CDT(-0500)] <anastasiac> Justin_o, I've tested your branch for FLUID-5201, which fixes the slider rendering, and now this problem (i.e. the controls not reflecting the defaults) is showing up for the slider, as well.

[13:13:50 CDT(-0500)] <anastasiac> I've commented to that effect on the 5202 jira

[13:15:15 CDT(-0500)] <Justin_o> anastasiac: thanks.. i'm working onf FLUID-5202 now, so this is perfect timing

[13:21:11 CDT(-0500)] <anastasiac> Justin_o, I filed the textfieldSlider issue: http://issues.fluidproject.org/browse/FLUID-5203

[13:21:16 CDT(-0500)] <anastasiac> and added it to the iteration page

[13:30:48 CDT(-0500)] <Justin_o> anastasiac: thanks

[13:54:12 CDT(-0500)] <colinclark> hey yzen

[13:54:38 CDT(-0500)] <colinclark> Have you met the-t-in-rtf yet?

[13:57:46 CDT(-0500)] <yzen> colinclark:

[13:57:58 CDT(-0500)] <yzen> hi

[13:58:04 CDT(-0500)] <yzen> hi the-t-in-rtf

[13:58:07 CDT(-0500)] <yzen> i think so last week

[13:58:46 CDT(-0500)] <colinclark> Cool

[13:59:04 CDT(-0500)] <colinclark> So, Tony (i.e. the-t-in-rtf) is working on the Common Terms Registry

[13:59:08 CDT(-0500)] <colinclark> I made a new repo for it today

[13:59:12 CDT(-0500)] <colinclark> in the GPII github org

[13:59:28 CDT(-0500)] <colinclark> He was also mentioning how terribly inadequate our Kettle documentation is, in terms of working examples

[13:59:32 CDT(-0500)] <colinclark> which of course is to be expected

[13:59:37 CDT(-0500)] <colinclark> for such new software

[13:59:55 CDT(-0500)] <colinclark> but I'm hoping you might be able to answer his questions and share some examples of how you've used it, if he needs it

[14:01:30 CDT(-0500)] <yzen> colinclark, the-t-in-rtf definitely

[14:01:37 CDT(-0500)] <colinclark> awesome

[14:01:45 CDT(-0500)] <colinclark> he tends to work on Thursdays and Fridays, I think

[14:08:42 CDT(-0500)] <Bosmon> anastasiac - thanks for all the detail on your composite panels JIRAs

[14:09:00 CDT(-0500)] <Bosmon> In fact, if I'm not mistaken, the detail is now sufficient to allow the problems to be straightforwardly resolved - is this right, Justin_o? : P

[14:09:02 CDT(-0500)] <anastasiac> Bosmon, thanks for your patience. I hope it's enough info

[14:16:51 CDT(-0500)] <Justin_o> Bosmon: i think so.. in fact i have fixes for FLUID-5201 and FLUID-5202 already in branches in my repo

[14:17:02 CDT(-0500)] <Justin_o> Bosmon: any recommendations on how to take care of FLUID-5200 though

[14:17:26 CDT(-0500)] <Bosmon> Justin_o - well, surely the answer is just that you should do the necessary rewriting? : P

[14:18:00 CDT(-0500)] <Justin_o> Bosmon: if i remember correctly "..::" is a reference to some parent.. and these could be or arbitrary length, depending on nesting within the tree.. is that correct?

[14:18:53 CDT(-0500)] <Bosmon> Justin_o - I imagine that's not relevant since you always rewrite the tree in order to keep the original siblings next to each other?

[14:19:05 CDT(-0500)] <Bosmon> So the first segment I imagine will always stay fixed as ..:: ?

[14:19:41 CDT(-0500)] <Bosmon> I imagine all you need to do is insert the correct prefix after that point

[14:20:03 CDT(-0500)] <Justin_o> Bosmon: but what about the trees within the sub panels that could themselves have complex renderer trees

[14:20:23 CDT(-0500)] <Bosmon> Justin_o - I believe you are already rewriting everything?

[14:20:24 CDT(-0500)] <Justin_o> or i guess i'll never change those one

[14:20:41 CDT(-0500)] <Bosmon> So you are presumably already touching the entire protoTree from each panel

[14:20:58 CDT(-0500)] <Justin_o> yes.. just wondering if i might inherit complex trees from the subpanels

[14:21:06 CDT(-0500)] <Justin_o> Bosmon: by the way here's my fluid-5185 branch https://github.com/jobara/infusion/tree/FLUID-5185

[14:21:20 CDT(-0500)] <Bosmon> THanks, Justin_o

[14:21:27 CDT(-0500)] <Bosmon> So many things beginning with J!

[14:21:36 CDT(-0500)] <Justin_o> which has the start of the upgrades, currently qUnit, as well as getting the new jQuery UI and removing old plugins

[14:21:49 CDT(-0500)] <Justin_o>

[14:22:19 CDT(-0500)] <Justin_o> Bosmon: so you think i could just do a search for ..:: and should not encounter anything more complex than that?

[14:23:34 CDT(-0500)] <Bosmon> Justin_o - that ought to be fine

[14:23:58 CDT(-0500)] <Bosmon> Hopefully we will have a proper renderer by the time people start writing things that are much more complex : P

[14:24:00 CDT(-0500)] <Justin_o> Bosmon: okay.. i'll try to fix that up quickly then so that i can get you a pull request before i leave.. if not i'll try to send you one later tonight

[14:24:07 CDT(-0500)] <Justin_o> Bosmon:

[14:24:38 CDT(-0500)] <Justin_o> i was telling cindyli we'd hopefully be able to throw this all away soon.. with the new renderer on the horizon

[14:26:13 CDT(-0500)] <Justin_o> Bosmon: is there a name for "..::"

[14:27:04 CDT(-0500)] <Justin_o> i guess it's parentRelativeId

[14:28:45 CDT(-0500)] <Bosmon> That is what it is

[14:38:42 CDT(-0500)] <anastasiac> Yay! Bosmon's 4258 branch has been merged!

[14:39:09 CDT(-0500)] <Bosmon> DECCKLARATTIVE CHANGES are now available for all

[14:39:17 CDT(-0500)] <Bosmon> All we need to do now is document them : P

[14:48:28 CDT(-0500)] <colinclark> classic DARApplier timing, there

[14:50:58 CDT(-0500)] <colinclark> Justin_o, Bosmon: I just filed this issue http://issues.fluidproject.org/browse/FLUID-5204

[14:51:31 CDT(-0500)] <colinclark> We've got failures in both IE8 and 10

[14:51:36 CDT(-0500)] <colinclark> Some, I think, are overlapping

[14:51:39 CDT(-0500)] <colinclark> others, different

[14:53:41 CDT(-0500)] <Justin_o> Bosmon: can you look at my pull request https://github.com/fluid-project/infusion/pull/432

[15:39:32 CDT(-0500)] <Bosmon> Justin_o - I think it looks good

Browser not supported