fluid-work IRC Logs-2013-07-19

[08:09:33 CDT(-0500)] <cindyli> Justin_o, yzen, i pushed more into my 4907 branch that has:

[08:09:47 CDT(-0500)] <cindyli> 1. the renaming of resourceLoader and resourcePath

[08:10:09 CDT(-0500)] <cindyli> 2. merged in antranig's pull request that fixed the issues we had yesterday

[08:10:25 CDT(-0500)] <cindyli> 3. removed the work-arounds for yesterday's issues

[08:11:06 CDT(-0500)] <cindyli> i wonder if one of you can review antranig's pull request https://github.com/fluid-project/infusion/pull/366, and push it if it looks fine

[08:13:12 CDT(-0500)] <Justin_o> cindyli: you merged this one into the FLUID-4907 branch already?

[08:13:19 CDT(-0500)] <cindyli> yes, Justin_o

[08:13:34 CDT(-0500)] <Justin_o> cindyli: and it sounds like it works right

[08:13:46 CDT(-0500)] <cindyli> ya

[08:15:49 CDT(-0500)] <yzen> cindyli: Justin_o I'm still trying to get the tts working, it looks like there are some issues with passing changes to the model to the enactor, somewhere in the custom model relay stuff (sad)

[08:20:13 CDT(-0500)] <cindyli> yzen: my pull request #360 that fixes our model relay is not in our 4907 branch yet, we should merge that in and try again

[08:20:30 CDT(-0500)] <yzen> cindyli: ok ill review it right now

[08:20:36 CDT(-0500)] <cindyli> thanks, yzen

[08:21:49 CDT(-0500)] <Justin_o> yzen: are you going to be reviewing both of those pull request?

[08:22:18 CDT(-0500)] <yzen> Justin_o: sure

[08:25:25 CDT(-0500)] <Justin_o> yzen: thanks

[08:34:44 CDT(-0500)] <yzen> anastasiac: hi

[08:44:30 CDT(-0500)] <yzen> anastasiac: ^

[08:49:56 CDT(-0500)] <anastasiac> yzen, I seem to have dropped out of the channel at a critical moment. What's your question?

[08:50:07 CDT(-0500)] <yzen> (smile)

[08:50:08 CDT(-0500)] <yzen> yzen: anastasiac: so what's the problem you are experiencing with the pref editors? i can't seem to pass the changes from the tts panel to the enactor, is that related ?

[08:50:36 CDT(-0500)] <anastasiac> yzen, the problem I was having will be fixed by cindy's fix yesterday

[08:50:44 CDT(-0500)] <anastasiac> so it's not a problem with the prefs editor

[08:50:51 CDT(-0500)] <anastasiac> it's a bug in UIO

[08:51:03 CDT(-0500)] <yzen> anastasiac: would you know if mine is related ?

[08:51:29 CDT(-0500)] <anastasiac> passing changed from the panel to the enactor… no, that doesn't sound anything like the problem I was experiencing.

[08:51:39 CDT(-0500)] <anastasiac> yzen, the tts panel used to talk to the enactor

[08:51:44 CDT(-0500)] <yzen> is saw a number of // this is being ignored - ?? in the custom panel defaults

[08:52:17 CDT(-0500)] <anastasiac> yes, that referred to some selectors

[08:52:33 CDT(-0500)] <anastasiac> yzen, your panel is not turning tts on?

[08:53:03 CDT(-0500)] <yzen> no (sad), when i click on the tts icon, the model change event in the enactor is not triggered

[08:54:17 CDT(-0500)] <anastasiac> yzen, yeah, I'm seeing it not working. The panel itself changes, but no tts, and the store doesn't get the new value.

[08:54:18 CDT(-0500)] <anastasiac> hm

[08:54:43 CDT(-0500)] <yzen> anastasiac: ya that (sad)

[08:55:11 CDT(-0500)] <anastasiac> yzen, I know we had this working… I'm going to check if my 118 branch still works, and if so compare them to see what's changed

