fluid-work IRC Logs-2013-07-22

[07:55:08 CDT(-0500)] <jhung> heidiv want to sync up this morning?

[07:55:28 CDT(-0500)] <heidiv> jhung fo shizzle. give me 5

[07:55:35 CDT(-0500)] <jhung> k

[08:20:31 CDT(-0500)] <Justin_o> heidiv: by the way, did you get the jQuery UI themes in for low contrast?

[08:21:26 CDT(-0500)] <heidiv> Justin_o on my list for today

[08:21:51 CDT(-0500)] <heidiv> just chatting w jon now

[08:22:59 CDT(-0500)] <Justin_o> heidiv: great, thanks

[08:24:38 CDT(-0500)] <Justin_o> yzen: is this one done http://issues.fluidproject.org/browse/FLUID-5059

[08:25:09 CDT(-0500)] <Justin_o> I think you merged in cindyli's branch, but I also saw that you had another issue that was a duplicate of this one.. so not sure if there is more work needed

[08:26:45 CDT(-0500)] <yzen> Justin_o: let me take a lookg

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

[08:30:00 CDT(-0500)] <yzen> Justin_o: yes working

[08:30:55 CDT(-0500)] <yzen> ill resolve it

[08:33:09 CDT(-0500)] <Justin_o> yzen: thanks.. and i've closed it since it's in the project repo

[08:33:50 CDT(-0500)] <yzen> cool

[09:05:35 CDT(-0500)] <greggy> jhung: trying to figure out why the UIO icon are not showing up, any ideas?

[09:05:37 CDT(-0500)] <greggy> http://idrc.ocad.ca/dev/

[09:06:48 CDT(-0500)] <jhung> let me see greggy

[09:11:13 CDT(-0500)] <jhung> greggy: nothing jumps out to me right away greggy. From what I can see the CSS on that span inside the button looks okay. Is the @font-family declared properly in the CSS (i.e. is the path to the font correct)? Does the font appear if you use a different element?

[09:12:13 CDT(-0500)] <greggy> jhung: will try in a moment

[09:13:18 CDT(-0500)] <jhung> ok greggy

[09:20:02 CDT(-0500)] <Justin_o> cindyli: does your fat panel unit test work?

[09:21:28 CDT(-0500)] <cindyli> let me try

[09:23:48 CDT(-0500)] <cindyli> doesn't work, Justin_o. failed at template loading. let me see if i can quickly fix it. the changes i made to template and messages loading is easy to break the loading part

[09:26:35 CDT(-0500)] <cindyli> hey, Justin_o

[09:27:14 CDT(-0500)] <cindyli> which branch do you use for trying fat panel unit test?

[09:27:35 CDT(-0500)] <Justin_o> cindyli: i'm using FLUID-4907 and am trying to merge in master

[09:31:53 CDT(-0500)] <cindyli> Justin_o: i fixed the fat panel unit test and push up into my 4907 branch. it also contains some work for message loading

[09:32:07 CDT(-0500)] <Justin_o> cindyli: okay.. thanks

[09:32:12 CDT(-0500)] <cindyli> np

[10:22:34 CDT(-0500)] <colinclark> yzen: Are you around?

[10:23:08 CDT(-0500)] <yzen> colinclark: hi

[10:23:20 CDT(-0500)] <colinclark> evgeni1 is looking for you

[10:23:31 CDT(-0500)] <colinclark> He has questions about events, I think

[10:24:29 CDT(-0500)] <yzen> colinclark: this channel ? or tech, i did not get any messages

[10:24:58 CDT(-0500)] <colinclark> He pinged me on Skype about it

[10:28:18 CDT(-0500)] <evgeni1> yzen: hi

[10:28:33 CDT(-0500)] <yzen> evgeni1: hi

[10:29:11 CDT(-0500)] <evgeni1> yzen: a quick question about injecting events in infusion

[10:30:02 CDT(-0500)] <yzen> evgeni1: sure

