fluid-work IRC Logs-2011-07-26

[07:57:19 CDT(-0500)] <huslage> hello humans
[09:14:48 CDT(-0500)] <jameswy> anastasiac: Where's the current UIO tech demo that you had?
[09:15:28 CDT(-0500)] <anastasiac> jameswy, the original test page is in the manual-tests folder: SomeKindOfNews.html
[09:15:46 CDT(-0500)] <anastasiac> my version of it with all UIOs in it is in my branch:
[09:15:48 CDT(-0500)] * anastasiac digs
[09:16:07 CDT(-0500)] <anastasiac> https://github.com/acheetham/infusion/tree/FLUID-4351-newUIOContextDemo
[09:16:20 CDT(-0500)] <anastasiac> in the demos/uiOptions/html folder
[09:19:34 CDT(-0500)] <jameswy> Thanks
[09:49:27 CDT(-0500)] <colinclark> Hey anastasiac, I have a quick question
[09:49:52 CDT(-0500)] <anastasiac> colinclark, shoot
[09:50:29 CDT(-0500)] <colinclark> I'm just updating all the demos, etc. to link to the new !important-injected themes
[09:50:54 CDT(-0500)] <colinclark> I'm wondering why they all link to themes like rust, coal, slate, and mist?
[09:51:00 CDT(-0500)] <colinclark> SomeKindOfNews, for example
[09:51:17 CDT(-0500)] <colinclark> I see, in the dropdown, the ability to select from the four contrast themes
[09:51:23 CDT(-0500)] <anastasiac> ah, that would be a bug
[09:51:25 CDT(-0500)] <colinclark> HC, HCI, and the two y/b
[09:51:30 CDT(-0500)] <colinclark> Okay, so there's no reason for it anymore?
[09:51:42 CDT(-0500)] <anastasiac> no, not for rush, slate, coal, mist
[09:51:46 CDT(-0500)] <colinclark> Okay
[09:51:54 CDT(-0500)] <colinclark> These are the times when I miss fj4000
[09:51:55 CDT(-0500)] <anastasiac> the hc themes need to be there, obviously
[09:52:02 CDT(-0500)] <colinclark> he could sing the Tidy Up song for us
[09:52:06 CDT(-0500)] <anastasiac>
[09:52:12 CDT(-0500)] <colinclark> anyone else here with kids want to sing us a Tidy Up song?
[09:52:37 CDT(-0500)] <anastasiac> jhung? ^
[09:52:59 CDT(-0500)] <jhung> lol
[09:53:15 CDT(-0500)] <jhung> I actually don't know the words. I start it and then mumble the rest.
[09:53:53 CDT(-0500)] <colinclark>
[10:02:36 CDT(-0500)] <colinclark> Justin_o: I'm seeing some interesting UI Options behaviour when I run the Sakai demo...
[10:02:47 CDT(-0500)] <colinclark> You can actually see UI Options' iFrame briefly, and then it folds up
[10:02:50 CDT(-0500)] <colinclark> while the page is loading
[10:03:43 CDT(-0500)] <colinclark> I see this also with SomeKindOfNews
[10:03:49 CDT(-0500)] <colinclark> the panel is initially open
[10:03:51 CDT(-0500)] <colinclark> but blank
[10:03:51 CDT(-0500)] <colinclark> a
[10:03:55 CDT(-0500)] <colinclark> and then folds up
[10:04:01 CDT(-0500)] <Justin_o> colinclark: do you see any of the panel options or just an empty iframe?
[10:04:10 CDT(-0500)] <colinclark> Just an empty iFrame
[10:04:21 CDT(-0500)] <Justin_o> colinclark: this is a known issue..
[10:04:28 CDT(-0500)] <colinclark> What's the fix?
[10:04:37 CDT(-0500)] <Justin_o> i'm not sure yet
[10:05:00 CDT(-0500)] <Justin_o> colinclark: this is because we are hiding the panel offscreen first to calculate the font and linespacing sizes
[10:05:21 CDT(-0500)] <colinclark> If it's off screen, why would I see it, though?
[10:05:54 CDT(-0500)] <Justin_o> because the sliding panel has been initialized yet
[10:05:57 CDT(-0500)] <Justin_o> so it is just open
[10:06:01 CDT(-0500)] <Justin_o> to start
[10:06:13 CDT(-0500)] <Justin_o> then it closes on init
[10:07:12 CDT(-0500)] <colinclark> So, I guess with the panelized version of UI Options, we're expecting integrators will potentially put UI Options into every page on their site?
[10:07:20 CDT(-0500)] <colinclark> If that's the case, would you want every page on your site doing this?
[10:09:32 CDT(-0500)] <colinclark> michelled: You mentioned the uPortal demo yesterday, among the ones that supports UI Options
[10:09:37 CDT(-0500)] <colinclark> What should it actually do/
[10:09:38 CDT(-0500)] <colinclark> ?
[10:11:59 CDT(-0500)] <Justin_o> colinclark: so I think cindyli was able to remove the offscreen hiding bit, so we might be able to switch back the instantiantion order to remove the stuttering
[10:12:17 CDT(-0500)] <colinclark> That would make sense, if it's possible
[10:12:22 CDT(-0500)] <Justin_o> previously though the option was either to have what you see now, or to have the content of the panel flicker
[10:12:33 CDT(-0500)] <colinclark> Well, again...
[10:12:37 CDT(-0500)] <colinclark> if you were a page author
[10:12:52 CDT(-0500)] <colinclark> would either of those be acceptable enough to actually want to put UI Options into every page on your site?
[10:16:32 CDT(-0500)] <Justin_o> I'm not sure what's changed now that would let the size calculations work.. if it will work when switching the initializations back, but before it had to be visible somehow to make those calculations in Firefox.. this was decided to be the best solution at the time
[10:19:17 CDT(-0500)] <mlam> cindyli: I've pushed the FullNoPreview UIO integration tests up to my FLUID-4324 branch. I'm nto sure if you wanted to merge my branch into your's so that it's in the pull request?
[10:20:09 CDT(-0500)] <cindyli> sure, mlam, one sec, let me grant u the push access to my github
[10:20:19 CDT(-0500)] <colinclark> Does anyone know what the uPortal demo is supposed to do?
[10:20:54 CDT(-0500)] <cindyli> mlam: granted
[10:21:10 CDT(-0500)] <colinclark> I'm guessing, based on the head, that it's designed to enhance the page without embedding an instance of UIO in it
[10:21:12 CDT(-0500)] <mlam> ok, thanks cindyli
[10:21:20 CDT(-0500)] <colinclark> maybe?
[10:21:23 CDT(-0500)] <colinclark> There are also these buttons...
[10:21:28 CDT(-0500)] <Justin_o> colinclark: it has a layout reorderer, nested image reorderer, simple inline edit, and a ui enhancer
[10:21:30 CDT(-0500)] <colinclark> Customize My Portal
[10:21:39 CDT(-0500)] <cindyli> thank YOU, mlam
[10:21:45 CDT(-0500)] <athena> colinclark: i think once upon a time it had a ui options interface that let you change the text and such
[10:21:45 CDT(-0500)] <colinclark> Are these supposed to work, somehow? The Customize My Portal buttons
[10:21:59 CDT(-0500)] <athena> most of those probably wouldn't, unless you added all our js
[10:22:11 CDT(-0500)] <athena> i think maybe it had ui options and demo'd page reordering?
[10:22:18 CDT(-0500)] <athena> er, layout reordering
[10:22:33 CDT(-0500)] <athena> dunno if you guys want updated HTML for that page at some point
[10:22:47 CDT(-0500)] <colinclark> it's pretty crusty, isn't it athena?
[10:22:56 CDT(-0500)] <athena> yeah that's a pretty old version of uportal
[10:23:00 CDT(-0500)] <colinclark> I'm on the verge of git rm'ing it before we update it
[10:23:12 CDT(-0500)] <colinclark> We might as well just link to all the real-world usage of Infusion you have in the real thing
[10:23:13 CDT(-0500)] <athena> probably not a bad idea
[10:29:09 CDT(-0500)] <Justin_o> colinclark: what do you think of dropping both the sakai and uportal mockups
[10:55:34 CDT(-0500)] <michelled> colinclark: I've been wanting to get rid of those integration demos for a while
[10:55:52 CDT(-0500)] <michelled> I don't think they serve the purpose we originally intended them too.
[10:55:54 CDT(-0500)] <colinclark> I still often show the Sakai integration demo when I'm talking about UIO
[10:56:01 CDT(-0500)] <colinclark> but I don't think I've ever shown anything but the real uPortal
[10:56:09 CDT(-0500)] <michelled> really the uportal demo became a test page along the lines of SomeKindOfNews
[10:56:16 CDT(-0500)] <colinclark> Yeah
[10:56:25 CDT(-0500)] <colinclark> our nested Reorderer test page, or whatever
[10:56:25 CDT(-0500)] <michelled> it has tons of things in it and we were trying to see if they all worked together reasonably
[10:57:16 CDT(-0500)] <michelled> Justin_o: if we are removing it from integration demos I think it's still valuable as a manual test
[10:57:37 CDT(-0500)] <michelled> but I can be convinced otherwise
[11:00:31 CDT(-0500)] <Justin_o> michelled: i think we should create a simpler example.. there's a lot stuff in there that is unrelated to any of our code. particularly in regards to styling
[11:01:25 CDT(-0500)] <michelled> Justin_o: that's true. and it's no longer very similar to what uportal actually ships with so it really serves little purpose
[11:01:33 CDT(-0500)] <Justin_o> colinclark: also for the sakai one, will you still show it when we have the new showcase demo in place?
[11:01:45 CDT(-0500)] <colinclark> I guess it depends on the demo, Justin_o
[11:02:02 CDT(-0500)] <Justin_o> colinclark: so i guess we should see what jameswy comes up with
[11:08:41 CDT(-0500)] <colinclark> Justin_o: Do we have a bug filed for the fact that we can't fully style dropdowns in Safari?
[11:10:02 CDT(-0500)] <Justin_o> colinclark: yep
[11:10:05 CDT(-0500)] <Justin_o> let me find it
[11:11:38 CDT(-0500)] <Justin_o> colinclark: http://issues.fluidproject.org/browse/FLUID-4339
[11:12:23 CDT(-0500)] <Justin_o> colinclark: this had only worked in FF, but is not working there any more either.. so we are probably going to hold off on it till we get the proper rich dropdowns that work cross browser
[11:12:29 CDT(-0500)] <colinclark> yeah
[11:12:39 CDT(-0500)] <colinclark> I remember back when we were thinking about it...
[11:12:44 CDT(-0500)] <colinclark> that you can't reliably style dropdowns
[11:13:43 CDT(-0500)] <Justin_o> colinclark: yep.. this is one area where we might be able to have collaborate with jQuery UI
[11:13:53 CDT(-0500)] <colinclark> I'm sure they'd be keen on it
[11:14:33 CDT(-0500)] <Justin_o> colinclark: jorn was showing me that they have one in the works for 1.9
[11:14:41 CDT(-0500)] <colinclark> yeah
[11:14:49 CDT(-0500)] <colinclark> i'm sure it's the one we've been looking at
[11:17:25 CDT(-0500)] <heidi_> Justin_o i played with the filament group one (i think that's where it was) but it wasn't accessible
[11:18:23 CDT(-0500)] <Justin_o> http://wiki.jqueryui.com/w/page/12138056/Selectmenu
[11:24:49 CDT(-0500)] <colinclark> So, even with !important, UIO is no longer successfully blasting background images
[11:25:01 CDT(-0500)] <colinclark> I'll commit all this stuff so you can have a look at it
[11:34:59 CDT(-0500)] <colinclark> Justin_o, heidi_: https://github.com/colinbdclark/infusion/tree/FLUID-4219
[11:35:17 CDT(-0500)] <colinclark> If you try out the version of UIO with the preview, you can see where we don't knock out background images
[11:35:22 CDT(-0500)] <colinclark> I'm not sure if that's a goal or not
[11:41:31 CDT(-0500)] <Justin_o> colinclark: not exactly anymore.. http://issues.fluidproject.org/browse/FLUID-4298
[11:42:03 CDT(-0500)] <Justin_o> basically the background images should be appropriate for the theme. You can see this with the icons next to the sliders, which i think are background images..
[12:18:07 CDT(-0500)] <anastasiac> harriswong, I just had a look at your pull request #115, for FLUID-4309, removing the white bg colour from the fat panel UIO. Did you test this change on the manual test page SomeKindOfNews.html?
[12:18:30 CDT(-0500)] <harriswong> manual test, no i haven't. Let me go test it
[12:19:33 CDT(-0500)] <anastasiac> I'll put a comment on the pull request about the use case, harriswong
[12:19:43 CDT(-0500)] <harriswong> just tested, seems fine.
[12:19:54 CDT(-0500)] <harriswong> thanks anastasiac
[12:20:24 CDT(-0500)] <anastasiac> harriswong, it tested fine? I just tried it, and I can't see the panel...
[12:39:03 CDT(-0500)] <huslage> awesome dev people. use this. it's cool: http://vagrantup.com/
[12:44:36 CDT(-0500)] <Justin_o> huslage: cool.. do they have anything like that for vmware
[12:44:54 CDT(-0500)] <huslage> no Justin_o, but virtual box is just as good these days
[12:45:49 CDT(-0500)] <huslage> i switched
[12:46:07 CDT(-0500)] <Justin_o> really...
[12:46:14 CDT(-0500)] <huslage> oh and…FREE
[12:46:15 CDT(-0500)] <huslage> hehe
[12:46:23 CDT(-0500)] <Justin_o> yes.. free is hard to argue with
[12:46:28 CDT(-0500)] <huslage> vmware's network hooey was causing my wifi to disconnect regularly
[12:46:36 CDT(-0500)] <huslage> it was awful, so i had it turned off most of the time
[12:46:43 CDT(-0500)] <huslage> which was dumb…so i threw it out
[12:46:59 CDT(-0500)] <Justin_o> oh really.. haven't had that problem, but i can see how that would be a deal breaker
[12:48:21 CDT(-0500)] <huslage> it was only on certain networks
[12:48:30 CDT(-0500)] <huslage> but once an hour, my wifi would reconnect
[12:48:41 CDT(-0500)] <huslage> and it would inevitably be during some Skype call or something
[12:48:44 CDT(-0500)] <Justin_o> that's pretty frequent
[12:49:04 CDT(-0500)] <Justin_o> oh.. of course.. so it would take out the wifi to the whole computer and not just the vm
[12:49:53 CDT(-0500)] <heidi_> harriswong so for 4309 did you test src/webapp/tests/manual-tests/html/SomeKindOfNews.html
[12:50:32 CDT(-0500)] <harriswong> heidi_: i did after my pull request, anastasiac added a commnt to the request as well.
[12:50:46 CDT(-0500)] <heidi_> ok
[12:51:10 CDT(-0500)] <harriswong> i think that line was needed to deal with dark background... so now i am trying to figure out how to make the div to extend to th ebottom
[12:51:18 CDT(-0500)] <harriswong> like a min-height or height if possible.
[12:51:23 CDT(-0500)] <huslage> yeah Justin_o
[12:52:02 CDT(-0500)] <heidi_> harriswong you could set bg value for the diff themes
[12:52:15 CDT(-0500)] <heidi_> by default it's white, for the others it's another colour
[12:54:17 CDT(-0500)] <harriswong> is it like setting the bg colour on all the different theme css files, and then override the white background?
[12:55:44 CDT(-0500)] <harriswong> heidi_^
[12:56:29 CDT(-0500)] <heidi_> harriswong look in uiOptions.css at the bottom where all the themey extras are
[12:56:40 CDT(-0500)] <harriswong> k thanks
[13:07:10 CDT(-0500)] <Bosmon> cindyli - how is Configurability today?
[13:12:49 CDT(-0500)] <cindyli> Bosmon: sort of done with page enhancer. working on fullNoPreview and fullPreview
[13:12:56 CDT(-0500)] <cindyli> ui options part
[13:19:07 CDT(-0500)] <Bosmon> cindyli - that's great
[13:19:11 CDT(-0500)] <Bosmon> What kind of method are you using?
[13:20:06 CDT(-0500)] <cindyli> Bosmon: page enhancer already has certain level of options chewing, pageEnhancer actually accepts uiEnhancer options
[13:20:38 CDT(-0500)] <cindyli> so, classnameMap can be defined as a top level option
[13:20:52 CDT(-0500)] <cindyli> what I did is to enable toc template url as another top level option
[13:21:01 CDT(-0500)] <Justin_o> anastasiac: i'm assuming you will be implementing the new UIO demo. Could you make sure that the demo portal links are all working for it. Currently the Link back to the demo portal in all the uio demos is broken
[13:21:13 CDT(-0500)]

[13:22:10 CDT(-0500)] <cindyli> Bosmon: in terms of ui options, the first i'm working on is to make templateLoader configurable thru options
[13:22:30 CDT(-0500)] <Bosmon> Yes - are you working with a version of Justin_o's munging code, or something else?
[13:22:41 CDT(-0500)] <cindyli> a modified version
[13:22:51 CDT(-0500)] <cindyli> of Justin_o's code
[13:22:56 CDT(-0500)] <Bosmon> ok
[13:23:02 CDT(-0500)] <Bosmon> What does it look like?
[13:23:32 CDT(-0500)] <cindyli> ok, let me pastie
[13:24:52 CDT(-0500)] <cindyli> Bosmon: http://pastie.org/2275202
[13:25:30 CDT(-0500)] <Bosmon> Thanks, cindyli
[13:25:40 CDT(-0500)] <Bosmon> It looks like you will be writing a new function for each configuration of UIOptions
[13:25:41 CDT(-0500)] <cindyli> Bosmon: np. not working yet. but it explains the method
[13:25:45 CDT(-0500)] <Bosmon> Have you given any thought to writing a shared version?
[13:26:05 CDT(-0500)] <cindyli> good point
[13:27:51 CDT(-0500)] <cindyli> actually, this chunk of code should be able to shared by 3 versions of ui options
[13:27:59 CDT(-0500)] <Bosmon> yes, that would be good
[13:35:16 CDT(-0500)] <harriswong> anastasiac: i have made changes to https://github.com/fluid-project/infusion/pull/115.
[13:38:14 CDT(-0500)] <cindyli> Bosmon: anastasiac describes the difficulty to config "uiEnhancer" used by uiOptions preview window, i don't have a good idea to improve it
[13:39:02 CDT(-0500)] <cindyli> preview is a bit different, as uiEnhancer is instantiated directly as a subcomponent of preview on the IoC tree
[13:39:11 CDT(-0500)] <cindyli> not thru pageEnhancer
[13:40:42 CDT(-0500)] <cindyli> the best way i can come up with is to allow the configuration of "preview" at the top level
[13:42:14 CDT(-0500)] <cindyli> with this method, the integrator still have to provide quite a few levels to config its subcomponent "uiEnhancer"
[13:44:23 CDT(-0500)] <Bosmon> cindyli - I think we should be able to configure all subcomponents "as if" they were top-level subcomponents
[13:44:50 CDT(-0500)] <Bosmon> We can't really afford to smush ALL configuration into one giant record at the very top level, since they will surely start to collide before long
[13:48:49 CDT(-0500)] <cindyli> Bosmon: exactly, that's why in my pastie you will see, for instance, options object for "templateLoader" is linked with a name "templateLoader", so that we know which sub-component the options belong to
[13:48:53 CDT(-0500)] <cindyli> Bosmon: is that right?
[13:49:26 CDT(-0500)] <Bosmon> Yes, that seems reasonable
[13:49:42 CDT(-0500)] <cindyli> but i would still have difficulty with sub-components that are at the different level of IoC tree but with the same name
[13:49:47 CDT(-0500)] <cindyli> for instance, eventBinder
[13:50:02 CDT(-0500)] <Bosmon> cindyli - we don't need to tackle EVERYTHING, remember
[13:50:08 CDT(-0500)] <cindyli> exactly
[13:50:18 CDT(-0500)] <cindyli> i plan to skip eventBinder
[13:50:19 CDT(-0500)] <Bosmon> Just the ones which are kinds of "everyday usable" components
[13:50:41 CDT(-0500)] <cindyli> understand
[13:51:06 CDT(-0500)] <Bosmon> We might want to make the configuration look a little more like the standard subcomponent configuration
[13:51:26 CDT(-0500)] <Bosmon> I understand why you took out "components" but you may want to leave in "options" - since otherwise they have no way to replace the component with one of a different type
[13:51:26 CDT(-0500)] <cindyli> underneath "components" keyword?
[13:51:48 CDT(-0500)] <cindyli> ic
[13:52:28 CDT(-0500)] <Bosmon> Oddly "templateLoader" may just be one of the cases that don't need too much special treatment since the situation isn't too bad - it really IS a top-level subcomponent
[13:52:42 CDT(-0500)] <Bosmon> The really bad cases are where components are nested several levels deep at unpredictable locations...
[13:53:42 CDT(-0500)] <cindyli> agree
[13:56:54 CDT(-0500)] <heidi_> jameswy fyi http://wiki.fluidproject.org/display/fluid/IDI+website+final+mockup
[13:56:58 CDT(-0500)] <heidi_> chi put those up
[13:58:19 CDT(-0500)] <jameswy> Great!
[15:11:20 CDT(-0500)] <Bosmon> Hi cindy - could you push as far as you have got before you go home today?
[15:11:23 CDT(-0500)] <Bosmon> Cheers
[15:11:28 CDT(-0500)] <Bosmon> cindyli
[15:11:43 CDT(-0500)] <cindyli> sure, np, Bosmon
[15:25:35 CDT(-0500)] <cindyli> Bosmon: what i've got so far has been pushed into https://github.com/cindyli/infusion/tree/FLUID-4353
[15:28:25 CDT(-0500)] <cindyli> Bosmon: full with and w/o preview are working with the options chewing. fat panel still needs more work. let me know if more configuratons you feel need to be chewed.
[15:48:27 CDT(-0500)] <harriswong> Justin_o: I have filed the UIO visibility issue here: http://issues.fluidproject.org/browse/FLUID-4359