[08:55:29 CDT(-0500)] <yzen> anastasiac: btw i have a pull with updated url

[08:55:34 CDT(-0500)] <yzen> the original does not work anymor

[08:55:39 CDT(-0500)] <yzen> perhaps you can review it

[08:55:46 CDT(-0500)] <yzen> you won't hear anything otherwise

[08:55:50 CDT(-0500)] <yzen> it 1 line

[08:55:51 CDT(-0500)] <anastasiac> could you give me the url and tell me what to do with it?

[08:55:55 CDT(-0500)] <yzen> ok

[08:56:10 CDT(-0500)] <anastasiac> or point me at the pull

[08:56:43 CDT(-0500)] <yzen> anastasiac: https://github.com/fluid-project/prefsEditors/pull/5/files

[08:58:08 CDT(-0500)] <cindyli> yzen: anastasiac, in terms of the enactors not responding to the panel change, Justin_o and i had the similar problem when we tried to remove "selections" layer from uio model. we traced down into fluid.fireSourcedChange() and suspected it might be a framework issue in there but haven't been able to succeeded in creating a test case

[08:58:09 CDT(-0500)] <anastasiac> ok, yzen, it definitely works in my older branch, using the same version of infusion. i'm going to compare the versions to see what's changed

[08:59:07 CDT(-0500)] <yzen> anastasiac: thanks

[08:59:11 CDT(-0500)] <yzen> cindyli: good to know

[09:01:23 CDT(-0500)] <Justin_o> cindyli: do you think you could pair with me for a bit.. i can't seem to find out what's going on with those two preferences.. the model relay isn't being triggered but i can't understand why

[09:01:52 CDT(-0500)] <cindyli> sure. the problem sounds familiar with what yzen is having

[09:02:01 CDT(-0500)] <cindyli> skype, Justin_o

[09:02:23 CDT(-0500)] <Justin_o> cindyli: sure

[09:02:37 CDT(-0500)] <yzen> cindyli: just a white space issue in your pull otherwise I will merge it

[09:03:07 CDT(-0500)] <cindyli> ok, yzen let me fix those. how do you see what white spaces are extra?

[09:03:31 CDT(-0500)] <yzen> cindyli: well it's in my diff for once, but i also have a plugin for eclipse that strips them on on save

[09:03:34 CDT(-0500)] <yzen> on file save

[09:03:37 CDT(-0500)] <yzen> sorry sublime

[09:03:42 CDT(-0500)] <yzen> not eclipse (smile)

[09:03:53 CDT(-0500)] <cindyli> cool plug in, which one it is

[09:03:57 CDT(-0500)] <yzen> one sec

[09:04:21 CDT(-0500)] <yzen> called trailing spaces

[09:04:32 CDT(-0500)] <yzen> make sure you set this in user prefs though

[09:04:33 CDT(-0500)] <yzen> "trailing_spaces_include_current_line": false

[09:04:37 CDT(-0500)] <anastasiac> yzen: found it!!

[09:04:39 CDT(-0500)] <yzen> {

[09:04:40 CDT(-0500)] <yzen> "trailing_spaces_include_current_line": false

[09:04:41 CDT(-0500)] <yzen> }

[09:05:21 CDT(-0500)] <anastasiac> yzen: the selector class in the Spoken panel template is wrong: it should be "flc-discoveryTool-spoken-choice" and not "flc-discoveryTool-speak-choice"

[09:05:29 CDT(-0500)] <anastasiac> make that change, and it works

[09:05:44 CDT(-0500)] <yzen> anastasiac: (smile)

[09:05:46 CDT(-0500)] <yzen> duh

[09:05:48 CDT(-0500)] <yzen> thank you (smile)

[09:05:49 CDT(-0500)] <anastasiac> add that to your pull request

[09:06:24 CDT(-0500)] <anastasiac> well, yzen, it would be hard to figure out without being able to compare with a working version. I could check that the only thing that changed as markup and css, so...