[10:31:09 CDT(-0500)] <evgeni1> yzen: I'm trying to capture this event

[10:31:09 CDT(-0500)] <evgeni1> https://github.com/tsakov/universal/blob/77870db5c3ed03be824ce633d49fb9e48c9dae98/gpii/node_modules/flowManager/configs/base.json#L23

[10:31:09 CDT(-0500)] <evgeni1> here:

[10:31:09 CDT(-0500)] <evgeni1> https://github.com/tsakov/universal/blob/77870db5c3ed03be824ce633d49fb9e48c9dae98/gpii/node_modules/flowManager/configs/base.json#L78

[10:31:09 CDT(-0500)] <evgeni1> but how do I refer it?

[10:33:34 CDT(-0500)] <yzen> evgeni1: looking

[10:33:38 CDT(-0500)] <yzen> give me one sec

[10:35:42 CDT(-0500)] <greggy> jhung: the problem with the fonts not displaying seems to be the mime type not being defined….

[10:36:06 CDT(-0500)] <greggy> using AddType font/opentype ttf

[10:36:18 CDT(-0500)] <greggy> in a .htaccess file fixes the problem

[10:37:30 CDT(-0500)] <greggy> this might be something that need documenting

[10:37:44 CDT(-0500)] <greggy> anastasiac: ^

[10:39:14 CDT(-0500)] <jhung> greggy: ah okay. Glad you figured out that issue. Although I wonder if there is another way of doing this without modifying .htaccess. Either way, glad it works now.

[10:39:59 CDT(-0500)] <greggy> jhung: yes, if you have root access, it can be added to the mime.types file

[10:41:55 CDT(-0500)] <jhung> greggy: can you email me exactly the line(s) you added to the .htaccess file so we can update the documentation?

[10:42:10 CDT(-0500)] <greggy> AddType font/opentype ttf

[10:42:14 CDT(-0500)] <greggy> thats it

[10:45:35 CDT(-0500)] <michelled> Justin_o: I just pushed Bosmon's Pager pull request

[10:46:06 CDT(-0500)] <Justin_o> michelled: that was fast.. thanks

[10:46:47 CDT(-0500)] <michelled> np

[10:46:58 CDT(-0500)] <michelled> let me know if it helps your bug

[10:48:28 CDT(-0500)] <Justin_o> cindyli: michelled just merged in a pull request from Bosmon. I'm getting some conflicts and I'm not sure what to do with them.. would you be able to take a look

[10:48:39 CDT(-0500)] <cindyli> sure

[10:50:06 CDT(-0500)] <evgeni1> yzen: any luck?

[10:50:21 CDT(-0500)] <yzen> evgeni1: yes i think there's just a reference to a different thing

[10:50:24 CDT(-0500)] <yzen> so here

[10:50:50 CDT(-0500)] <Justin_o> cindyli: thanks

[10:51:07 CDT(-0500)] <cindyli> np

[10:52:17 CDT(-0500)] <yzen> evgeni1: oh actually one sec, so you want 2 things if i understand correctly: to update the composite event and to capture one of the events it's composed of ?

[10:53:02 CDT(-0500)] <yzen> namely the one called token ?

[10:53:04 CDT(-0500)] <evgeni1> yzen: capture all three, actually, but it's the same idea

[10:55:06 CDT(-0500)] <evgeni1> I'm interested in capturing the composing events

[10:55:41 CDT(-0500)] <yzen> evgeni1: ok got it and the current version that you have does not work , is that right

[10:55:42 CDT(-0500)] <yzen> ?

[10:56:11 CDT(-0500)] <yzen> ok one thing i can suggest is

[10:56:11 CDT(-0500)] <evgeni1> yzen: yes

[10:57:53 CDT(-0500)]

[10:58:31 CDT(-0500)]

[10:58:57 CDT(-0500)] <yzen> evgeni1: so onUserListener fires when we just get the user request

[10:59:09 CDT(-0500)] <yzen> evgeni1: referencing that thing should be fine i think

