fluid-work IRC Logs-2011-09-15

[09:07:25 CDT(-0500)] <heidi_> Justin_o i'll be working on the IDI site today instead of testing fyi
[09:11:08 CDT(-0500)] <Justin_o> heidi_:
[09:11:09 CDT(-0500)] <Justin_o> okay
[09:11:13 CDT(-0500)] <Justin_o> i understand
[09:11:17 CDT(-0500)] <Justin_o> thanks for letting me konw
[09:11:18 CDT(-0500)] <Justin_o> know
[09:11:45 CDT(-0500)] <heidi_> Justin_o yeah jutta sent a bunch of site content this morning so organising it all/adding it... i'll keep ya posted
[09:12:12 CDT(-0500)] <Justin_o> heidi_: sure.. no problem.. thanks again for letting me know
[10:00:41 CDT(-0500)] <colinclark> Morning, everyone
[10:01:37 CDT(-0500)] <colinclark> huslage: Do you have some time to chat this afternoon?
[10:01:43 CDT(-0500)] <colinclark> We should catch up, post-vacation
[10:01:55 CDT(-0500)] <huslage> yes!
[10:01:59 CDT(-0500)] <colinclark> awesome!
[10:02:35 CDT(-0500)] <colinclark> huslage: 2 pm?
[10:02:46 CDT(-0500)] <huslage> looks good
[10:02:49 CDT(-0500)] <colinclark> great
[10:19:34 CDT(-0500)] <huslage> heidi_: can you login to dev.snow and make an obvious change?
[10:19:47 CDT(-0500)] <huslage> something i can see on the homepage?
[10:19:49 CDT(-0500)] <heidi_> huslage oh yay - yes one sec
[10:19:58 CDT(-0500)] <huslage> the saga might be over
[10:20:04 CDT(-0500)] <huslage> brute force FOR JUSTICE
[10:21:21 CDT(-0500)] <heidi_> huslage i forget which cms snow uses ha - drupal? admin url?
[10:21:34 CDT(-0500)] <huslage> drupal
[10:21:35 CDT(-0500)] <huslage> no idea
[10:21:41 CDT(-0500)] <huslage> /admin?
[10:22:08 CDT(-0500)] <huslage> liana or smriti would know
[10:22:20 CDT(-0500)] <huslage> liana stupid autocorrect in new macos. must disable
[10:22:24 CDT(-0500)] <huslage> lizna
[10:22:27 CDT(-0500)] <huslage> ARGH
[10:22:37 CDT(-0500)] <heidi_> huslage hehe... ok checking my emails... sec
[10:24:52 CDT(-0500)] <heidi_> jhung drupal admin is usually /administrator ya?
[10:32:27 CDT(-0500)] <heidi_> huslage checking in with smriti... will lt you know when i've changed something
[10:32:36 CDT(-0500)] <huslage> lizna is working on it
[10:35:02 CDT(-0500)] <heidi_> smriti's adding an acct for me on dev
[10:35:23 CDT(-0500)] <huslage> ok
[10:37:09 CDT(-0500)] <heidi_> huslage do you still need me to test stuff or is lizna?
[10:39:29 CDT(-0500)] <huslage> she's done it heidi_ the www site isn't working now :/
[10:39:38 CDT(-0500)] <heidi_> uh ohhh
[10:39:45 CDT(-0500)] <heidi_> keep me posted.
[10:46:40 CDT(-0500)] <huslage> ok heidi_ looks like dev is now separate from www.
[10:47:05 CDT(-0500)] <heidi_> huslage so dev is ready? is it up to date with the live site?
[10:48:56 CDT(-0500)] <huslage> yep
[10:49:01 CDT(-0500)] <huslage> they are copies of each other
[10:49:14 CDT(-0500)] <huslage> i had to sanitize some URLs when i copied it over though
[10:49:20 CDT(-0500)] <huslage> there were absolute links
[10:49:41 CDT(-0500)] <huslage> but i can do that again when i have to do the update, not a big deal
[10:50:43 CDT(-0500)] <huslage> god. if i never see drupal again it will be too soon.
[10:54:55 CDT(-0500)] <mlam> Justin_o: I have a solution for FLUID-4444. I'm going to do a bunch of testing before I make a pull request
[10:56:30 CDT(-0500)] <colinclark> what was the fix, mlam?
[10:57:51 CDT(-0500)] <mlam> colinclark: the chunk of markup that was ajaxed into the demo template didn't include the single file uploader markup
[10:58:00 CDT(-0500)] <heidi_> huslage thank you know it wasn't fun
[10:58:11 CDT(-0500)] <colinclark> mlam: Yeah, that's right
[10:58:18 CDT(-0500)] <colinclark> although I guess that has to be by design
[10:58:40 CDT(-0500)] <mlam> so i created a new selector, wrapped both uploader types within a <div> and ajaxed the markup within the div to the demo template
[10:58:41 CDT(-0500)] <colinclark> So I guess we could enumerate the different cases where we need to degrade gracefully to the single file uploader markup
[10:59:07 CDT(-0500)] <colinclark> mlam: What are those cases, can you list them just so we're clear about whether this solution will work?
[10:59:51 CDT(-0500)] <mlam> Yup. 1) In IE only - No flash is installed 2) In all browsers, javascript is turned off
[11:01:14 CDT(-0500)] <mlam> We have #2 covered; the case with javascript being disabled. The single file uploader markup is already in the demo template. And since ajax won't work with js being disabled, the single file uploader renders by default
[11:01:38 CDT(-0500)] <colinclark> So how come that case doesn't work for #1?
[11:03:53 CDT(-0500)] <mlam> The ajax call that we have in our demo originally grabbed the markup for the multiFile uploader only. And after we injected that markup in, the markup for the single file uploader disappeared. So it is overrode by the ajax call
[11:07:31 CDT(-0500)] <mlam> And now to guarantee that we will have single file uploader markup in the case Flash isn't installed, we grab that markup along with the multifile uploader as before and everything seems to be working (pre-extensive testing)
[11:14:41 CDT(-0500)] <colinclark> hmm
[11:14:44 CDT(-0500)] <colinclark> I think you might have to show me
[12:31:51 CDT(-0500)] <Justin_o> Bosmon: hello are you there?
[13:30:55 CDT(-0500)] <Bosmon> Hi Justin_o
[13:31:03 CDT(-0500)] <Justin_o> hello
[13:31:18 CDT(-0500)] <Justin_o> i'm looking at your pull request.. seems good, but could use some linting
[13:31:25 CDT(-0500)] <Bosmon> That's odd
[13:31:32 CDT(-0500)] <Bosmon> I carefully linted it this time before pushing.....
[13:31:57 CDT(-0500)] <Bosmon> Oh
[13:32:00 CDT(-0500)] <Bosmon> Perhaps it is in the test file
[13:32:40 CDT(-0500)] <Bosmon> Did anyone get a chance to look at the Uploader issue?
[13:32:41 CDT(-0500)] <Justin_o> Bosmon: yes.. it's the test file
[13:32:47 CDT(-0500)] <Bosmon> Ok, sorry about that
[13:33:28 CDT(-0500)] <Justin_o> np
[13:36:04 CDT(-0500)] <Bosmon> Oh
[13:36:08 CDT(-0500)] <Bosmon> I see my mail to the list didn't get there
[13:36:10 CDT(-0500)] <Bosmon> Good grief
[13:36:20 CDT(-0500)] <Bosmon> huslage!
[13:36:29 CDT(-0500)] <huslage> Bosmon: !
[13:36:39 CDT(-0500)] <huslage> good grief
[13:36:41 CDT(-0500)] <Bosmon> I got: This is the mail system at host gateway1.nyi.mail.srv.osa.
[13:36:51 CDT(-0500)] <Bosmon> <fluid-work@fluidproject.org>: connect to
[13:36:51 CDT(-0500)] <Bosmon> mx1.fluidproject.org[205.211.169.99]:25: No route to host
[13:36:56 CDT(-0500)] <huslage> whaaaa
[13:36:58 CDT(-0500)] <huslage> um
[13:36:59 CDT(-0500)] <huslage> odd
[13:37:21 CDT(-0500)] <Bosmon> Your message could not be delivered for more than 6 hour(s).
[13:37:21 CDT(-0500)] <Bosmon> It will be retried until it is 1 day(s) old.
[13:37:28 CDT(-0500)] <Bosmon> etc. etc.
[13:37:56 CDT(-0500)] <huslage> yeah ok
[13:38:05 CDT(-0500)] <huslage> something is strange with the network
[13:38:08 CDT(-0500)] <huslage> looking
[13:39:53 CDT(-0500)] <mlam> Justin_o: can you check my pull request for FLUID-4444?
[13:40:26 CDT(-0500)] <Justin_o> mlam: sure... does colinclark need too look at it at all?
[13:40:34 CDT(-0500)] <mlam> cindyli: maybe try merging my FLUID-4444 branch into the image gallery to see if it will fix your issue
[13:40:56 CDT(-0500)] <cindyli> mlam: sure. thanks
[13:41:06 CDT(-0500)] <Justin_o> Bosmon: for the uploader issue, if you were refering to the flickering one, i don't think anyone has looked at that yet
[13:41:11 CDT(-0500)] <Bosmon> Ok
[13:41:14 CDT(-0500)] <mlam> Justin_o: colinclark came by earlier to take a look and offered a cleaner solution
[13:41:20 CDT(-0500)] <Bosmon> If anyone wants to, I have a nice mail explaining what it is
[13:41:26 CDT(-0500)] <Bosmon> Currently being held in a queue somewhere....
[13:41:58 CDT(-0500)] <Justin_o> Bosmon: ah.. in virtual traffic
[13:43:58 CDT(-0500)] <Bosmon> So mlam, colinclark - anyone who is an "uploader guy"... to summarise my mail, oddly, FLUID-4442 doesn't seem to be a regression at all
[13:44:06 CDT(-0500)] <Bosmon> Since all the code which is causing it has been there all year
[13:44:28 CDT(-0500)] <Bosmon> But I thought I would run this insane conclusion past some people who knew more about it
[13:45:11 CDT(-0500)] <Bosmon> As far as I can see, it is caused by the "getChunkSize" method in the HTML5UploaderSupport.js
[13:45:16 CDT(-0500)] <Justin_o> Bosmon: so is something else causing it, because i don't remember seeing it before.. or has it just always been because our network was good or something like that?
[13:45:25 CDT(-0500)] <Bosmon> Which I can't quite see why it is there - since there is nothing equivalent on the Flash side
[13:45:41 CDT(-0500)] <Bosmon> Justin_o - I have no explanation why we didn't see it before....
[13:46:04 CDT(-0500)] <Bosmon> But yes, it may be that the person who tested against the live server had a connection that was so fast that they missed the flicker
[13:46:12 CDT(-0500)] <Bosmon> And we have no other kinds of tests that would expose the issue
[13:47:19 CDT(-0500)] <Justin_o> Bosmon: yep.. can you see anyway of addressing the issue?
[13:47:59 CDT(-0500)] <Justin_o> mlam: so for your change, you literally just moved the single file uploader markup out of the uploader container?
[13:48:11 CDT(-0500)] <colinclark> Bosmon: I can fill you in on the details of that
[13:48:17 CDT(-0500)] <Bosmon> Justin_o - as far as I can see, simply removing the "getChunkSize" method would fix the issue
[13:48:20 CDT(-0500)] <colinclark> but I will need a few minutes to finish up what I'm doing and remind myself of all the details
[13:48:25 CDT(-0500)] <Bosmon> colinclark - cool
[13:49:10 CDT(-0500)] <colinclark> Justin_o: If I review mlam's FLUID-4444, perhaps it would make sense for him to take a look at the potentially-related IE9 issue with UploadeR?
[13:49:20 CDT(-0500)] <colinclark> What do you think?
[13:49:31 CDT(-0500)] <Justin_o> colinclark: sounds good to me
[13:50:26 CDT(-0500)] <Bosmon> So bizarre that Crockford doesn't like space after "++"
[13:50:31 CDT(-0500)] <Bosmon> When he has it for every other operator....
[13:51:20 CDT(-0500)] <mlam> Justin_o: yes
[13:51:28 CDT(-0500)] <mlam> what's hte IE9 issue for the uploader?
[13:51:40 CDT(-0500)] <colinclark_> brb
[13:52:36 CDT(-0500)] <Bosmon> Ok Justin_o - I've pushed linting fixes to my branch
[13:52:37 CDT(-0500)] <Justin_o> Bosmon: please let me know when you have the linting all wrapped up
[13:52:39 CDT(-0500)] <Justin_o> thanks
[13:53:09 CDT(-0500)] <huslage> that's gross Bosmon
[13:53:22 CDT(-0500)] <huslage> you too Justin_o i'm ashamed
[13:54:37 CDT(-0500)] <Bosmon> huslage - perhaps the lint is clogging up our MX recrod?
[13:54:45 CDT(-0500)] <huslage> maybe so
[13:54:52 CDT(-0500)] <huslage> no. it's a host unreachable thing
[13:54:57 CDT(-0500)] <huslage> i can't get to lists.idrc either
[13:55:10 CDT(-0500)] <huslage> i can get to it locally via the machine it's hosted on
[13:55:14 CDT(-0500)] <huslage> so it's speaking
[14:01:13 CDT(-0500)] <Bosmon> Do we have people who we can raise tickets to about this kind of thing?
[14:47:52 CDT(-0500)] <Justin_o> Bosmon: did you test your FLUID-4449 in IE?