[09:06:48 CDT(-0500)] <yzen> anastasiac: will do

[09:07:41 CDT(-0500)] <anastasiac> yzen, it doesn't seem to want to turn off if you reload the page while it's on...

[09:24:36 CDT(-0500)] <anastasiac> Justin_o, cindyli, yzen: I'm looking for work. Is there anything I can help anyone with, or should I start on the 'more text' enactor and panel?

[09:25:20 CDT(-0500)] <Justin_o> anastasiac: are the presets all working with the designs from jhung and cindyli ?

[09:25:59 CDT(-0500)] <anastasiac> Justin_o, yes. cindyli has an Infusion pull request that will fix the problem with the panels not reflecting the settings on reload, but other than that, current panels all working

[09:26:08 CDT(-0500)] <anastasiac> yzen is working on the tts stuff

[09:27:35 CDT(-0500)] <Justin_o> anastasiac: excellent.. i think starting in on the new enactor makes sense then

[09:28:00 CDT(-0500)] <anastasiac> will do, Justin_o. Maybe michelle can apri with me when she gets in

[09:28:51 CDT(-0500)] <Justin_o> anastasiac: that would be great

[09:41:19 CDT(-0500)] <cindyli> yzen: lovely plugin, works like a charm. the pull request for model relay is ready

[09:41:34 CDT(-0500)] <yzen> (smile)

[09:41:35 CDT(-0500)] <yzen> thanks

[09:47:31 CDT(-0500)] <cindyli> Justin_o: i'm continuing with the mysterious model relay. keep in touch with our findings

[09:48:06 CDT(-0500)] <Justin_o> cindyli: sounds good

[09:49:52 CDT(-0500)] <Justin_o> yzen: have you gotten in those pull requests to infusion.. greggy will probably want to do a check out soon, but i told him he'd probably want to wait till you got those in

[09:50:12 CDT(-0500)] <yzen> Justin_o: going through Bosmon's one right now

[09:50:17 CDT(-0500)] <yzen> cindyli's one is in

[09:50:32 CDT(-0500)] <cindyli> super, thanks, yzen

[09:50:57 CDT(-0500)] <Justin_o> yzen: thanks

[09:54:15 CDT(-0500)] <greggy> yzen: Justin_o: let me know when the push is done. I've already cloned, but will pull in any changes you make

[10:00:35 CDT(-0500)] <yzen> anastasiac,Justin_o cindyli: all pull requests are in

[10:00:42 CDT(-0500)] <anastasiac> yay!

[10:00:45 CDT(-0500)] <cindyli> great!

[10:02:35 CDT(-0500)] <Justin_o> yzen: great, thanks

[10:02:37 CDT(-0500)] <Justin_o> greggy: ^^

[10:03:05 CDT(-0500)] <greggy> Justin_o: got it

[10:10:10 CDT(-0500)] <cindyli> Justin_o: the problem with the demo text font is not caused by the model relay, it was from uiEnhancerRelay

[10:10:29 CDT(-0500)] <cindyli> everytime at uio model change, uie model gets updated by uiEnhancerRelay

[10:11:15 CDT(-0500)] <cindyli> if u set a break point at uiOptions.js, line 332 where uiEnhancerRelay is doing its work, u will see text font change doesn't trigger the uie model update

[10:11:26 CDT(-0500)] <cindyli> i'm looking into why

[10:13:55 CDT(-0500)] <Justin_o> cindyli: thanks.. i'll look from here too

[10:16:26 CDT(-0500)] <avtar> yzen: ping

[10:16:34 CDT(-0500)] <yzen> avtar: sup

[10:17:10 CDT(-0500)] <avtar> how does tts.idrc.ocadu.ca make you feel?

[10:19:49 CDT(-0500)] <Justin_o> cindyli: the updateEnhancerModel event doesn't fire

[10:20:33 CDT(-0500)] <cindyli> ya, which comes from onUIOptionsRefresh