[11:00:33 CDT(-0500)] <evgeni1> yzen: may I call you on skype?

[11:01:03 CDT(-0500)] <yzen> evgeni1: sure, ill call you in a min

[12:49:55 CDT(-0500)] <Justin_o> cindyli1: yura found the problem

[12:50:15 CDT(-0500)] <cindyli1> what's about, Justin_o

[12:50:31 CDT(-0500)] <Justin_o> cindyli1: it's because the selector name is the same as the model path.. so it thinks it's coming from the same source

[12:50:42 CDT(-0500)] <cindyli1> which selector?

[12:55:48 CDT(-0500)] <Justin_o> cindyli1: the one that is used for rendering

[12:56:08 CDT(-0500)] <cindyli1> ok

[12:57:08 CDT(-0500)] <Justin_o> cindyli1: here's an example https://github.com/jobara/infusion/blob/FLUID-4907/src/webapp/components/uiOptions/js/Panels.js#L124

[12:57:39 CDT(-0500)] <Justin_o> cindyli1: this probably worked before because the model path was "selections.textFont"

[12:58:03 CDT(-0500)] <cindyli1> ah, good finding

[12:59:48 CDT(-0500)] <Justin_o> cindyli1: yes.. i wouldn't have figured that out…

[13:03:18 CDT(-0500)] <heidiv> Justin_o have we figured out which browsers the demo needs to support

[13:03:24 CDT(-0500)] <heidiv> ie8?

[13:04:03 CDT(-0500)] <greggy> Justin_o: should UIO linespacing work on both % and em

[13:06:41 CDT(-0500)] <Justin_o> greggy: hmm.. cindyli1 do you remember.. i think it just works with em but i'm not 100% sure

[13:07:21 CDT(-0500)] <cindyli1> trying to rmb

[13:08:02 CDT(-0500)] <greggy> it turns out that if line-height is defined in em or % there is no affect when chaning ZUIO line-height

[13:08:12 CDT(-0500)] <greggy> UIO

[13:08:33 CDT(-0500)] <greggy> http://idrc.ocad.ca/dev/

[13:09:02 CDT(-0500)] <Justin_o> greggy: yes.. i think you can't specify that ahead of time

[13:09:03 CDT(-0500)] <greggy> It does work when no line-heights are defined

[13:10:10 CDT(-0500)] <Justin_o> greggy: http://issues.fluidproject.org/browse/FLUID-4491

[13:11:04 CDT(-0500)] <greggy> got it

[13:13:42 CDT(-0500)] <Bosmon> Justin_o - subtle

[13:17:24 CDT(-0500)] <Justin_o> Bosmon: yes.. i'm trying to write a jira right now.. it will be more than two lines, but can't guarantee that it will make more sense than a tlj

[13:17:40 CDT(-0500)] <Bosmon> Justin_o - what will the selector say?

[13:17:44 CDT(-0500)] <Bosmon> er, the JIRA

[13:18:17 CDT(-0500)] <Justin_o> http://issues.fluidproject.org/browse/FLUID-5099

[13:19:02 CDT(-0500)] <Bosmon> Well it's a good JIRA, Justin_o - but what do you suggest be done to resolve it

[13:19:32 CDT(-0500)] <Justin_o> Bosmon: well that's a good question.. i was talking to yzen about this and we weren't quite sure.. does node.id have to be the selector?

[13:20:34 CDT(-0500)] <Bosmon> It's not normally exactly the selector - do you mean the class name?

[13:20:43 CDT(-0500)] <Justin_o> Bosmon: yes.. i guess so

[13:21:01 CDT(-0500)] <Bosmon> https://upload.wikimedia.org/wikipedia/commons/d/d6/Manoel.jpg

[13:21:57 CDT(-0500)] <Justin_o> Bosmon: and you are write.. so in more complex cases like the contrast in UIO it still works because it's themeRow:3:themeInput

[13:22:28 CDT(-0500)] <Bosmon> I guess we will just have to put prefixes in front of things

