fluid-work IRC Logs-2008-12-15

Owned by Fluid IRC Log Bot
Last updated: Dec 15, 2008Version comment
47 min read

[08:22:24 EST(-0500)] * Justin_o (n=Justin@142.150.154.101) has joined #fluid-work
[08:24:22 EST(-0500)] * phiggins (n=dante@c-68-34-199-67.hsd1.tn.comcast.net) has joined #fluid-work
[09:31:32 EST(-0500)] * anastasiac (n=team@142.150.154.160) has joined #fluid-work
[09:41:51 EST(-0500)] * jessm (n=Jess@c-24-34-214-137.hsd1.ma.comcast.net) has joined #fluid-work
[09:48:16 EST(-0500)] * clown (n=clown@guiseppi.atrc.utoronto.ca) has joined #fluid-work
[09:58:22 EST(-0500)] * colinclark (n=colin@bas1-toronto09-1279534686.dsl.bell.ca) has joined #fluid-work
[09:59:10 EST(-0500)] <colinclark> Justin_o: Morning.
[09:59:25 EST(-0500)] <Justin_o> colinclark: Hello, good morning
[09:59:40 EST(-0500)] <colinclark> Are our servers all back up now?
[10:00:13 EST(-0500)] <Justin_o> no, the daily build site is still down
[10:00:24 EST(-0500)] <colinclark> ok. That's a drag.
[10:00:36 EST(-0500)] <colinclark> Did you get a chance to take a peek at my FLUID-1768 branch?
[10:00:42 EST(-0500)] <Justin_o> I was thinking it more of a sign
[10:01:06 EST(-0500)] <colinclark> (smile)
[10:01:08 EST(-0500)] <Justin_o> not yet i'm going to be doing that shortly though... i guess i have to pull down the branch seperate from the trunk?
[10:01:49 EST(-0500)] <colinclark> Justin_o: Yep, just check the branch out. I tried to make it as easy as possible to evaluate without actually committing such a substantial change.
[10:02:22 EST(-0500)] <anastasiac> Justin_o, fj4000 asked for reviews of FLUID-1758 and FLUID-1966 - I'll have a look at those
[10:02:55 EST(-0500)] <colinclark> Justin_o: I found a bug this weekend that I wasn't sure if you knew about.
[10:03:09 EST(-0500)] <Justin_o> anastasiac: thanks
[10:03:14 EST(-0500)] <colinclark> It looks like when the Uploader css consolidation happened, it broken the Uploader springboard.
[10:03:28 EST(-0500)] <Justin_o> colinclark: what broke
[10:03:36 EST(-0500)] <colinclark> It throws an error on start.
[10:03:58 EST(-0500)] <Justin_o> hmm... yes... that doesn't sound like something I knew about
[10:04:10 EST(-0500)] <Justin_o> that was part of the fluid-1935 commit
[10:04:11 EST(-0500)] <colinclark> It seems that there are two element .fl-uploader-queue classes within the same container.
[10:04:31 EST(-0500)] <colinclark> It should be insanely easy to fix, but I didn't feel up to it at 2:30 am last night. (wink)
[10:04:47 EST(-0500)] <Justin_o> understandable, i'm surprised you are up already
[10:04:57 EST(-0500)] <colinclark> me too
[10:05:11 EST(-0500)] <colinclark> Couldn't sleep... dreaming of virtual machines and Flash installations. (sad)
[10:07:16 EST(-0500)] <colinclark> Well, I'm glad you guys were into the idea of easing into code freeze...
[10:07:31 EST(-0500)] <colinclark> I just saw so many requests for reviews and interesting questions that hadn't yet been discussions.
[10:07:57 EST(-0500)] <colinclark> It seems to me that we should be very careful not to take on any new bugs, but just do get the reviews done and fix any issues that arise from the current crop of fixes.
[10:08:31 EST(-0500)] <Justin_o> i think anastasiac and I are in total agreement with that... i think i'll send out an updated bug parade with only those needing review on it
[10:08:42 EST(-0500)] <colinclark> Cool.
[10:12:15 EST(-0500)] <colinclark> Justin_o: I think I've found a pretty disappointing bug in my Flash 10 fix.
[10:12:30 EST(-0500)] <colinclark> I'll be curious to see if you encounter it, and then how we decide to classify it.
[10:12:52 EST(-0500)] <colinclark> In short, it looks to me like SWFUpload 2.2.0b3 is broken on the server in FF3 (at least on Windows).
[10:13:40 EST(-0500)] <Justin_o> interesting... i'll take a look at it
[10:14:12 EST(-0500)] <colinclark> That's what kept me up so late last night... tracking it down. This morning I just tripled checked with a totally unmodified version of swfupload.js.
[10:14:27 EST(-0500)] <colinclark> That said, my Firefox installation is somewhat fried.
[10:14:35 EST(-0500)] <colinclark> FF2 decided to upgrade itself to FF3.
[10:14:45 EST(-0500)] <colinclark> And then when I reinstalled 2, confusion ensued. (smile)
[10:14:54 EST(-0500)] <colinclark> Wish I had taken a snapshot of my VM.
[10:31:52 EST(-0500)] <anastasiac> Justin_o, my review of FLUID-1758 has been added to the JIRA and emailed to the list.
[10:31:59 EST(-0500)] <anastasiac> It still needs work
[10:32:22 EST(-0500)] <Justin_o> (sad) oh well that's what today is for
[10:32:25 EST(-0500)] <anastasiac> but it's a patch that wasn't committed, so no damage actually done
[10:49:39 EST(-0500)] <anastasiac> Justin_o, I've commented on FLUID-1966 and emailed the list. The patch doesn't seem to help
[10:52:22 EST(-0500)] <Justin_o> really... that's too bad
[11:25:07 EST(-0500)] * athena7 (n=athena7@99.129.101.69) has joined #fluid-work
[11:34:57 EST(-0500)] <anastasiac> Justin_o, regarding FLUID-1935: I did review this last week, and commented on the JIRA.
[11:35:14 EST(-0500)] <anastasiac> Same for FLUID-1964
[11:36:13 EST(-0500)] <Justin_o> oh yes.... sorry some of those are still marked because the review, testing, or a resolution may not be complete
[11:44:58 EST(-0500)] <anastasiac> Justin_o, I just had a look at FLUID-1875, the Sakai UI Options sample
[11:45:09 EST(-0500)] <anastasiac> one of the three issues you found is still present
[11:45:14 EST(-0500)] <anastasiac> the other two seem to be fine now
[11:48:58 EST(-0500)] <Justin_o> yes... i have broken that one out into a seperate issue http://issues.fluidproject.org/browse/FLUID-1963
[11:49:09 EST(-0500)] <Justin_o> I believe it was something that was already present
[11:49:45 EST(-0500)] <Justin_o> I'm trying to remember now, but i believe that was the conclusion that we came to when i was speaking with fj4000 about it last weeke
[11:49:53 EST(-0500)] <Justin_o> i should have marked that on the bug
[11:52:05 EST(-0500)] * Bosmon (n=Antranig@ginger.caret.cam.ac.uk) has joined #fluid-work
[11:52:10 EST(-0500)] * michelled (n=team@142.150.154.197) has joined #fluid-work
[11:52:11 EST(-0500)] <Justin_o> in reflections i'm not sure if FLUID-1963 should be a blocker... any thoughts
[11:52:32 EST(-0500)] <Justin_o> it currently isn't but maybe it should be
[11:54:39 EST(-0500)] * ecochran (n=ecochran@dhcp-169-229-212-73.LIPS.Berkeley.EDU) has joined #fluid-work
[12:03:41 EST(-0500)] * colinclark (n=colin@142.150.154.101) has joined #fluid-work
[12:03:58 EST(-0500)] <anastasiac> Justin_o: regarding FLUID-1963, I'd be tempted to make it a blocker. The lack of "pre-popluation" of the dialog with the current settings means it's very easy to lose all your settings
[12:04:50 EST(-0500)] <anastasiac> I'll have a look at FLUID-1947
[12:05:00 EST(-0500)] * everettz (n=chatzill@user147-141.wireless.utoronto.ca) has joined #fluid-work
[12:05:09 EST(-0500)] <Justin_o> anastasiac: that is true
[12:05:24 EST(-0500)] <Justin_o> michelled: how hard do you think FLUID-1936 would be to fix
[12:10:35 EST(-0500)] <everettz> Justin_o: Are you / Colin available to meet with Rob and I tomorrow after lunch?
[12:11:03 EST(-0500)] <michelled> Justin_o: I don't see the error in FF2 or FF3
[12:12:01 EST(-0500)] <michelled> oops - looking at 1936 instead of 1963
[12:13:21 EST(-0500)] <michelled> Justin_o: it's a tiny change.
[12:13:27 EST(-0500)] <michelled> actually, this is a regresstion
[12:13:35 EST(-0500)] <michelled> I'd fixed this problem a long time ago
[12:13:41 EST(-0500)] <Justin_o> oh... really... okay...
[12:13:53 EST(-0500)] <Justin_o> my mistake on that...
[12:14:02 EST(-0500)] <michelled> no, it's not your mistake
[12:14:12 EST(-0500)] <michelled> the problem has been reintroduced at some point
[12:14:24 EST(-0500)] <michelled> shall I create a patch and have it reviewed?
[12:14:48 EST(-0500)] <Justin_o> yes please... you may want to take a look at 1875, i'm not sure if it was reintroduced there
[12:14:57 EST(-0500)] <michelled> it should be a matter of changing the init script to happen only once instead of every time 'edit appearance' is clicked
[12:16:29 EST(-0500)] <Justin_o> michelled: I have set Fluid-1963 as a blocker and will put it on the bug parade
[12:21:29 EST(-0500)] <anastasiac> ok, I said I'd look at 1947, but ecochran has reviewed it, so I will of course defer to the expert (smile)
[12:24:52 EST(-0500)] <ecochran> anastasiac: colinclark and I just discussed 1947 and I'm going to commit. There is a very small cosmetic problem that was not introduced with the patch but become more obvious with the patch but it can wait until the release. I'll write that up in my resolution for the bug, and add a new bug.
[12:30:40 EST(-0500)] <anastasiac> Justin_o, I think FLUID-1964 is good to go. The only issue is font size, and I don't think that's sufficient to hold it up.
[12:32:31 EST(-0500)] <anastasiac> Justin_o, I'd concur with ecochran on FLUID-1935 - good to go.
[12:36:26 EST(-0500)] <michelled> I've just attached a patch to FLUID-1963
[12:36:59 EST(-0500)] <anastasiac> michelled, Justin_o, I'll review FLUID-1963
[12:37:07 EST(-0500)] <Justin_o> anastasiac: did you want the name space issues fixed for FLUID-1935
[12:37:57 EST(-0500)] <anastasiac> Justin_o, I think it should be changed eventually, but I'm not sure if it's important for this release. Not sure what others think - michelled, colinclark, ecochran?
[12:39:31 EST(-0500)] <ecochran> Justin_o: I personally think that the CSS name space issue needs a little thoughtful talk before fixing. It's not a hard problem but with a little chatter I think that we could nail it down tight. So not for this release!
[12:39:58 EST(-0500)] <Justin_o> ecochran: it seems that the fix for FLUID-1935 has caused a problem with the uploader springboards
[12:40:19 EST(-0500)] <ecochran> Justin_o: ouch! I thought I checked that...
[12:40:31 EST(-0500)] <ecochran> it looked like the Springboard was independent
[12:40:35 EST(-0500)] <ecochran> I'll look at that now
[12:44:03 EST(-0500)] <Justin_o> thanks... colinclark caught this one, he was saying that it probably has to do with both of the uploaders having the same class
[12:46:16 EST(-0500)] <colinclark> Looks like there are two elements with the ".fl-uploader-queue" class.
[12:46:27 EST(-0500)] <colinclark> And thus the FileQueueView is failing to init.
[12:48:17 EST(-0500)] <michelled> anastasiac, I just realized that the js file didn't have a license and the file had never been linted
[12:48:33 EST(-0500)] <michelled> the changes are just declaring globals and a couple of spaces
[12:48:38 EST(-0500)] <anastasiac> yes, I was wondering about the license
[12:48:38 EST(-0500)] <michelled> do you want me to create a new patch?
[12:48:42 EST(-0500)] <anastasiac> user
[12:48:44 EST(-0500)] <anastasiac> sure
[12:48:47 EST(-0500)] * anastasiac can type!
[12:49:15 EST(-0500)] <ecochran> colinclark: do you want to fix or shall I? Or maybe Jacob? Where fj4000 this morning?
[12:49:35 EST(-0500)] <colinclark> fj4000 is away this morning
[12:49:41 EST(-0500)] <colinclark> if you could fix it, that would be huge
[12:49:45 EST(-0500)] <ecochran> np
[12:49:48 EST(-0500)] <colinclark> i am in the midst of scary debugging hell
[12:50:07 EST(-0500)] <ecochran> colinclark: I don't want to distract you! (wink)
[12:50:11 EST(-0500)] <ecochran> I'm on it
[12:52:45 EST(-0500)] <Bosmon> I have looked over FLUID-1768 "generally"
[12:52:54 EST(-0500)] <colinclark> Bosmon: (smile)
[12:53:10 EST(-0500)] <Bosmon> Anything specific I could be doing?
[12:53:12 EST(-0500)] <michelled> anastasiac, I've uploaded a fresh patch
[12:54:18 EST(-0500)] * jessm (n=Jess@c-24-34-214-137.hsd1.ma.comcast.net) has joined #fluid-work
[12:55:06 EST(-0500)] <colinclark> ecochran: I think I have uncovered a somewhat horrible bug.
[12:55:25 EST(-0500)] <ecochran> I am so listening!
[12:55:38 EST(-0500)] <colinclark> The layers of evil present in SWFUpload seem to be endless.
[12:55:47 EST(-0500)] <colinclark> evil all the way down
[12:56:08 EST(-0500)] <Bosmon> And in less than 1000 lines of code (tongue)
[12:57:02 EST(-0500)] <colinclark> In SWFUploadManager.removeFiles() we call "swfUpload.cancelUpload(fileId);"
[12:57:21 EST(-0500)] <colinclark> This seems to blow up in the demo mode with SWFUpload 2.2.0
[12:57:42 EST(-0500)] <colinclark> It seems as if they have systematically dismantled the functionality of their entire library with this release.
[12:57:49 EST(-0500)] <Bosmon> .....
[12:59:18 EST(-0500)] <anastasiac> Justin_o, I have reviewed michelled's patch for FLUID-1963 and commented on the JIRA. It looks good to me, I think she should commit it.
[13:00:53 EST(-0500)] <colinclark> well, we are all standing up
[13:02:10 EST(-0500)] <ecochran> colinclark: my mind boggles
[13:15:29 EST(-0500)] * fj4000 (n=Main@CPE00095bc35ea1-CM001692f5798c.cpe.net.cable.rogers.com) has joined #fluid-work
[13:18:54 EST(-0500)] <ecochran> fj4000: so maybe I don't need you right away... it looks like I broke a few things in the Uploader Springboard, either when I was updating the CSS (most likely) or with the latest change that I made to events (or both)
[13:19:23 EST(-0500)] <fj4000> ecochran: i'll be here....just ping me when youre ready
[13:19:30 EST(-0500)] <ecochran> I'm going to bang on it and look at code for 10 or 15 and then I'll let you know if I need your intimate knowledge of the code
[13:19:51 EST(-0500)] <fj4000> ok
[13:22:05 EST(-0500)] <Justin_o> fj4000: I'm wondering how closely tied the uploader springboard is to the component. Basically when
[13:22:22 EST(-0500)] <fj4000> it shouldnt be that tight
[13:22:42 EST(-0500)] <Justin_o> when it gets flash 10 support will you need to update it
[13:22:44 EST(-0500)] <fj4000> except the springboard needs to know there is no more uploader.css file
[13:24:07 EST(-0500)] <ecochran> fj4000: the Uploader spring board appears to not use the old uploader.css (now fluid.components.uploader.csss
[13:25:07 EST(-0500)] <fj4000> so there is no css dependency, yet it still broke?
[13:25:53 EST(-0500)] <ecochran> the dependency is in the selectors
[13:25:59 EST(-0500)] <fj4000> ok
[13:26:15 EST(-0500)] <fj4000> so when your done your review, we can see what works and what doesnt
[13:26:25 EST(-0500)] <ecochran> so I updated the selectors in the Springboard to match the selectors in fluid.components.uploader.css and now the code
[13:26:49 EST(-0500)] <ecochran> problem was that you already had a fl-uploader-queue class
[13:26:53 EST(-0500)] <ecochran> so now there are two
[13:27:04 EST(-0500)] <ecochran> but the Remove buttons also stopped working
[13:27:14 EST(-0500)] <ecochran> which is what I'm chewing on now
[13:29:36 EST(-0500)] <ecochran> fj4000: by the way, this was not your problem... this was my sloppy search and replace!
[13:29:40 EST(-0500)] <ecochran> my bad
[13:30:15 EST(-0500)] <ecochran> cut and paste, and search and replace, the power tools that will cut your hand off!
[13:30:21 EST(-0500)] <fj4000> (tongue)
[13:31:31 EST(-0500)] <Bosmon> I prefer to think of them more as tinker toys (tongue)
[13:31:50 EST(-0500)] <ecochran> that will poke your eye out? :-P
[13:31:57 EST(-0500)] <Bosmon> yes
[13:32:06 EST(-0500)] <Bosmon> The Tinker Toys that Kill (tongue)
[13:33:57 EST(-0500)] <ecochran> fj4000: This is a really interesting example of why we might want more than one Fluid CSS namespace... one for the components and one for the skin.
[13:34:34 EST(-0500)] <ecochran> As I go through the CSS, it's hard for me to parse which classes are specific to the component and which is skinning stuff
[13:34:48 EST(-0500)] <fj4000> they should be in separate files
[13:35:01 EST(-0500)] <ecochran> it's when I look at them in mark up
[13:35:31 EST(-0500)] <ecochran> when I scan the markup, it'd be nice to know at a glance
[13:35:36 EST(-0500)] <fj4000> fl-uploader is usually the prefix for the uploader specific stuff
[13:36:03 EST(-0500)] <ecochran> yes, it's just a little more parsing ...
[13:36:04 EST(-0500)] <fj4000> "fl-component-whatItIs" is typically the setup
[13:36:16 EST(-0500)] <fj4000> yes
[13:36:40 EST(-0500)] <fj4000> something simpler would be very cool
[13:37:56 EST(-0500)] <ecochran> fj4000: and I said "might", I'm not wedded to the idea... just a point for discussion
[13:41:01 EST(-0500)] <Bosmon> FEEES has shown me a worrying integration issue with Reorderer
[13:41:07 EST(-0500)] <Bosmon> That I fear is too late to turn into a blocker
[13:41:15 EST(-0500)] <Bosmon> But I think can only really be addressed by fixing the base component
[13:41:23 EST(-0500)] <Bosmon> She is suggesting I turn the fix into a patch for now
[13:41:53 EST(-0500)] <Bosmon> But again, I wonder if this takes priority for anything else I might do for Parade/Freeze
[13:45:13 EST(-0500)] <anastasiac> Justin_o, fj4000, I've reviewed the new patch for FLUID-1966 and commented on the JIRA
[13:45:17 EST(-0500)] <anastasiac> it breaks in Opera
[13:45:28 EST(-0500)] <fj4000> I was just going to ask you
[13:45:39 EST(-0500)] <anastasiac> and I'm not sure about modifying one of the files that come with the jquery plug-in, which is not our code
[13:45:42 EST(-0500)] <fj4000> it seems to work in mine...i dont get it
[13:45:54 EST(-0500)] <fj4000> thats where the problem was
[13:45:54 EST(-0500)] <anastasiac> it breaks in Windows & Mac
[13:46:03 EST(-0500)] <anastasiac> hm..
[13:46:11 EST(-0500)] <fj4000> strange
[13:46:13 EST(-0500)] <anastasiac> so you're saying "it's not our bug" ?
[13:46:16 EST(-0500)] <fj4000> it looks good here
[13:46:29 EST(-0500)] <anastasiac> what version of Opera are you using?
[13:46:30 EST(-0500)] <fj4000> absolutely has nothing to do with our CSS or JS code
[13:46:37 EST(-0500)] <Bosmon> What is going on?
[13:46:43 EST(-0500)] <fj4000> its how their selectbox css works
[13:46:57 EST(-0500)] <Bosmon> No, you are absolutely authorized to change that
[13:47:00 EST(-0500)] <fj4000> Bosmon: just discussing 1966
[13:47:01 EST(-0500)] <Bosmon> It is just some crappy plugin
[13:47:08 EST(-0500)] <fj4000> i was just saying
[13:47:12 EST(-0500)] <anastasiac> fj400, what version of Opera are you testing with?
[13:47:13 EST(-0500)] <fj4000> how I was working on their css
[13:47:18 EST(-0500)] <anastasiac> fj4000^
[13:47:24 EST(-0500)] <fj4000> 9.9.62
[13:47:28 EST(-0500)] <fj4000> sorry
[13:47:29 EST(-0500)] <Bosmon> I just picked it because it was the "most suitable" thing I could lay my hands on
[13:47:29 EST(-0500)] <fj4000> 9.62
[13:47:39 EST(-0500)] <Bosmon> If we can fix it, there's nothing that stops us "keeping" it
[13:47:55 EST(-0500)] <Bosmon> If we decide the most efficient thing to do is to throw it away and start from scratch, we can do that too
[13:48:00 EST(-0500)] <fj4000> i was saying, its working on my local copy in all browsers
[13:48:07 EST(-0500)] <Bosmon> Although the most useful thing would be to be able to push a patch batck them
[13:48:14 EST(-0500)] <Bosmon> fj4000: Is it in trunk? Can I test?
[13:48:20 EST(-0500)] <anastasiac> fj4000, I'm at 9.52. maybe I should upgrade
[13:48:23 EST(-0500)] <fj4000> no, all I have is a patch
[13:48:25 EST(-0500)] <fj4000> but
[13:48:25 EST(-0500)] <Bosmon> OK
[13:48:31 EST(-0500)] <Bosmon> 1966-b patch?
[13:48:32 EST(-0500)] <fj4000> im thinking maybe I made a bad patch
[13:48:33 EST(-0500)] <fj4000> yes
[13:48:44 EST(-0500)] <Bosmon> It's certainly a lot smaller than your other one
[13:48:45 EST(-0500)] <anastasiac> fj4000, let me upgrade Opera first
[13:48:49 EST(-0500)] <fj4000> it could have gotten all wonky
[13:48:50 EST(-0500)] <Bosmon> Weren't you saying that this required a listener in any case?
[13:48:51 EST(-0500)] <fj4000> (sad)
[13:48:53 EST(-0500)] <Bosmon> cabberwonky?
[13:48:57 EST(-0500)] <fj4000> Bosmon
[13:48:58 EST(-0500)] <fj4000> yes
[13:49:08 EST(-0500)] <fj4000> but I figured out a css-only solution
[13:49:15 EST(-0500)] <fj4000> i was making it more complicated than it was
[13:49:18 EST(-0500)] <Bosmon> cool
[13:49:20 EST(-0500)] <Bosmon> Let me test it, then
[13:49:46 EST(-0500)] <fj4000> in the end, only about 5 lines needed to be changed in the css file
[13:49:51 EST(-0500)] <Bosmon> Sounds great
[13:50:01 EST(-0500)] <fj4000> so thats all you should get, plus i removed some <br /> tags
[13:50:04 EST(-0500)] <fj4000> just for effect
[13:50:05 EST(-0500)] <Bosmon> Yes
[13:50:07 EST(-0500)] <Bosmon> Why did you do that? (tongue)
[13:50:13 EST(-0500)] <fj4000> so I could test overlap
[13:50:16 EST(-0500)] <fj4000> behaviour
[13:50:21 EST(-0500)] <Bosmon> ...
[13:50:33 EST(-0500)] <Bosmon> Oh, you mean the general document layout displacement?
[13:50:36 EST(-0500)] <fj4000> yes
[13:50:45 EST(-0500)] <Bosmon> Rather than the original bug, that is (tongue)
[13:50:55 EST(-0500)] <fj4000> i guess both at the same time
[13:52:07 EST(-0500)] <Bosmon> Although the "concept" of this patch is not "self-contained"....
[13:52:11 EST(-0500)] <Bosmon> But let me test it in any case (tongue)
[13:52:15 EST(-0500)] <Justin_o> Bosmon: sorry, i've been trying to catch up on the irc channel... a lot going on here... michelled showed me the issue and i think making a patch and giving that to gonzalo is a good start. I'm thinking that with the day about half over it may and the fact that we should have already gone into code freeze, we probably won't have time to include the fix in this release
[13:52:17 EST(-0500)] <anastasiac> fj4000, that bug is still happening for me in opera
[13:52:30 EST(-0500)] <Bosmon> .....
[13:52:31 EST(-0500)] <Bosmon> It is terrible
[13:52:34 EST(-0500)] <fj4000> anastasiac: can you elaborate what files got patched?
[13:52:37 EST(-0500)] <Bosmon> The control jumps to the top left of the window
[13:52:38 EST(-0500)] <fj4000> i think its a bad patch
[13:52:40 EST(-0500)] <Bosmon> Only selectbox.css got patched
[13:52:41 EST(-0500)] <fj4000> yes
[13:52:45 EST(-0500)] <fj4000> thats what shes saying too
[13:52:49 EST(-0500)] <Bosmon> You can see this by looking inside the patch file
[13:52:53 EST(-0500)] <Bosmon> HOw many should be patched?
[13:53:03 EST(-0500)] <fj4000> i dont get that problem
[13:53:20 EST(-0500)] <fj4000> let me see this thing.....
[13:53:24 EST(-0500)] <Bosmon> All that is in the patch is essentially this:
[13:53:25 EST(-0500)]

<Bosmon> #combo-editable

Unknown macro: {position}

[13:53:25 EST(-0500)]

<Bosmon> .editContainer

Unknown macro: {position}

[13:53:25 EST(-0500)] <Bosmon> .text {}
[13:53:26 EST(-0500)] * fj4000 is patch hunting
[13:53:35 EST(-0500)] <fj4000> well, thats not right!
[13:53:46 EST(-0500)] <fj4000> there should be 5 lines, not 3
[13:53:55 EST(-0500)] <fj4000> argh
[13:54:03 EST(-0500)] <fj4000> pls give me a moment
[13:54:24 EST(-0500)] <anastasiac> ok, regarding the selectbox bug: I can see the bug in the sample provided by the author, so - confirmed as pre-existing
[13:54:55 EST(-0500)] <fj4000> can i send the contents of the patch to you guys privately, and you can tell me if thats what you have?
[13:55:04 EST(-0500)] <fj4000> something didnt work right
[13:55:28 EST(-0500)] <anastasiac> fj4000, try creating the patch from a higher up folder, like the root folder of the project
[13:55:33 EST(-0500)] <fj4000> ok
[13:57:18 EST(-0500)] <Bosmon> Well, there are a couple more lines yes
[13:57:30 EST(-0500)] <fj4000> ok, Im sending you my latest patch
[13:57:32 EST(-0500)] <Bosmon> You have commented out margin-top
[13:57:36 EST(-0500)] <fj4000> yes
[13:58:00 EST(-0500)] <Bosmon> But there's a problem in that this file now depends on .editContainer
[13:58:05 EST(-0500)] <Bosmon> I am assuming that was just experimental (tongue)
[13:58:22 EST(-0500)] <fj4000> experimental?
[13:58:29 EST(-0500)] <Bosmon> I mean, you weren't intending to leave it that way
[13:58:56 EST(-0500)] <fj4000> I just took a snapshot of what was in the markup, and used the classnames there
[14:00:07 EST(-0500)] <anastasiac> ah! now the dropdown looks fine in Opera
[14:00:19 EST(-0500)] <anastasiac> ah, nice. much better
[14:00:28 EST(-0500)] <fj4000> there you go
[14:00:30 EST(-0500)] <Bosmon> OK, this patch is the same, only it has "top 1.5em" in as well
[14:00:32 EST(-0500)] <fj4000> bad patch
[14:00:54 EST(-0500)] <fj4000> im going to add a comment to the jira
[14:00:58 EST(-0500)] <Bosmon> wait!
[14:01:15 EST(-0500)] <Bosmon> The point is, it seems to be the removal of the brs that causes this to work
[14:01:20 EST(-0500)] <Bosmon> Which means that it is not a fix
[14:01:31 EST(-0500)] * colinclark (n=colin@142.150.154.101) has joined #fluid-work
[14:01:31 EST(-0500)] <Bosmon> I applied just the portion of the patch which relates to selectbox.css
[14:01:33 EST(-0500)] <Bosmon> And it still jumps
[14:01:48 EST(-0500)] <anastasiac> yes, Bosmon's right
[14:02:00 EST(-0500)] <anastasiac> also, he's going a point about putting the extra css in the selectbox css file
[14:02:16 EST(-0500)] <Bosmon> OK, we will really need to put our heads together on this one (tongue)
[14:02:17 EST(-0500)] <anastasiac> it's one thing to fix their bug, but "our" css shouldn't be added to that file
[14:02:52 EST(-0500)] <Bosmon> Still, the upside is that the bug is at least fixed on FF2 (tongue)
[14:03:08 EST(-0500)] <Bosmon> The component looks really nice
[14:03:19 EST(-0500)] <fj4000> im really not sure what you guys are seeing, im sending you a screenshot of my opera screen (tongue)
[14:03:41 EST(-0500)] <Bosmon> I am on Opera 9.6.2
[14:03:45 EST(-0500)] <Bosmon> 9.62
[14:05:34 EST(-0500)] <anastasiac> fj4000, any update on FLUID-1758?
[14:05:49 EST(-0500)] <fj4000> im afraid to patch it now (tongue)
[14:05:50 EST(-0500)] <fj4000> but yes
[14:05:54 EST(-0500)] <fj4000> i've got a fix
[14:06:03 EST(-0500)] <anastasiac> post the patch, I'll try it out
[14:06:09 EST(-0500)] <fj4000> before I make a patch again, I would like to know whats going wriong with this one
[14:06:33 EST(-0500)] <Bosmon> Well, is there anything wrong with the patch?
[14:06:41 EST(-0500)] <Bosmon> I think the patch contains just what you meant to put in it
[14:06:44 EST(-0500)] <fj4000> anastasiac, Bosmon did you get my email?
[14:06:54 EST(-0500)] <anastasiac> fj4000, checking...
[14:06:57 EST(-0500)] <Bosmon> not yet
[14:07:19 EST(-0500)] <fj4000> Bosmon: I could have done it from the wrong directory, messed it up any # of ways...who knows
[14:07:44 EST(-0500)] <anastasiac> fj4000, I notice that you're essentially serving the demo up
[14:07:53 EST(-0500)] <anastasiac> can you try it with a file url?
[14:08:15 EST(-0500)] <Bosmon> fj4000 why don't you just "pastie" for now
[14:08:19 EST(-0500)] <Bosmon> So we can rule that out as a source of error
[14:08:21 EST(-0500)] <fj4000> ok
[14:08:26 EST(-0500)] <Bosmon> Just pastie your entire copy of the CSS file
[14:08:33 EST(-0500)] <fj4000> yes
[14:08:41 EST(-0500)] <Bosmon> I still haven't got your mail
[14:09:07 EST(-0500)] <anastasiac> fj4000, use fluid.pastebin.com
[14:09:16 EST(-0500)] <fj4000> it said you email address had a permanent error
[14:09:20 EST(-0500)] <fj4000> (sad)
[14:09:25 EST(-0500)] <Bosmon> what on earth
[14:09:52 EST(-0500)] <Bosmon> I just got an email from SVN 6 minute ago...
[14:09:53 EST(-0500)] <anastasiac> it's just a screen shot
[14:10:02 EST(-0500)] <anastasiac> it shows that it works for fj4000
[14:10:04 EST(-0500)] <fj4000> http://fluid.pastebin.com/m3f239aa6
[14:10:45 EST(-0500)] <fj4000> ALL i changed was line 3, 8, 56-58
[14:11:10 EST(-0500)] <fj4000> im pretty sure thats it
[14:11:48 EST(-0500)] <anastasiac> fj4000: are you testing using a server?
[14:12:00 EST(-0500)] <fj4000> local server, yes
[14:12:09 EST(-0500)] <anastasiac> can you try it using a regular file url?
[14:12:22 EST(-0500)] <anastasiac> i.e. just open file, and browse on your hard drive to the test file
[14:12:26 EST(-0500)] <fj4000> wont I have to commit for that to work?
[14:12:31 EST(-0500)] <anastasiac> no!
[14:12:38 EST(-0500)] <anastasiac> it's on your hard drive, right?
[14:12:43 EST(-0500)] <Bosmon> fj4000: Your file is identical to mine
[14:12:50 EST(-0500)] <fj4000> very weird
[14:13:00 EST(-0500)] <fj4000> let me try anastasiac
[14:13:04 EST(-0500)] <Bosmon> But I guess you are testing in Opera Mac, right?
[14:13:15 EST(-0500)] <anastasiac> both windows and mac
[14:13:28 EST(-0500)] <anastasiac> Bosmon, do you see the error in Opera?
[14:14:08 EST(-0500)] <fj4000> Ok, I just dragged "dropdown.html" into Opera
[14:14:12 EST(-0500)] <fj4000> and it looks normal
[14:14:18 EST(-0500)] <Bosmon> It does not jump?
[14:14:20 EST(-0500)] <Bosmon> When activated?
[14:14:20 EST(-0500)] <fj4000> no
[14:14:24 EST(-0500)] <Bosmon> GLARG
[14:14:28 EST(-0500)] <anastasiac> indeed
[14:14:29 EST(-0500)] <Bosmon> This is Bedlam
[14:14:29 EST(-0500)] <fj4000> wait
[14:14:33 EST(-0500)] <fj4000> when you say jump
[14:14:41 EST(-0500)] <fj4000> what EXACTLY do you mean?
[14:14:42 EST(-0500)] <Bosmon> The entire control appears at the top left point of the browser
[14:14:43 EST(-0500)] <fj4000> which parts
[14:14:45 EST(-0500)] <fj4000> and where
[14:14:46 EST(-0500)] <Bosmon> Rather than at the point of activation
[14:14:51 EST(-0500)] <Bosmon> Well, actually all I see is the dropdown portion
[14:14:57 EST(-0500)] <fj4000> no, I only get what I sent anastasia
[14:15:11 EST(-0500)] <Bosmon> The section which is meant to be the "dropdown selector" appears to be completely missing
[14:15:12 EST(-0500)] <fj4000> this is very weird
[14:15:16 EST(-0500)] <Bosmon> Perhaps it is above the viewport
[14:15:31 EST(-0500)] <Bosmon> Which Opera version do you have there?
[14:15:36 EST(-0500)] <fj4000> 9.62
[14:15:40 EST(-0500)] <anastasiac> hm. hard for fj4000 to try to fix a bug that he can't reproduce...
[14:15:46 EST(-0500)] <fj4000> yup ")
[14:16:01 EST(-0500)] <fj4000> Im very sure something obvious is missing here
[14:16:10 EST(-0500)] <fj4000> like an update wasnt made by me or something
[14:16:13 EST(-0500)] <fj4000> give me a moment
[14:16:19 EST(-0500)] <Bosmon> Well, why don't you go over to AC's machine
[14:16:25 EST(-0500)] <Bosmon> Isn't she like 3 feet away from you? (tongue)
[14:16:26 EST(-0500)] <fj4000> nope
[14:16:30 EST(-0500)] <ecochran> Justin_o fj4000 : I've tracked down the problems with the Uploader springboard. They are two fold. The first problem was a search and replace bug that I introduced with the recent CSS refactoring. It was just as Colin said, we ended up with two fl-uploader-queue classes which confused the UploaderFileView code. The fix was to remove the extra class (in the right place) and move the styling that was hanging off of it to anothe
[14:16:31 EST(-0500)] <ecochran> The second problem was introduced in an older refactoring when I changed the name of the Remove button class. This problem was a little more complex to fix, but not a biggie. I renamed the class in the Springboard in one case, and in another I overroad the class in the initter... which is a good test anyway.
[14:16:31 EST(-0500)] <fj4000> 3000
[14:16:36 EST(-0500)] <Bosmon> ....
[14:16:40 EST(-0500)] <fj4000> im at home
[14:16:43 EST(-0500)] <fj4000> shes at u of t
[14:16:47 EST(-0500)] <Bosmon> mega-glarg
[14:16:49 EST(-0500)] <fj4000> roughly 15 km away
[14:16:54 EST(-0500)] <fj4000> (tongue)
[14:17:00 EST(-0500)] <anastasiac> ok, hold on here.
[14:17:04 EST(-0500)] <Bosmon> More like 30,000 feet
[14:17:09 EST(-0500)] <fj4000> can we skype this?
[14:17:10 EST(-0500)] <Bosmon> Only she does not have a Plane
[14:17:12 EST(-0500)] <Bosmon> ok
[14:17:18 EST(-0500)] <anastasiac> yes, let's Skype
[14:17:20 EST(-0500)] <Bosmon> Don't we have an Everett Skype coming up soon anyway?
[14:17:22 EST(-0500)] * anastasiac goes online
[14:17:34 EST(-0500)] * fj4000 does too
[14:17:37 EST(-0500)] <ecochran> Justin_o fj4000 : I would like to reopen FLUID-1935, and submit a patch for these fixes for fj4000 to review
[14:18:04 EST(-0500)] <Justin_o> ecochran: sounds like a good idea
[14:18:23 EST(-0500)] <ecochran> Justin_o: then I have your blessing! Thanks o'King!
[14:18:27 EST(-0500)] <anastasiac> Bosmon, fj4000: I'm on Skype - a.cheetham
[14:18:32 EST(-0500)] <anastasiac> what are your contacts?
[14:18:36 EST(-0500)] <fj4000> Bosmon anastasiac; ok, I was just going to ask
[14:19:11 EST(-0500)] <Bosmon> Sorry, I just missed your call
[14:19:14 EST(-0500)] <Bosmon> Didn't have my headphones in yet
[14:19:29 EST(-0500)] <fj4000> jacob.farber for me
[14:19:37 EST(-0500)] <fj4000> i cant see either of you
[14:20:07 EST(-0500)] <anastasiac> fj4000, got your call, heard you
[14:20:12 EST(-0500)] <anastasiac> I'm guessing you didn't hear me
[14:20:15 EST(-0500)] <fj4000> nope.
[14:20:19 EST(-0500)] <fj4000> said you were offline
[14:20:20 EST(-0500)] <anastasiac> Bosmon, what is your id?
[14:20:24 EST(-0500)] <fj4000> Ah
[14:27:45 EST(-0500)] <ecochran> to anyone who cares... on the Mac, do not do a file replace when creating a patch in Aptana... Aptana blows up! Grumph!
[14:28:08 EST(-0500)] <Justin_o> ecochran: i was testing your fix for 1947
[14:28:28 EST(-0500)] <Justin_o> i notice that if you stop on the same file, multiple times... it won't finish uploading
[14:28:47 EST(-0500)] <ecochran> how fast did you click the button?
[14:29:21 EST(-0500)] <fj4000> anastasiac, Bosmon: all I have to say is thank goodness for Skype
[14:29:24 EST(-0500)] <ecochran> Justin_o: I'll take a look as soon as I build this patch
[14:29:31 EST(-0500)] <Justin_o> ecochran: depended on how big the file was
[14:29:42 EST(-0500)] <Justin_o> ecochran: it was on the server version
[14:29:53 EST(-0500)] <ecochran> really!?
[14:29:55 EST(-0500)] <ecochran> ah
[14:29:56 EST(-0500)] <anastasiac> fj4000: (smile)
[14:29:57 EST(-0500)] <ecochran> good
[14:30:06 EST(-0500)] <Justin_o> ecochran: you know what it is?
[14:30:06 EST(-0500)] <ecochran> Justin_o: let me look
[14:30:10 EST(-0500)] <Justin_o> ecochran: thanks
[14:30:18 EST(-0500)] <ecochran> Justin_o: no, but that gives me more to work with
[14:30:30 EST(-0500)] <Justin_o> ecochran: oh okay...
[14:30:33 EST(-0500)] <ecochran> Justin_o: the server and demo are such different beasts
[14:31:00 EST(-0500)] <ecochran> they share much code and behavior but they also diverge in very specific ways
[14:31:06 EST(-0500)] <ecochran> and Uploading is the heart of it
[14:31:32 EST(-0500)] <Justin_o> because the demo is just a simulation
[14:33:12 EST(-0500)] <ecochran> fj4000: http://issues.fluidproject.org/secure/attachment/10373/FLUID-1935.b.patch
[14:37:29 EST(-0500)] <michelled> Justin_o: what's next for me?
[14:38:04 EST(-0500)] <ecochran> brb
[14:38:07 EST(-0500)] <Justin_o> sending out an update of the bug parade right now
[14:38:08 EST(-0500)] <fj4000> ecochran; got it, thanks. I will apply it when im done with 1758
[14:38:12 EST(-0500)] <anastasiac> fj4000, your second patch for 1758: is that to be applied on top of the first? or instead of the first?
[14:38:19 EST(-0500)] <fj4000> instead of
[14:38:26 EST(-0500)] <anastasiac> k
[14:39:18 EST(-0500)] <fj4000> damn, actually I think not
[14:39:41 EST(-0500)] <anastasiac> I think not, also (wink)
[14:39:51 EST(-0500)] <fj4000> your right
[14:39:54 EST(-0500)] <fj4000> on top of
[14:42:44 EST(-0500)] <anastasiac> fj4000, the second patch won't work on top of the first.
[14:42:53 EST(-0500)] <fj4000> what happens?
[14:42:58 EST(-0500)] <anastasiac> it just doesn't work
[14:43:03 EST(-0500)] <fj4000> i need to merge the two patches?
[14:43:04 EST(-0500)] <anastasiac> can you generate a single patch?
[14:43:06 EST(-0500)] <fj4000> yes
[14:43:09 EST(-0500)] <anastasiac> don't merge, just re-generate
[14:43:13 EST(-0500)] <fj4000> ok
[14:45:02 EST(-0500)] <fj4000> anastasiac pls see the jira for 3rd patch
[14:45:09 EST(-0500)] <anastasiac> k
[14:49:08 EST(-0500)] <ecochran> back
[14:49:31 EST(-0500)] <fj4000> anastasiac: any luck?
[14:49:42 EST(-0500)] <anastasiac> fj4000: the patch works, and the drop-markers are all there
[14:49:43 EST(-0500)] <anastasiac> but:
[14:49:46 EST(-0500)] <Bosmon> Shall I look at 1758 too?
[14:49:58 EST(-0500)] <anastasiac> the drop markers for the tabs seem smaller than they were in your first patch
[14:50:06 EST(-0500)] <anastasiac> basically a bit too small for my taste
[14:50:17 EST(-0500)] <fj4000> themed tabs or normal tabs or both?
[14:50:51 EST(-0500)] <anastasiac> themes
[14:50:56 EST(-0500)] <anastasiac> forgot to test normal :-/
[14:51:42 EST(-0500)] <anastasiac> in the normal ones, they seem nice
[14:51:58 EST(-0500)] <anastasiac> just the themed ones look small enough that I'm not sure they're there
[14:52:14 EST(-0500)] <fj4000> ok
[14:52:21 EST(-0500)] <fj4000> heres what I suggest
[14:52:21 EST(-0500)] <anastasiac> they're not tall enought
[14:52:23 EST(-0500)] <anastasiac> that's the problem
[14:52:36 EST(-0500)] <fj4000> can we commit what we've got, to avoid any more patching problems
[14:52:44 EST(-0500)] <fj4000> and then tweak the height
[14:53:19 EST(-0500)] <anastasiac> the patching isn't really a problem, as long as you generate full replacement patches, and not patches on patches (smile)
[14:54:05 EST(-0500)] <Justin_o> ecochran: did you get a chance to look at the issue i mentioned to you
[14:54:10 EST(-0500)] <anastasiac> think of it as practice making patches (tongue)
[14:54:40 EST(-0500)] <ecochran> Justin_o: not yet... will look now. It's running on build? or do I need to fire up my own server right now?
[14:55:01 EST(-0500)] <Justin_o> you can use the online one
[14:55:05 EST(-0500)] <Justin_o> with build
[14:57:11 EST(-0500)] <Justin_o> ecochran: it also looks like the "Add More" button isn't disabled during upload
[14:57:28 EST(-0500)] <ecochran> Justin_o: grrrrr!
[14:57:35 EST(-0500)] <ecochran> Justin_o: not at you
[14:57:39 EST(-0500)] <ecochran> just grrrr
[14:57:49 EST(-0500)] * phiggins (n=dante@c-68-34-199-67.hsd1.tn.comcast.net) has joined #fluid-work
[14:58:26 EST(-0500)] <Justin_o> ecochran: i undestand....
[14:58:35 EST(-0500)] <ecochran> Justin_o: I'm pretty sure that I know the first problem. And there is a very simple test for it.
[15:00:37 EST(-0500)] <michelled> Justin_o: 1970 is reviewed and I commented on the JIRA.
[15:00:38 EST(-0500)] <ecochran> We still have this problem that if you pause the upload right near the end... there is the possibility that the server will report that the file successfully uploaded after the Upload was canceled and the message gets lost. Interestingly, this problem is not all that different than the problem that we have in the demo. (Up until this moment I assumed that I was going to have to fix this problem on the server-side but now I thi
[15:01:36 EST(-0500)] <ecochran> Anyway, the test is to try the upload again... and if the file errors on completion, it is most likely that this is the same bug as before (I'll look it up for you.)
[15:01:41 EST(-0500)] <michelled> ecochran, you got cut off there. The last thing we saw was: "but now I thi"
[15:02:04 EST(-0500)] <ecochran> OK, looks like IRC doesn't like it when you are long winded
[15:02:17 EST(-0500)] <ecochran> (Up until this moment I assumed that I was going to have to fix this problem on the server-side but now I think that I might have another fix, but I digress).
[15:02:18 EST(-0500)] <Bosmon> Yeah, it is designed for small kids
[15:02:22 EST(-0500)] <Bosmon> Who like to flood each other (tongue)
[15:03:15 EST(-0500)] <Justin_o> flood each other?
[15:03:30 EST(-0500)] <Bosmon> Kids will attempt to generate "message floods"
[15:03:38 EST(-0500)] <ecochran> The reason why the file errors the second time is that it's already on the server and Ray's code throws an error if you try to upload two files with the same binary.
[15:03:44 EST(-0500)] <ecochran> great for testing error cases
[15:03:52 EST(-0500)] <Bosmon> In order, to make each others comments to scroll away as fast as possible (tongue)
[15:04:37 EST(-0500)] <ecochran> Justin_o: ^did you get all that?
[15:04:47 EST(-0500)] <ecochran> now on to the button
[15:05:47 EST(-0500)] <ecochran> Bosmon flooded me!
[15:05:49 EST(-0500)] <ecochran> wah!
[15:06:28 EST(-0500)] <Justin_o> Bosmon: that was a good demonstration of your explanation
[15:06:45 EST(-0500)] <Justin_o> ecochran: i think i got that... so basically it is a known issue
[15:06:54 EST(-0500)] <ecochran> Justin_o: FLUID-822
[15:06:57 EST(-0500)] <ecochran> found it
[15:07:18 EST(-0500)] <Bosmon> (smile)
[15:08:16 EST(-0500)] <Justin_o> ecochran: oh yes that one...
[15:09:13 EST(-0500)] <Justin_o> michelled: thanks for the review of 1970
[15:10:09 EST(-0500)] <michelled> np - do you have another to toss to me?
[15:30:38 EST(-0500)] <ecochran> Justin_o: the Browse button issue is really a bear to fix
[15:30:57 EST(-0500)] <ecochran> the easiest fix is to make the button a real button, right now it is a link...
[15:31:05 EST(-0500)] <ecochran> actually might not be too bad
[15:31:14 EST(-0500)] <ecochran> and it would be consistent with the other buttons
[15:31:26 EST(-0500)] <ecochran> Justin_o: worth fixing?
[15:31:47 EST(-0500)] <ecochran> I'd have to do some CSS work, functionally it seems to be pretty much the same
[15:32:01 EST(-0500)] <anastasiac> fj4000, just tried your latest patch for FLUID-1758
[15:32:06 EST(-0500)] <anastasiac> it looks great
[15:32:10 EST(-0500)] <fj4000> ok, great
[15:32:11 EST(-0500)] <ecochran> might even fix that Opera Browse button bug
[15:32:20 EST(-0500)] <fj4000> so I can commit that then?
[15:32:27 EST(-0500)] <anastasiac> Justin_o, I think fj4000 should go ahead and commit that
[15:32:33 EST(-0500)] <Bosmon> Is this 1758-d?
[15:32:40 EST(-0500)] <anastasiac> yes, -d
[15:32:53 EST(-0500)] <Bosmon> Shall I try it out too, or just wait for trunk
[15:33:56 EST(-0500)] <Bosmon> michelled: Actually 1830 is in
[15:34:18 EST(-0500)] <fj4000> Justin_o: is it ok to commit 1758-d ?
[15:35:17 EST(-0500)] <michelled> Bosmon: meaning it's been done already?
[15:35:23 EST(-0500)] <anastasiac> fj4000, Justin_o is testing some uploader stuff for colinclark
[15:35:26 EST(-0500)] <Bosmon> Yes, it has been in trunk for a week or so...
[15:35:39 EST(-0500)] <michelled> oh, great - do you want to close it then?
[15:35:45 EST(-0500)] <Bosmon> ok
[15:35:48 EST(-0500)] <michelled> thanks
[15:35:54 EST(-0500)] <fj4000> ok, until he gives the go ahead I will hold off
[15:36:36 EST(-0500)] <anastasiac> he is the King...
[15:36:43 EST(-0500)] <Bosmon> Wow, that is some big patch
[15:37:34 EST(-0500)] <Bosmon> Looks more reasonable in Opera now
[15:37:47 EST(-0500)] <Bosmon> fj4000: Were you aware that the last drop target, in Opera, is twice the width of the others?
[15:38:21 EST(-0500)] <fj4000> its not the last, its all of them - but the "last" just pokes through more
[15:38:26 EST(-0500)] <Bosmon> ok
[15:38:30 EST(-0500)] <fj4000> its a fix im getting ready for
[15:38:31 EST(-0500)] <Bosmon> I still don't see any targets in FF2
[15:38:37 EST(-0500)] <fj4000> you'll notice its not in the themed tabs
[15:38:44 EST(-0500)] <fj4000> sorry
[15:38:50 EST(-0500)] <fj4000> its ONLY in the themed tabs
[15:38:59 EST(-0500)] <Bosmon> The extra width?
[15:38:59 EST(-0500)] <Bosmon> Yes
[15:39:14 EST(-0500)] <Bosmon> The unthemed ones look good in every browser
[15:39:21 EST(-0500)] <Bosmon> The themed ones still have no drop targets in FF2
[15:39:25 EST(-0500)] <fj4000> yeah, FF2 is a bit of a problem
[15:39:57 EST(-0500)] <anastasiac> shoot - fj4000, Bosmon's right
[15:39:57 EST(-0500)] <fj4000> yes
[15:39:57 EST(-0500)] <anastasiac> FF2 is bad on the mac too
[15:39:57 EST(-0500)] <fj4000> I noticed too
[15:39:57 EST(-0500)] <ecochran> Poor Justin_o has a lot of catching up to do
[15:39:57 EST(-0500)] <Justin_o> fj4000: if your patch is ready, you can commit
[15:40:01 EST(-0500)] <michelled> ecochran: I'm hearing that you may have a patch that allows me to load the Uploader springboard without getting an error
[15:40:06 EST(-0500)] <fj4000> it should be a separate ticket though, to refactor the tabbing helper to avoid the problems as of late
[15:40:19 EST(-0500)] <Justin_o> ecochran: sorry for the delay, yes i think the button needs to be fixed
[15:40:23 EST(-0500)] <ecochran> michelled: I do
[15:40:34 EST(-0500)] <ecochran> waiting for fj4000 to review
[15:40:39 EST(-0500)] <ecochran> would you like to do it?
[15:40:43 EST(-0500)] <michelled> sure
[15:40:54 EST(-0500)] <fj4000> Justin_o: thanks, but as mentioned ^ I think im going to write another jira ticket for them
[15:40:56 EST(-0500)] <ecochran> I'll point you to the bug
[15:41:27 EST(-0500)] <Justin_o> fj4000: sorry i've been skimming through the channel... can you give me a quick summary
[15:41:33 EST(-0500)] <ecochran> michelled: http://issues.fluidproject.org/browse/FLUID-1935
[15:41:37 EST(-0500)] <ecochran> it's the "b" patch
[15:41:55 EST(-0500)] <ecochran> the other two are committed
[15:41:59 EST(-0500)] <fj4000> Justin_o: 1758 works in all but FF2
[15:42:12 EST(-0500)] <ecochran> Justin_o: OK, I'm working on it now.
[15:42:13 EST(-0500)] <fj4000> and I noticed there are other problems with the way the tabs are made
[15:42:17 EST(-0500)] <michelled> thanks ecochran
[15:42:22 EST(-0500)] <fj4000> so I need to refactor them
[15:42:29 EST(-0500)] <Justin_o> ecochran: i'll write you up a bug for it
[15:42:40 EST(-0500)] <ecochran> Justin_o: cool
[15:43:33 EST(-0500)] <fj4000> Bosmon, anastasiac: did we agree not to patch other people's css for fluid-1966?
[15:44:37 EST(-0500)] <Justin_o> fj4000: you should probably committ what you have so far and then file a patch for the new jira you create
[15:44:46 EST(-0500)] <Justin_o> i guess that's what your idea was though, eh
[15:44:59 EST(-0500)] <fj4000> yes, but thanks for the confirmation
[15:45:09 EST(-0500)] <Justin_o> (smile)
[15:45:57 EST(-0500)] <anastasiac> fj4000, Bosmon: We should patch the selectbox css with the fix, but anything that references our classes (i.e. the combo-editable and edit-container stuff) should not be added to the file
[15:46:36 EST(-0500)] <anastasiac> we should probably add a patch file as well, showing our patch to fix the bug, in case we upgrade to any upgraded version and need to re-apply the patch
[15:46:37 EST(-0500)] <fj4000> ok, then I need to talk to Bosmon about whats ours and whats theirs
[15:46:50 EST(-0500)] <anastasiac> the selectbox.css is theirs
[15:47:09 EST(-0500)] <anastasiac> anything inside that folder is theirs
[15:47:25 EST(-0500)] <fj4000> right, but there are some classes that just "appear"
[15:47:27 EST(-0500)] <fj4000> on load
[15:47:31 EST(-0500)] <fj4000> which arent anywhere
[15:47:39 EST(-0500)] <anastasiac> ?
[15:48:25 EST(-0500)] <fj4000> .editContainer is not in the markup
[15:48:29 EST(-0500)] <anastasiac> fj4000, do you want
[15:48:30 EST(-0500)] <fj4000> yet its there on load
[15:48:53 EST(-0500)] <anastasiac> editContainer is in the markup
[15:49:04 EST(-0500)] <anastasiac> dropdown.html, line 29
[15:54:41 EST(-0500)] <ecochran> fj4000: so FF3 will not style buttons with text-decoration: underline... fascists!
[15:55:33 EST(-0500)] <fj4000> yes, it sucks....I ran into this problem a couple weeks ago too
[15:55:53 EST(-0500)] <fj4000> wanted buttons to behave exactly like links
[15:57:50 EST(-0500)] <ecochran> my problem exactly!
[15:58:07 EST(-0500)] <ecochran> now... I'm not sure what to do
[15:58:16 EST(-0500)] <ecochran> do I make it look more button-y?
[15:58:23 EST(-0500)] <fj4000> be forwarned...there is also a width problem in IE too
[15:58:28 EST(-0500)] <fj4000> *forewarned
[15:58:34 EST(-0500)] <fj4000> but its solveable
[15:58:53 EST(-0500)] <ecochran> fj4000: tell me more...
[15:59:42 EST(-0500)] <fj4000> IE forces addition horizontal padding in relation to the width of the content inside the button....the more content, the more padding it "hardwires" into the button
[15:59:55 EST(-0500)] <fj4000> you cant turn it off with setting padding: 0
[16:00:05 EST(-0500)] <fj4000> but you can using overflow:visible
[16:00:07 EST(-0500)] <fj4000> go figure
[16:00:13 EST(-0500)] <ecochran> fj4000: I think that I can live with that
[16:00:49 EST(-0500)] <fj4000> yeah, in the end it looks normal, otherwise you have very stretched IE buttons
[16:00:53 EST(-0500)] <anastasiac> ecochran, Justin_o: Does FLUID-1935 need a review?
[16:01:28 EST(-0500)] <ecochran> anastasiac: michelled said that she'd look at it
[16:01:36 EST(-0500)] <ecochran> Justin_o: ^
[16:01:40 EST(-0500)] <anastasiac> cool, thanks
[16:01:51 EST(-0500)] * anastasiac is having trouble keeping up with the busy logs!
[16:02:13 EST(-0500)] <fj4000> ecochran: the best I could do to simulate the underline was to use border-bottom, but it wasnt the same at all
[16:03:07 EST(-0500)] <michelled> ecochran: the main change in the patch for 1935 seems to be related to fl-uploader-queue-wrapper. I guess the error I was seeing was because of the default selector for fileQueue.
[16:03:20 EST(-0500)] <michelled> yes?
[16:04:34 EST(-0500)] <ecochran> michelled: yes, kind of... the problem was that the Springboard already had a fl-uploader-queue
[16:05:07 EST(-0500)] <ecochran> so when I changed the name of another element to that class, we ended up with two and the FileQueueView code choked
[16:05:26 EST(-0500)] <michelled> yes
[16:06:52 EST(-0500)] <michelled> ok, the patch looks fine to me and the uploader works again in the springboard. One question, do we want all the style information in the html template?
[16:07:29 EST(-0500)] * colinclark (n=colin@142.150.154.101) has joined #fluid-work
[16:07:44 EST(-0500)] <colinclark> So it looks like Justin_o and I have narrowed in on the latest SWFUpload problem.
[16:07:48 EST(-0500)] <colinclark> Though I can't quite explain it.
[16:08:00 EST(-0500)] <colinclark> We've found two issues that appear to ultimately be interrelated.
[16:08:23 EST(-0500)] <colinclark> 1. In FF3, the Upload simply won't start. An error is thrown inside the JS-ActionScript bridge.
[16:08:49 EST(-0500)] <colinclark> 2. In some browsers (FF3, IE, Opera), you can't remove files. Another error is thrown inside the JS-ActionScript bridge.
[16:09:01 EST(-0500)] <colinclark> Interestingly, both of these issues go away if the Flash button is visible, not transparent.
[16:09:48 EST(-0500)] <colinclark> Does this sound correct, Justin_o?
[16:11:10 EST(-0500)] <Justin_o> colinclark: Yes, this is what I'm seeing too
[16:12:09 EST(-0500)] <colinclark> If we have to ship with a visible button, we can.
[16:12:22 EST(-0500)] <colinclark> This is a very unpleasant compromise, though, for a number of reasons.
[16:15:18 EST(-0500)] <ecochran> michelled: fj4000 found another bug with my patch for 1935
[16:15:42 EST(-0500)] <fj4000> one more q for ecochran
[16:15:56 EST(-0500)] <ecochran> fj4000: michelled has a comment above that you should respond to
[16:16:04 EST(-0500)] <ecochran> fj4000: shoot
[16:16:28 EST(-0500)] <fj4000> michelled: re: styles in the template, prob. not
[16:17:19 EST(-0500)] <fj4000> ecochran: did you notice the individual green progress "bars" are sort of showing at the beginnig?
[16:17:39 EST(-0500)] <fj4000> for the second example, after you brwose for a file
[16:17:48 EST(-0500)] <fj4000> there is a little green dot at the end of the name
[16:17:48 EST(-0500)] <ecochran> fj4000: I saw that. I wondered if that was part of your design
[16:17:59 EST(-0500)] <fj4000> no, its not supposed to be there
[16:18:06 EST(-0500)] <ecochran> Ok, I'll fix that too
[16:18:13 EST(-0500)] <fj4000> ok, thank you
[16:19:07 EST(-0500)] <ecochran> fj4000 and michelled: there are no styles in the template... only in the Springboard
[16:19:07 EST(-0500)] <michelled> Is it known that the remove icons stay visible in the second example even after the upload is done?
[16:19:25 EST(-0500)] <ecochran> michelled: I thought that I fixed that
[16:19:41 EST(-0500)] <michelled> I better make sure my cache is clean
[16:19:48 EST(-0500)] <fj4000> no, i noticed that too
[16:19:59 EST(-0500)] <ecochran> michelled: bummer
[16:20:29 EST(-0500)] <ecochran> fj4000 and michelled : OK, let's throw that patch out. Back to the drawing board
[16:20:36 EST(-0500)] <ecochran> I'm getting closer though
[16:20:39 EST(-0500)] <fj4000> yknow, I wonder if we have a tally of how long each components take to make in man-hours....
[16:21:07 EST(-0500)] <fj4000> some seem to pose the greatest challenges
[16:21:09 EST(-0500)] <ecochran> part of the problem was that I wasn't very familar with how these Springboards worked before I started hacking the code. '
[16:22:18 EST(-0500)] <fj4000> ecochran: i dont mean anything in particular, its just with swfupload and all the work going into some components, its like others must feel..a little left out (tongue)
[16:22:45 EST(-0500)] <fj4000> we should refactor all the little guys just to make them feel special (smile)
[16:22:56 EST(-0500)] <colinclark> fj4000: Even the little guys get some attention.
[16:23:04 EST(-0500)] <colinclark> Pager is looking rather shiny, for example.
[16:23:11 EST(-0500)] <fj4000> i was just going to say.....
[16:23:25 EST(-0500)] <fj4000> Pretty little pager
[16:29:20 EST(-0500)] <anastasiac> Justin_o et al: what is left for me to work on/look at/review/whatever?
[16:29:37 EST(-0500)] <Justin_o> i'm just about to send out the latest version of the bug parade
[16:30:18 EST(-0500)] <Justin_o> i've been a little disconnected from what's been going on for the last bit, so if you could make sure it is up to date after i send it out. that would be helpful
[16:32:38 EST(-0500)] <Justin_o> anastasiac: ^ just sent it out
[16:32:52 EST(-0500)] <anastasiac> cool
[16:38:45 EST(-0500)] <ecochran> Justin_o and fj4000 : I have a fix for the Browse button being active during Upload... I have gone ahead and changed the link button to a button button.
[16:38:53 EST(-0500)] <ecochran> I need to test it on Windows
[16:39:00 EST(-0500)] <colinclark> ecochran: Ack!
[16:39:18 EST(-0500)] <colinclark> Can you make sure you test it very well against the FLUID-1768 branch?
[16:39:31 EST(-0500)] <colinclark> It "should work."
[16:39:39 EST(-0500)] <colinclark> But you know how it goes with SWFUpload. (wink)
[16:39:53 EST(-0500)] <ecochran> ah... yes... now I'm thinking that this is a bit more risky than I thought
[16:41:11 EST(-0500)] <colinclark> In theory it shouldn't be.
[16:41:29 EST(-0500)] <colinclark> We just seem to be accumulating risk at every level with this Flash 10 nonsense.
[16:42:31 EST(-0500)] <colinclark> So try it.
[16:42:33 EST(-0500)] <colinclark> See how it goes.
[16:42:43 EST(-0500)] <colinclark> It's just that the king is nearly ready to freeze us all.
[16:42:50 EST(-0500)] <fj4000> does this mean we "must" use a button element?
[16:43:21 EST(-0500)] <colinclark> fj4000: Good question.
[16:43:23 EST(-0500)] <ecochran> fj4000: well unless we can think of a way to reliably disabled the link during upload
[16:43:48 EST(-0500)] <fj4000> could we not use CSS + JS ?
[16:43:50 EST(-0500)] <anastasiac> Justin_o, fj4000 has committed the fix for FLUID-1758, and it looks good. He's files the related issue, so I think we're ready for you to test FLUID-1758, and close it if you deem appropriate.
[16:43:57 EST(-0500)] <ecochran> the only other thing that I can think of is for the browse code to check whether we're currently uploading
[16:44:08 EST(-0500)] <colinclark> ecochran: That would be easy to do, yes.
[16:44:12 EST(-0500)] <ecochran> but I didn't want to create that dependancy
[16:44:19 EST(-0500)] <ecochran> OK, let me look at that
[16:44:40 EST(-0500)] <ecochran> we already do the cSS part very nicely
[16:44:51 EST(-0500)] <ecochran> it's the code part that is a little tricky
[16:44:55 EST(-0500)] <ecochran> but not much
[16:44:55 EST(-0500)] <fj4000> oh
[16:45:08 EST(-0500)] <ecochran> and I like it better than what I just did which looks like crap on IE
[16:45:22 EST(-0500)] <ecochran> and is going to need more work anyway
[16:45:45 EST(-0500)] <ecochran> so I'm going to through off a patch, revert and then try this new approach
[16:46:22 EST(-0500)] <ecochran> Justin_o: did you ever write a bug for this bug?
[16:46:27 EST(-0500)] <ecochran> JIRA
[16:46:38 EST(-0500)] <Justin_o> ecochran: yes... sorry i guess i forgot to tell you...
[16:46:43 EST(-0500)] <Justin_o> i think it's 1977
[16:47:01 EST(-0500)] <Justin_o> sorry 1976
[16:47:26 EST(-0500)] <ecochran> thxs
[16:47:29 EST(-0500)] <anastasiac> Justin_o, I think FLUID-1901 is good to go, and can be closed now.
[16:47:40 EST(-0500)] <ecochran> what? you busy or sometin'? :-P
[16:47:55 EST(-0500)] <anastasiac> It will need some work, for the next release, but new JIRAs should be files
[16:47:56 EST(-0500)] <anastasiac> filed
[16:55:49 EST(-0500)] <ecochran> colinclark and fj4000 : that worked much better... much smaller safer fix... thank god for that.queue.isUploading
[16:56:01 EST(-0500)] * apetro (n=apetro@wsip-98-174-242-39.ph.ph.cox.net) has joined #fluid-work
[16:56:07 EST(-0500)] <colinclark> (smile)
[16:56:17 EST(-0500)] <colinclark> ecochran: Yes. A visible model is a wonderful thing.
[16:56:34 EST(-0500)] <colinclark> If only Gyphie had thought of that. (tongue)
[16:56:54 EST(-0500)] <ecochran> colinclark: f- Gyphie!
[16:57:00 EST(-0500)] <colinclark> lol
[16:57:05 EST(-0500)] <fj4000> colinclark: one day, I need to sit down with you so you can explain how that works
[16:57:13 EST(-0500)] <fj4000> the "visible model" part
[16:57:14