[10:22:24 CDT(-0500)] <Justin_o> cindyli: but i'm not seeing anything trigger that event, not even the ones that work

[10:22:59 CDT(-0500)] <cindyli> ya, i'm also looking for that

[10:23:41 CDT(-0500)] <cindyli> Justin_o: it likely comes from panels.js line 314

[10:24:17 CDT(-0500)] <cindyli> wrong, line 371

[10:26:35 CDT(-0500)] <Justin_o> i think the panels are refreshed when UIO is.. that is to say, the event triggers the refresh

[10:26:40 CDT(-0500)] <Justin_o> cindyli: ^

[10:27:14 CDT(-0500)] <cindyli> ok

[10:29:16 CDT(-0500)] <cindyli> right

[10:40:51 CDT(-0500)] <yzen> avtar: hi, sorry missed your messages, very warm and fuzzy

[10:40:54 CDT(-0500)] <yzen> it's good

[10:47:45 CDT(-0500)] <avtar> yzen: awesome

[10:47:57 CDT(-0500)] <yzen> avtar: thanks!

[10:52:07 CDT(-0500)] <cindyli> Justin_o: should we bug yzen on this? he wrote uiEnhancerRelay

[10:52:22 CDT(-0500)] <yzen> oh noes

[10:53:45 CDT(-0500)] <cindyli> yzen: not sure if you know the problem we have with new demo, text font and inputs larger don't work because the uie model doesn't get updated by uiEnhancerRelay when those two things change. other prefs work fine tho

[10:54:15 CDT(-0500)] <yzen> hmm and you think it's a relay ?

[10:54:16 CDT(-0500)]

<cindyli> the update of uie model seems triggered by event "updateEnhancerModel", which comes from "

Unknown macro: {fluid.uiOptions}


[10:55:06 CDT(-0500)] <cindyli> if u set a break point at uiOptions.js, line 332 where uiEnhancerRelay is doing its work, u will see text font change doesn't trigger the uie model update

[10:55:23 CDT(-0500)] <cindyli> but other pref changes do tho

[10:56:21 CDT(-0500)] <cindyli> we couldn't find the place where onUIOptionsRefresh is triggered that chain trigger "updateEnhancerModel". you might know better on this

[10:57:47 CDT(-0500)] <yzen> cindyli: i think some of those events are specific to the fat panel..

[10:58:01 CDT(-0500)] <colinclark> vjoanna: Can you fire me a link to the latest Discovery Tool wireframes, along with any kind of description of its functionality that you might have?

[10:58:33 CDT(-0500)] <colinclark> michelled: As far as you know, is your Discovery Tool roadmap all up to date and awesome?

[11:04:40 CDT(-0500)] <vjoanna> colinclark: here are the latest wireframes: http://wiki.fluidproject.org/display/fluid/Preference+Discovery+Tool+%28June+9%29 there's some description on that page and a more detailed one here: http://wiki.fluidproject.org/pages/viewpage.action?pageId=34578234#PreferenceManagementEcosystemFeatures%26Functionality%28June7%29-discovery

[11:07:46 CDT(-0500)] <colinclark> thanks vjoanna!

[11:08:00 CDT(-0500)] <vjoanna> np

[11:29:48 CDT(-0500)] <michelled> colinclark: yep, we update it as we work

[11:30:12 CDT(-0500)] <colinclark> ok, awesome

[11:30:13 CDT(-0500)] <colinclark> thank you!

[11:30:58 CDT(-0500)] <michelled> np - it's been really helpful for all of us to keep that wiki page up to date. then no one person holds to knowledge of what's going on (wink)

[11:37:55 CDT(-0500)] <Justin_o> cindyli: are you free to do a DT checkin at 1pm

[11:38:06 CDT(-0500)] <cindyli> sure, Justin_o

[11:38:28 CDT(-0500)] <cindyli> here's yura's update: "i added some fixes to the tts but it's still weird in firefox, it speaks 1st sentence and then stops Β i m leaving shortly but i pushed everything into my GPII-125 branch"