[13:22:33 CDT(-0500)] <Bosmon> Like DOM: or model: or whatever

[13:23:04 CDT(-0500)] <Justin_o> Bosmon: that should work

[13:23:19 CDT(-0500)] <Justin_o> Bosmon: i guess we didn't hit this before because the model paths had bee selector.textFont and the like

[13:27:18 CDT(-0500)] <heidiv> hey colinclark , do we know what browsers the disc tool demo needs to support? would ie8 be one we could skip?

[13:27:39 CDT(-0500)] <Justin_o> colinclark, cindyli1, michelled, anastasiac, yzen, jhung, heidiv: we talked a bit before about browser support for the discovery tool demo. We decided to only support "Modern Browsers" but didn't really clarify what that meant for IE

[13:28:27 CDT(-0500)] <Justin_o> I'd like to suggest that for the purposes of the demo we only support latest Chrome, Firefox, and Safari. Any objections of modifications to that ?

[13:28:40 CDT(-0500)] <heidiv> Justin_o not IE at all? ie10?

[13:29:36 CDT(-0500)] <clown> avtar: http://davmail.sourceforge.net/

[13:29:55 CDT(-0500)] <Justin_o> heidiv: yes.. that's not to say that we'll actively try to break it with vendor prefixes and etc. but that we won't test in it and things like text to speech may not work there

[13:30:44 CDT(-0500)] <heidiv> Justin_o not sure what you mean. it's ok if it doesn't work in latest IE?

[13:32:14 CDT(-0500)] <Justin_o> heidiv: yes. that's a much more concise answer

[13:32:24 CDT(-0500)] <Justin_o> at least that's what i'm proposing

[13:32:32 CDT(-0500)] <heidiv> "but we'll try" kinda thing?

[13:33:23 CDT(-0500)] <Justin_o> heidiv: this is one of the things that worries me about IE 10 for example http://issues.fluidproject.org/browse/VP-308

[13:33:48 CDT(-0500)] <avtar> clown: thank youuuuu

[13:33:59 CDT(-0500)] <Justin_o> and that we might get stuck spending a lot of time trying to get the styling of the page right as well as the new enactors

[13:36:14 CDT(-0500)] <heidiv> Justin_o gotcha. as long as that's ok with demo observers, works for me.

[13:36:45 CDT(-0500)] <clown> avtar: wlcm

[13:37:31 CDT(-0500)] <heidiv> Justin_o is there anyone we need to run this decision by other than our own team

[13:39:38 CDT(-0500)] <Justin_o> heidiv: good question. jessm and colinclark would probably know that

[13:45:18 CDT(-0500)] <colinclark> I think it's a reasonable approach that you're suggesting, Justin_o

[13:45:31 CDT(-0500)] <colinclark> If I understand correctly, you're saying:

[13:45:45 CDT(-0500)] <colinclark> 1. We won't be lazy, and we'll do our best to support IE 10

[13:45:58 CDT(-0500)] <colinclark> 2. If we run into major bugs or issues, we'll set them aside and keep going

[13:46:04 CDT(-0500)] <colinclark> is that correct?

[13:47:20 CDT(-0500)] <Justin_o> colinclark: basically yes, although the line between 1 and 2 might be a bit fine

[13:47:36 CDT(-0500)] <Justin_o> i guess we'll evaluate those on a case by case basis

[13:47:38 CDT(-0500)] <colinclark> yes, I imagine so

[13:47:49 CDT(-0500)] <colinclark> I guess the key point is that we shouldn't abandon testing on IE 10, right?

[13:48:04 CDT(-0500)] <colinclark> Otherwise we won't really know where the line between 1 and 2 actually is

[13:48:24 CDT(-0500)] <Justin_o> colinclark: i was kind of hoping we could, but i guess we have to

[13:48:44 CDT(-0500)] <colinclark> I think we can probably test less frequently on it, but we should probably keep an eye onit

[13:49:04 CDT(-0500)] <Justin_o> colinclark: okay