[11:55:22 CDT(-0500)] <Justin_o> cindyli: did you make any progress on the issue with the preferences not changing in our branch?

[11:55:33 CDT(-0500)] <cindyli> not yet

[11:56:06 CDT(-0500)] <Justin_o> cindyli: okay.. i'll keep digging into it

[11:56:13 CDT(-0500)] <cindyli> thanks

[12:05:14 CDT(-0500)] <Justin_o> cindyli, jhung, anastasiac, michelled: lets check in now about the DT work

[12:05:25 CDT(-0500)] <cindyli> ok, Justin_o

[12:05:39 CDT(-0500)] <Justin_o> cindyli: we'll try to Skype you in

[12:05:48 CDT(-0500)] <cindyli> thanks

[14:05:32 CDT(-0500)] <Bosmon> Hi cindyli - how are things going on?

[14:06:07 CDT(-0500)] <cindyli> Bosmon: the fat panel demo is running with the schema input but a couple of bugs

[14:06:37 CDT(-0500)] <Bosmon> cool

[14:06:41 CDT(-0500)] <cindyli> one is that the page doesn't respond to the changes on text font and inputs larger, Justin_o and i are investigating this issue

[14:07:28 CDT(-0500)] <Justin_o> Bosmon: i'm also looking at it with michelled at the moment, but no luck yet

[14:07:35 CDT(-0500)] <Justin_o> cindyli: ^

[14:07:39 CDT(-0500)] <cindyli> seems it's not specific with the schema work, even if I switch back to use regular options api, the same issue persists. something in uiOptions.js or fatPanel.js is broken

[14:07:56 CDT(-0500)] <Bosmon> Well, that's promising at least (smile)

[14:08:02 CDT(-0500)] <Justin_o> Bosmon, cindyli : the model is changing in the panel, but it isn't transmitting that through the model really, the call in the model relay just isn't' happening

[14:08:27 CDT(-0500)] <cindyli> Justin_o: did you try to remove "selections" layer from uio model?

[14:09:12 CDT(-0500)] <Justin_o> cindyli: the daily build is still working.. so it's an issue in our branch at least

[14:09:27 CDT(-0500)] <Justin_o> cindyli: the selections layer should be completely removed from the uio model already

[14:09:36 CDT(-0500)] <Justin_o> i made that change the other day to get the panels working at all

[14:09:51 CDT(-0500)] <cindyli> i think that's where it breaks the thing. we had the same issue before

[14:11:53 CDT(-0500)] <ramil> anastasiac, it seems that the default FatPanelUIOptionsFrame.html file is still being loaded even with the code that you added

[14:12:16 CDT(-0500)] <anastasiac> ramil, have you pushed your code to github

[14:12:17 CDT(-0500)] <anastasiac> ?

[14:12:31 CDT(-0500)] <ramil> ill do that right now

[14:14:35 CDT(-0500)] <ramil> anastasiac, i've pushed it

[14:19:26 CDT(-0500)] <anastasiac> ugh, my mistake, ramil. The fix I proposed would work for fat-panel, but not for full-page (doh! pretty obvious, when you look at the first argument to fluid.demands())

[14:19:36 CDT(-0500)] <anastasiac> so how to do it for full page? ...

[14:19:57 CDT(-0500)] <ramil> what do you mean?

[14:20:27 CDT(-0500)] <ramil> i ve actually figured out how to load the necessary css files conditionally so the full page should be working now but its not working properly

[14:22:50 CDT(-0500)] <anastasiac> ramil, I'm not clear what you're referring to when you say "it's not working properly" - do you mean the css loading?

[14:23:19 CDT(-0500)] <ramil> no i mean when you try to change the settings like the text size and try to apply it , it doesnt really work

[14:24:48 CDT(-0500)] <anastasiac> right, it's not styled properly, and it doesn't work

[14:25:15 CDT(-0500)] <anastasiac> ramil, you said you got the necessary css files to load conditionally? should that be working in the code you pushed?

[14:25:40 CDT(-0500)] <ramil> yes it should

[14:25:56 CDT(-0500)] <ramil> and you can check when you go to the full page, all the necessary css files are there

[14:26:18 CDT(-0500)] <anastasiac> for the full page version, ramil? It certainly isn't styled properly for me

[14:26:26 CDT(-0500)] <Justin_o> cindyli: i don't see anywhere left where selections is present. Also i just tried to change inputsLarger, to input, and links to emphasizeLinks in the internal models..and have the same problems.. the emphasizeLinks still works, but the inputsLarger do not

[14:27:14 CDT(-0500)] <ramil> oh try clearing the cache

[14:27:26 CDT(-0500)] <Justin_o> cindyli: how did you test that without the schema it was also broken

[14:27:27 CDT(-0500)] <Justin_o> ?

[14:27:32 CDT(-0500)] <cindyli> Justin_o: toc stops working too when i switch back to use old api

[14:27:51 CDT(-0500)] <cindyli> exactly, also broken, and broken even more

[14:29:08 CDT(-0500)] <Justin_o> cindyli: i mean.. how were you able to test it

[14:29:43 CDT(-0500)] <cindyli> Justin_o: just uncommented out the old code in uiOptionsDemo.js and uiOptions.html

[14:30:34 CDT(-0500)] <Justin_o> cindyli: i have a broken IoC reference when i do that.. for the page enhancer option thing.. did you push any fixes up today

[14:30:35 CDT(-0500)] <Justin_o> ?

[14:30:37 CDT(-0500)] <anastasiac> ramil, much better (smile)

[14:30:56 CDT(-0500)] <anastasiac> there are still some styling problems with the 'colour and contrast' icons, ramil, but it's still way better

[14:31:27 CDT(-0500)] <anastasiac> ramil, I'm getting a jQuery error when I try to use the sliders, do you see that?

[14:31:41 CDT(-0500)] <Justin_o> cindyli: i get this error

[14:31:42 CDT(-0500)]

<Justin_o> "Failed to resolve reference

Unknown macro: {originalEnhancerOptions}

.options.originalUserOptions - could not match context with name originalEnhancerOptions from component leaf [object Object]"

[14:31:45 CDT(-0500)] <cindyli> Justin_o: i pushed up my branch with the old code uncommented

[14:32:01 CDT(-0500)] <ramil> anastasiac, i didnt see those errors

[14:32:03 CDT(-0500)] <cindyli> ya, i had that too, fixed in my branch, Justin_o

[14:32:16 CDT(-0500)] <anastasiac> ramil, are you able to drag the sliders to adjust the values?

[14:32:22 CDT(-0500)] <ramil> yes

[14:32:24 CDT(-0500)] <Justin_o> cindyli: did you push that fix up?

[14:32:34 CDT(-0500)] <cindyli> yes, into my github 4907

[14:32:39 CDT(-0500)] <anastasiac> odd, it doesn't work for m

[14:32:59 CDT(-0500)] <Justin_o> cindyli: thanks, i'll try that out

[14:33:01 CDT(-0500)] <anastasiac> oh well, we can start with the text fields and figure out why the values aren't applying

[14:34:08 CDT(-0500)] <Justin_o> cindyli: lots not working there.. that's so strange

[14:34:48 CDT(-0500)] <cindyli> 3 things working there: text size, line spacing, emphasized links

[14:34:50 CDT(-0500)] <anastasiac> ramil, the settings are being saved, so the fat panel itself is doing its job. The pageEnhancer is the one responsible for applying the settings to the page, so that's the next place to start investigating what's going wrong

[14:35:01 CDT(-0500)] <cindyli> oh, as well as theme, Justin_o

[14:35:32 CDT(-0500)] <ramil> anastasiac, are you looking at the fat panel or the full page uio?

[14:35:40 CDT(-0500)] <anastasiac> right now, full page