[13:49:46 CDT(-0500)] <colinclark> that was a suggestion, by the way

[13:49:54 CDT(-0500)] <colinclark> if you think it isn't feasible, I'm open to alternatives

[13:51:05 CDT(-0500)] <Justin_o> colinclark: i was mostly trying to save time since we know there will be problems with the video player and possibly text-to-speech

[13:51:25 CDT(-0500)] <colinclark> yup

[13:51:41 CDT(-0500)] <colinclark> You think TTS will be problematic in IE10?

[13:52:39 CDT(-0500)] <Justin_o> colinclark: we're still having problems with it in FF so i'm not exactly optimistic although I'm not sure IE 10 support has really been looked into yet.

[13:54:31 CDT(-0500)] <colinclark> lol

[13:54:38 CDT(-0500)] <colinclark> I'm hoping that once we fix FF, it'll work elsewhere

[13:59:24 CDT(-0500)] <Justin_o> colinclark: that would be great

[14:00:32 CDT(-0500)] <Justin_o> colinclark, Bosmon: do you remember what day we were talking about why we couldn't use arrays for the auxiliary schema.. i wanted to review the logs about it

[14:00:52 CDT(-0500)] <Bosmon> Justin_o - I think it was Friday

[14:00:58 CDT(-0500)] <Bosmon> But I could be wrong

[14:02:01 CDT(-0500)] <Justin_o> Bosmon: thanks.. i just found it.. it was actually Wednesday July 17

[14:15:14 CDT(-0500)] <Justin_o> Bosmon: how does this work for cases where the same panel is used for multiple preferences https://gist.github.com/colinbdclark/6022369

[14:15:19 CDT(-0500)] <Justin_o> like out links controls panel

[14:16:08 CDT(-0500)] <Justin_o> *our

[14:17:33 CDT(-0500)] <Justin_o> Bosmon: i assume it will just merge.. and that every panel used would have to have a unique grade

[14:17:48 CDT(-0500)] <Justin_o> meaning they would have all be different components

[14:23:34 CDT(-0500)] <Justin_o> Bosmon, yzen, cindyli1: does this look right for our modified auxiliary schema http://pastie.org/private/htwytl11vdahidlewjv1a

[14:24:44 CDT(-0500)] <Justin_o> and it would have to be able to handle the case of multiple enactors/panels for the same preference as illustrated here https://gist.github.com/colinbdclark/6022369

[14:24:45 CDT(-0500)] <cindyli1> looks fine to me. is this the option 2 on our schema proposal, Justin_o

[14:24:54 CDT(-0500)] <Justin_o> cindyli1: it's like option 2 yes..

[14:25:09 CDT(-0500)] <Justin_o> we should re-write that page and only show this one

[14:26:30 CDT(-0500)] <Justin_o> although i'm a bit confused still how it will know what to use for the key in the shared model..since this used to be what the auxiliary schema was keyed off of, but now we could have multiple keys which refer to the same model path

[14:28:30 CDT(-0500)] <heidiv> jhung for "replace CK12 logo with a dummy logo" - do we have one?

[14:29:44 CDT(-0500)] <jhung> We don't have anything at the moment. Just create a container. We'll probably make an icon font for it.

[14:29:50 CDT(-0500)] <jhung> ^heidiv

[14:30:31 CDT(-0500)] <heidiv> jhung maybe ill wait till it's ready, might as well do it all at once.

[14:30:54 CDT(-0500)] <jhung> okay. Can you make a Jira for that and put it in the iteration plan?

[14:32:29 CDT(-0500)] <Justin_o> cindyli1, yzen: the pastie i posted earlier should be the modification of our auxiliary schema.. i didn't push it to my branch just to make sure that you wouldn't accidentally merge my code and break your branches

[14:32:55 CDT(-0500)] <Justin_o> i'm heading out now.. see you wednesday

[14:56:12 CDT(-0500)] <jessm> yzen: ping

[14:56:24 CDT(-0500)] <yzen> jessm: pong