[14:35:50 CDT(-0500)] <anastasiac> and I think I might know why the settings aren't applying

[14:36:05 CDT(-0500)] <Justin_o> cindyli: i'm not seeing emphasize links working

[14:36:22 CDT(-0500)] <anastasiac> ramil, your demo.initPageEnhancer function is using "fluid.uiEnhancer.starterActions" when it should be using "fluid.uiEnhancer.starterEnactors"

[14:36:37 CDT(-0500)] <cindyli> u r right, Justin_o. it doesn't work

[14:36:49 CDT(-0500)] <Justin_o> cindyli: okay.. well something wrong (smile)

[14:37:03 CDT(-0500)] <cindyli> ya (sad)

[14:37:04 CDT(-0500)] <anastasiac> ramil, that fixes it for me!

[14:38:00 CDT(-0500)] <ramil> anastasiac, which file is this?

[14:38:24 CDT(-0500)] <anastasiac> ramil, where you define your demo.initPageEnhancer() function, in your functions.js file

[14:39:30 CDT(-0500)] <Justin_o> cindyli: sorry, i have to run.. i'lll try to look into this again monday if you haven't figured it out first

[14:39:44 CDT(-0500)] <cindyli> ok, see u monday, Justin_o

[14:40:19 CDT(-0500)] <ramil> anastasiac, it seems to work only on that page itself?

[14:40:24 CDT(-0500)] <cindyli> hi, Bosmon

[14:41:03 CDT(-0500)] <Bosmon> Hi cindyli!

[14:41:26 CDT(-0500)] <anastasiac> ah, right ramil. The pageEnhancer is responsible for applying the settings to the page, so your module is going to have to make sure there's a pageEnhancer on every page somehow. That should probably be your next task

[14:41:56 CDT(-0500)] <ramil> anastasiac, there should be because im including that same js file for each page

[14:41:59 CDT(-0500)] <cindyli> i wonder if i can do something like this with IoCSS:

[14:42:07 CDT(-0500)] <cindyli> {

[14:42:07 CDT(-0500)]

<cindyli> source: "

Unknown macro: {that}


[14:42:07 CDT(-0500)]

<cindyli> target: "

Unknown macro: {that templateLoader resourcePath}


[14:42:07 CDT(-0500)] <cindyli> }, {

[14:42:07 CDT(-0500)]

<cindyli> source: "

Unknown macro: {that}


[14:42:07 CDT(-0500)]

<cindyli> target: "

Unknown macro: {that messageLoader resourcePath}


[14:42:08 CDT(-0500)] <cindyli> }

[14:42:11 CDT(-0500)] <cindyli> Bosmon: ^

[14:42:19 CDT(-0500)] <anastasiac> ramil, you need to invoke those two initialization functions on every page

[14:43:50 CDT(-0500)] <Bosmon> cindyli - yes, you should be able to do that

[14:44:01 CDT(-0500)] <Bosmon> And it would be a good use of the framework facility

[14:44:11 CDT(-0500)] <cindyli> awesome!

[14:44:21 CDT(-0500)] <Bosmon> But isn't there just one templateLoader in general?

[14:44:36 CDT(-0500)] <Bosmon> Btw, re our discussion earlier in the week, we were considering renaming that as the "resourceLoader"

[14:44:47 CDT(-0500)] <Bosmon> Since it will shortly be used to load I18n resource bundles too

[14:47:00 CDT(-0500)] <cindyli> i'm thinking to use two instances of currently called "resourceLoader" for different things.

[14:47:22 CDT(-0500)] <Bosmon> cindyli - why 2 instances?

[14:47:25 CDT(-0500)] <cindyli> resourceLoader doesn't really loading anything in, it's not calling fetchResources, it just expands the urls

[14:47:32 CDT(-0500)] <Bosmon> Oh, I see

[14:47:44 CDT(-0500)] <Bosmon> Why not name it "urlExpander" (smile)

[14:47:47 CDT(-0500)] <cindyli> so if we have 2 instances, might be more clear

[14:47:55 CDT(-0500)] <cindyli> yes, the name is misleading

[14:48:04 CDT(-0500)] <cindyli> damn, i just renamed it to resourceLoader

[14:48:11 CDT(-0500)] <cindyli> seems have to do the renaming again

[14:49:46 CDT(-0500)] <Bosmon> If only we had a.... AUTOMATED REFACTTORING TOOL!

[14:49:53 CDT(-0500)] <Bosmon> Unfortunately sgithens seems to be asleep : P

[14:50:19 CDT(-0500)] <Bosmon> cindyli - so you do you expect more than one of "resourcePath" nested underneath the loaders?

[14:50:30 CDT(-0500)] <Bosmon> Yes, we may as well combine "templateLoader" and "messageLoader" into one component

[14:52:29 CDT(-0500)] <ramil> anastasiac, is FullNoPreviewUIOptions.css only supposed to be loaded on the Full Page UIO page or on every page? And what about FullUIOptions.css?

[14:53:47 CDT(-0500)] <cindyli> Bosmon, the inline currently contains 2 subcomponents: templateURLExpander and loader. the new structure is to add in another sub of "messageURLExpander", loader will be doing loading: 1. load in templates for panels, 2. load in messages and create another UIO subcomponent of messageResolver

[14:54:25 CDT(-0500)] <cindyli> how do you think of that?

[14:54:30 CDT(-0500)] <anastasiac> ramil, you can figure it out by the filenames themselves, and by looking at the demos: the full page versions use FullUIOptions.css. The noPreview version also uses the FullNoPreviewUIOptions.css, while the full-page with preview uses FullPreviewUIOptions.css, etc. Make sense?

[14:55:00 CDT(-0500)] <anastasiac> the full page css files are only used on the actual UIO controls page, ramil

[14:55:09 CDT(-0500)] <anastasiac> those files style the controls

[14:55:29 CDT(-0500)] <ramil> but the fulluiOptions.css is supposed to be on every page?

[14:55:35 CDT(-0500)] <Bosmon> cindyli - how is the messageURLExpander different from the templateURLExpander?

[14:56:31 CDT(-0500)]

<cindyli> Bosmon: they take in different prefixes and templates block. templates block is like

Unknown macro: {uiOptions}

[14:56:56 CDT(-0500)] <Bosmon> cindyli - ok - but they have the same grade content hopefully

[14:57:03 CDT(-0500)] <cindyli> yes they do

[14:57:07 CDT(-0500)] <Bosmon> But we have managed to centralise on a single "loader" component?

[14:57:09 CDT(-0500)] <Bosmon> Cool

[14:57:19 CDT(-0500)] <Bosmon> or "resourceLoader"

[14:58:11 CDT(-0500)] <cindyli> i'm thinking of different loading for different outputs. but let me think about one loader for all

[14:59:35 CDT(-0500)] <Bosmon> cindyli - it seems we can't begin instantiating child components until ALL I/O has finished

[14:59:36 CDT(-0500)] <cindyli> Bosmon: gonna leave for an appointment. if u have more idea on this, drop it here or an email. i will check back. thanks

[14:59:40 CDT(-0500)] <Bosmon> Whether it is for templates or messages

[14:59:52 CDT(-0500)] <Bosmon> So it seems to imply we must have a single loader component

[15:00:18 CDT(-0500)] <cindyli> ya, makes sense

[15:05:01 CDT(-0500)] <ramil> anastasiac, i think i fixed it. I just need the right code to load my own template file for the Fat Panel frame

[15:05:13 CDT(-0500)] <anastasiac> ramil, excellent

[15:18:38 CDT(-0500)] <ramil> anastasiac, the current php file that I'm using as the Fat Panel Frame template doesn't really work right now, I need to create that file in a different way which is not that simple so if youre using that to try out your code, you're probably gonna get an error.

[15:19:05 CDT(-0500)] <anastasiac> ok, ramil, thanks for the warning