[14:58:07 CDT(-0500)] <jessm> yzen: do you know if OERPub has integrated UIO?

[14:59:04 CDT(-0500)] <yzen> jessm: well we did a prototype of that that had uio and table of contents integrated, but i am not sure they put it into production..

[15:01:20 CDT(-0500)] <jessm> yzen: anastasiac: we probably need to go look

[15:02:18 CDT(-0500)] <anastasiac> I was just checking out the oerpub site. I don't see anything there. still poking around to be sure

[15:02:36 CDT(-0500)] <jessm> anastasiac: their blog, which michelled reads often has updates

[15:02:42 CDT(-0500)] <yzen> jessm: i looked around a month ago or so it was not there

[15:02:51 CDT(-0500)] <jessm> yzen: ah, ok

[15:03:55 CDT(-0500)] <ramil> anastasiac, theres a css file called 'fss-base-global.css' that seems to be messing up the default font size making them smaller? what is this css for?

[15:05:14 CDT(-0500)] <anastasiac> ramil, that's the base css file for the Fluid Skinning System (FSS). It basically resets all the defaults for a page, which allows for a more consistent cross-browser experience. If the default it's using is too small, you can override it in the site's own CSS

[15:12:36 CDT(-0500)] <ramil> anastasiac, when using the Full Page UIO , I get this error: "Instantiation of autoInit component with type fluid.uiOptions.fullNoPreview failed, since selector " #myUIOptions " did not match any markup in the document [VM] MyInfusion.js?mqb2aj (989):287

[15:12:36 CDT(-0500)] <ramil> while constructing component of type fluid.uiOptions.fullNoPreview with arguments "

[15:13:17 CDT(-0500)] <ramil> should there be a div with an id of #myUIOptions on every page?

[15:14:06 CDT(-0500)] <anastasiac> ramil, that div should be on the page where the controls will appear. On the other pages, since you don't want the controls to appear, you shouldn't instantiate the uiOptions component, only the settings store and the page enhancer

[15:44:14 CDT(-0500)] <ramil> anastasiac, these css files: lib/jquery/ui/css/fl-theme-yb/yb.css Are these supposed to be part of the jQuery UI?

[15:45:26 CDT(-0500)] <anastasiac> ramil, they are created by using jQuery UI's theme roller http://jqueryui.com/themeroller/ They provide themes for any jQuery components to match the themes we have in UIO

[15:48:03 CDT(-0500)] <ramil> anastasiac, what do you suggest is the best way for me to include these? should I just make a copy of these files and put them inside my module and link to them there?

[15:49:03 CDT(-0500)] <anastasiac> I'm not sure, ramil. What does Drupal recommend as best practice for accessing CSS files provided by a library? Do they have any recommendations for that?

[15:51:30 CDT(-0500)] <ramil> anastasiac, that themeroller is like a custom build library right?

[15:52:10 CDT(-0500)] <anastasiac> ramil, theme roller is a tool that allows you to select the colours, etc. and it generates CSS to style jQuery widgets

[15:52:22 CDT(-0500)] <anastasiac> we used the tool to generate those CSS files

[15:53:08 CDT(-0500)] <ramil> so it kind of gives you a customized set of files right?

[15:57:45 CDT(-0500)] <ramil> anastasiac, when i built my custom Infusion , and exluded the jquery, jquery ui, jquery core, is this part of what is being excluded?

[15:58:17 CDT(-0500)] <anastasiac> ah, ramil, good question. You should be able to check the folder where you built infusion to double-check

[15:58:27 CDT(-0500)] <ramil> i see that those files are there

[15:59:25 CDT(-0500)] <ramil> if theyre not being excluded then i guess i could just point to those files since i know that theyre gonna be there even if someone excludes the jquery stuff

[16:00:11 CDT(-0500)] <ramil> oh i guess theyre not there after all

[16:02:41 CDT(-0500)] <ramil> anastasiac, i guess im just gonna copy them into my modules folder and link to it there