fluid-work IRC Logs-2011-09-07

[08:41:48 CDT(-0500)] <heidi_> Justin_o is there a jira for the IE text size/line spacing bug?
[08:42:31 CDT(-0500)] <Justin_o> heidi_: yes.. i added it to the bug parade yesterday.. i think cindyli had filed it, but i can't remember it off the top of my head
[08:42:58 CDT(-0500)] <cindyli> heidi_: 4427
[08:43:00 CDT(-0500)] <heidi_> so it's happening in IE6 and IE7, but not IE8?
[08:43:11 CDT(-0500)] <Justin_o> heidi_: i haven't gotten to IE 8 yet
[08:43:11 CDT(-0500)] <heidi_> and we're not sure when the bug was introduced
[08:43:19 CDT(-0500)] <Justin_o> yes
[08:43:42 CDT(-0500)] <heidi_> k
[08:45:40 CDT(-0500)] <heidi_> Justin_o looks like text size works in IE8, but not line spacing
[08:51:51 CDT(-0500)] <Justin_o> hmm.. that's interesting
[08:59:44 CDT(-0500)] <huslage> the snow site is broken. i'm fixing it
[09:10:02 CDT(-0500)] <michelled> cindyli: how is the blocker you're working on coming?
[09:10:33 CDT(-0500)] <cindyli> michelled: working on. haven't found out why
[09:11:50 CDT(-0500)] <michelled> cindyli: it's 4427 right?
[09:12:06 CDT(-0500)] <cindyli> yes
[09:12:32 CDT(-0500)] <heidi_> cindyli it appears line spacing doesn't work in any of the IE's - instant or saved
[09:12:56 CDT(-0500)] <cindyli> yes, heidi_. that's the first thing i'm currently working on
[09:13:07 CDT(-0500)] <cindyli> then "text size" issue in ie6
[09:13:09 CDT(-0500)] <heidi_> cindyli any thoughts on why?
[09:14:13 CDT(-0500)] <cindyli> heidi_: just thoughts, we have special handling on ie at calculating the initial line spacing size
[09:14:15 CDT(-0500)] <heidi_> cindyli text size also an issue in IE7 too - instant or saved
[09:14:27 CDT(-0500)] <cindyli> might be something going wrong with it
[09:14:39 CDT(-0500)] <cindyli> ah, interesting
[09:15:05 CDT(-0500)] <cindyli> different ie version has different behavior.
[09:15:55 CDT(-0500)] <heidi_> michelled Justin_o while cindy's working on 4427 (last blocker?) should we start testing or? how can i help?
[09:16:11 CDT(-0500)] <Justin_o> heidi_: yes please.. i just sent off an e-mail to fluid-work about it
[09:16:28 CDT(-0500)] <heidi_> ah cool, just got it
[09:40:50 CDT(-0500)] <huslage> heidi_: i think i might shoot myself.
[09:54:10 CDT(-0500)] <heidi_> huslage don't do that!! what's up?
[09:57:13 CDT(-0500)] <heidi_> Justin_o why are we testing FF5 and not 6 ?
[09:57:35 CDT(-0500)] <Justin_o> heidi_: well.. that's a good point..
[09:58:01 CDT(-0500)] <Justin_o> when we had decided on these things, FF6 hadn't come out yet.. i think we're safe to assume people are on 6 though.. so we can switch that.. i'll make those changes
[09:58:01 CDT(-0500)] <Justin_o> {color}
[09:59:00 CDT(-0500)] <mlam1> Justin_o: I'm still working on a critical for UIO. When do you plan on freezing?
[09:59:09 CDT(-0500)] <Justin_o> fluid-everyone: heidi_ just pointed out that we had FF 5 listed for testing, but that we should probably be testing FF 6 now since it was released a bit ago. Any objections to switching?
[09:59:28 CDT(-0500)] <Justin_o> mlam1: you should probably just switch to testing now
[09:59:41 CDT(-0500)] <heidi_> Justin_o cool FF6 it is
[09:59:51 CDT(-0500)] <mlam1> oh ok
[10:07:37 CDT(-0500)] <Justin_o> heidi_: i've changed the tasks and support page to say FF 6
[10:07:46 CDT(-0500)] <heidi_> k
[10:08:30 CDT(-0500)] <cindyli> heidi_, michelled, Justin_o, i've fixed line spacing in ie7, 8. now working on text size. "text size" doesn't work in ie6, 7 because container.css("font-size") returns a string "small" rather than an actual px
[10:09:15 CDT(-0500)] <cindyli> i guess it worked before becuase the font-size was using px then
[10:09:20 CDT(-0500)] <Justin_o> cindyli: hmm.. i think we talked about that before.. can't remember what we said about that
[10:09:24 CDT(-0500)] <heidi_> ah, good catch cindyli
[10:09:33 CDT(-0500)] <cindyli> i'm thinking what's the best solution for this since the strings are valid font-size too
[10:09:53 CDT(-0500)] <heidi_> cindyli perhaps an em value?
[10:10:17 CDT(-0500)] <cindyli> Justin_o: did we talk about this before? as i rmb, the thing we talked about is the interesting line-height return in ie, not text size
[10:10:32 CDT(-0500)] <Justin_o> cindyli: maybe that was it
[10:10:48 CDT(-0500)] <cindyli> yes, heidi_, i'm thinking should we hardcode the conversion between each string value to a px value
[10:10:58 CDT(-0500)] <cindyli> say, "small" === "13px" etc
[10:11:08 CDT(-0500)] <heidi_> cindyli an em value would be better than px
[10:11:09 CDT(-0500)] <cindyli> but not sure if this is the best solution. any thoughts?
[10:12:05 CDT(-0500)] <cindyli> fyi, heidi_, Justin_o, michelled, the valid font-size in string: http://www.w3schools.com/cssref/pr_font_font-size.asp
[10:20:01 CDT(-0500)] <Justin_o> cindyli: did you see any exact translations to ems or pxs
[10:20:25 CDT(-0500)] <cindyli> Justin_o: u mean thru jquery? no
[10:20:55 CDT(-0500)] <Justin_o> cindyli: i meant on the site or somewhere else.. wondering if small == some specific value
[10:21:51 CDT(-0500)] <cindyli> Justin_o: not yet, searching..
[10:21:57 CDT(-0500)] <Justin_o> cindyli: thanks
[10:24:11 CDT(-0500)] <heidi_> Justin_o should we be doing swarm stuff
[10:26:30 CDT(-0500)] <Justin_o> heidi_: you should run the unit tests.. you can do it through the storm or manually
[10:26:37 CDT(-0500)] <Justin_o> i think the storm should be up-to-date
[10:27:01 CDT(-0500)] <Justin_o> swarm ^
[10:27:20 CDT(-0500)] <Justin_o> but it may fail on a iframe html page that was misnamed
[10:27:25 CDT(-0500)] <heidi_> it is already up to date?
[10:27:37 CDT(-0500)] <Justin_o> it should be up-to-date with the project repo
[10:27:38 CDT(-0500)] <heidi_> whta's the url again
[10:27:40 CDT(-0500)] <heidi_> ah
[10:27:45 CDT(-0500)] <Justin_o> swarm.fluidproject.org
[10:46:09 CDT(-0500)] <mlam1> Justin_o: there's a known layout reorderer test that fails , right?
[10:46:39 CDT(-0500)] <mlam1> "drop warning visibility for and down - 10. temporary moved state is cleared"
[10:47:24 CDT(-0500)] <Bosmon> mlam1 - it is one of those that will succeed when you run it again
[10:47:33 CDT(-0500)] <mlam1> oh ok
[10:47:42 CDT(-0500)] <mlam1> bingo.
[10:48:03 CDT(-0500)] <Bosmon> I noticed a UIEnhancer test failing last night
[10:48:14 CDT(-0500)] <Bosmon> But it may just be because of my before-the-flood copy of FF 3.6
[10:48:25 CDT(-0500)] <Bosmon> Test 2.2
[10:50:16 CDT(-0500)] <mlam1> yah, it fails for me as well
[10:50:31 CDT(-0500)] <mlam1> i'm running FF 3.6.21 on xp
[10:56:37 CDT(-0500)] <michelled> Justin_o: a failing test! that must be a blocker
[10:58:01 CDT(-0500)] <mlam1> it's also breaking in chrome/IE in xp
[11:00:03 CDT(-0500)] <michelled> Bosmon: did you see anastasiac's comment on your pull request?
[11:00:55 CDT(-0500)] <Justin_o> michelled: yes.. i'd think so
[11:01:34 CDT(-0500)] <Bosmon> Yes, I did
[11:01:53 CDT(-0500)] <michelled> thoughts on it? are you still good with me pushing your pull request?
[11:02:01 CDT(-0500)] <Bosmon> If the KINGG is
[11:02:12 CDT(-0500)] <Bosmon> I could look at fixing it now so we don't need 2 reviews though
[11:02:17 CDT(-0500)] <Bosmon> Well
[11:02:20 CDT(-0500)] <Bosmon> I guess it doesn't matter
[11:02:25 CDT(-0500)] <Bosmon> it would always need 2 reviews
[11:02:27 CDT(-0500)] <michelled> Justin_o: does it bother you that the old fashioned way to specifying options- the way we don't want people to use it - doesn't work?
[11:02:55 CDT(-0500)] <Bosmon> A CATT is excited by the DAMPP weather
[11:03:10 CDT(-0500)] <Bosmon> The like of which we haven't seen for 6 months
[11:03:35 CDT(-0500)] <Justin_o> Bosmon, michelled: is that going to be one of those undocumented features
[11:03:48 CDT(-0500)] <Bosmon> I think it is already undocumented
[11:03:59 CDT(-0500)] <michelled> mlam1: can you file and fix the broken test please?
[11:04:01 CDT(-0500)] <Bosmon> Noone could reasonably guess what the containment structure of UIOptions actually is
[11:05:00 CDT(-0500)] <Justin_o> Bosmon: okay.. then i guess for now it's okay.. but we should sort that out for the next release
[11:06:00 CDT(-0500)] <michelled> Bosmon: could you file that for 1.5?
[11:06:01 CDT(-0500)] <Bosmon> The whole component will be destroyed for the next release
[11:06:04 CDT(-0500)] <anastasiac> Justin_o, I'm just filing a JIRA for the fact that some of the demos aren't working (links to JS files not updated to new naming scheme). Am I correct in assuming this would be a blocker for 1.4?
[11:06:23 CDT(-0500)] <Bosmon> anastasiac - I noticed I had a few dirty files in my commit
[11:06:36 CDT(-0500)] <Bosmon> Unfortunately they went to a completely random branch in my repo, I'll fix that today
[11:06:36 CDT(-0500)] <Justin_o> anastasiac: i guess what do you mean they aren't working, and yes.. i think it would be a blocker
[11:06:53 CDT(-0500)] <anastasiac> Justin_o: they don't work
[11:09:54 CDT(-0500)] <mlam1> michelled: yup
[11:10:02 CDT(-0500)] <Justin_o> anastasiac: yes.. not working would be a blocker
[11:10:11 CDT(-0500)] <Justin_o> anastasiac: so i guess we should halt testing for a bit
[11:11:06 CDT(-0500)] <Bosmon> Just give me a few minutes
[11:11:44 CDT(-0500)] <anastasiac> Bosmon: http://issues.fluidproject.org/browse/FLUID-4429
[11:12:36 CDT(-0500)] <Justin_o> fluid-everyone: please don't test "inline edit", "list reorderer" till after FLUID-4429 has been addressed
[11:13:09 CDT(-0500)] <anastasiac> fluid-everyone: Just to clarify: that's simple inline-edit, not rich. Rich is fine
[11:15:03 CDT(-0500)] <heidi_> Justin_o who worked on uploader for 1.4?
[11:15:12 CDT(-0500)] <heidi_> it doesn't work w keyboard on FF6
[11:27:12 CDT(-0500)] <Bosmon> michelled, Justin_o - I just pushed an updated fix to my FLUID-4428 branch which resolves anastasiac's direction options drilling issue
[11:27:16 CDT(-0500)] <Bosmon> Now looking at FLUID-4429
[11:28:37 CDT(-0500)] <michelled> Bosmon: can you reissue the pull request for 4428? I already pushed it
[11:28:51 CDT(-0500)] <heidi_> Justin_o ignore my last one. it's something to do with focus moving
[11:32:02 CDT(-0500)] <cindyli> michelled: Justin_o, can you have a look on the fix for 4427 @ commit https://github.com/cindyli/infusion/commit/fb80094f2407c5775b3a0801c6a6c3f414cd89ce
[11:34:04 CDT(-0500)] <cindyli> michelled, Justin_o, the place that I'm not quite happy with is the newly-introduced "fontSizeMap" which is the mapping between font-size in string and the actual px value. as mentioned in the standup, the px value varies depending on the OS display setting, according to this article - http://style.cleverchimp.com/font_size_intervals/altintervals.html
[11:35:07 CDT(-0500)] <Bosmon> Pull requests up for 4428 and 4429
[11:35:25 CDT(-0500)] <michelled> thx Bosmon - I'll take a look
[11:35:41 CDT(-0500)] <michelled> can you take a look at the work cindyli is talking about?
[11:36:30 CDT(-0500)] <Bosmon> Yes
[11:36:51 CDT(-0500)] <Bosmon> cindyli - I don't think the approach of looking in the component defaults directly on line 453 is correct
[11:37:06 CDT(-0500)] <Bosmon> If this mapping is configurable, it should be possible for options which the user supplies to override it
[11:39:14 CDT(-0500)] <Bosmon> cindyli - where do the strings like "xx-large" etc. come from?
[11:39:35 CDT(-0500)] <Bosmon> Under what conditions will these be seen in the "font-size" CSS value?
[11:41:16 CDT(-0500)] <mlam1> Justin_o: can you check my pull request for FLUID-4430 (broken UIE test) ?
[11:43:27 CDT(-0500)] <anastasiac> Bosmon, I'm testing your update to 4428. Should I be able to attach two different event listeners, one using the "old-fashioned" way and one through options munging, at the same time?
[11:46:43 CDT(-0500)] <cindyli> Bosmon: yes, "xx-large" is a valid font-size value - http://www.w3schools.com/cssref/pr_font_font-size.asp
[11:46:49 CDT(-0500)] <Justin_o> mlam1: sure.. i'll take a look
[11:47:27 CDT(-0500)] <cindyli> however, in ie 8 and above, the string values like "xx-large" is interpreted into actual px values, but not in ie6 & 7
[11:48:18 CDT(-0500)] <cindyli> Bosmon: that's where i think we need the mapping between the strings and values
[11:50:52 CDT(-0500)] <cindyli> Bosmon: i agree that "the approach of looking in the component defaults directly on line 453" doesn't seem right. but at the point textSizer and lineSpacer get called, that.options.fontSizeMap isn't available yet. where else can i define the map and it's available to be accessed at then?
[11:53:57 CDT(-0500)] <Justin_o> mlam1: michelled will actually be looking at your test
[11:54:34 CDT(-0500)] <cindyli> Bosmon: A summary to ur question - Under what conditions will these be seen in the "font-size" CSS value?
[11:55:09 CDT(-0500)] <mlam1> ok, thanks Justin_o
[11:56:18 CDT(-0500)] <michelled> mlam1: it's pushed - thanks for fixing that - I'm the one that broke it
[11:56:45 CDT(-0500)]

[11:57:52 CDT(-0500)] <Bosmon> cindyli - you can actually get access to the merged options by "boiling" arguments to the invokers
[11:58:01 CDT(-0500)] <Bosmon> Since invoker arguments are not evaluated until the point of call
[11:58:23 CDT(-0500)] <cindyli> ah. good point. trying
[11:58:36 CDT(-0500)] <mlam1> michelled: anytime
[11:58:38 CDT(-0500)] <Bosmon> Is "xx-large" some recognised part of CSS?
[11:58:45 CDT(-0500)] <Bosmon> I hadn't seen it before....
[11:58:50 CDT(-0500)] <Bosmon> Ah, I see your link now
[11:59:20 CDT(-0500)] <Bosmon> Ouch, w3schools :PP
[12:02:12 CDT(-0500)] <Bosmon> cindyli - do you think you could try heidi_'s suggestion of using values in ems rather than px?
[12:02:29 CDT(-0500)] <Bosmon> It sounds like it might provide values which are more stable wrt. "ppem variation"
[12:02:53 CDT(-0500)] <cindyli> Bosmon: jquery (container.css("font-size")) always return the font-size in px rather than em
[12:03:14 CDT(-0500)] <cindyli> we could do the conversion of course, if it's necessary
[12:04:34 CDT(-0500)] <Bosmon> cindyli - I think we might have to - assuming we can discover the conversion factor
[12:05:40 CDT(-0500)] <cindyli> ic. looking
[12:12:26 CDT(-0500)] <Bosmon> This is an issue that looks complex enough we should probably punt it for the release
[12:12:36 CDT(-0500)] <Bosmon> As long as the configuration of this table by the integrator actually works
[12:12:46 CDT(-0500)] <Bosmon> It's always been pretty suspicious that UIO only dealt with fonts in units of px...
[12:13:03 CDT(-0500)] <Bosmon> This page is a good guide to the affordances of the various kinds of measurements: http://css-tricks.com/2580-css-font-size/
[12:13:33 CDT(-0500)] <Bosmon> Given that this discussion started out being focused on IE, we may as well just make sure that our solution works there
[12:17:17 CDT(-0500)] <anastasiac> Bosmon, I'm testing your update to 4428. Should I be able to attach two different event listeners, one using the "old-fashioned" way and one through options munging, at the same time?
[12:17:34 CDT(-0500)] <Bosmon> anastasiac - you should but you can't
[12:17:41 CDT(-0500)] <Bosmon> And it will be too hard to fix it today
[12:17:43 CDT(-0500)] <anastasiac>
[12:17:50 CDT(-0500)] <anastasiac> sounds reasonable to me
[12:18:19 CDT(-0500)] <Bosmon> Actually it would require a fundamental revision to the way that options merging works in the framework
[12:18:50 CDT(-0500)] <Bosmon> I think asking users to namespace their listeners in the bizarre case they actually deliver options through the two different schemes at the same time is a reasonable workaround
[12:19:08 CDT(-0500)] <anastasiac> indeed
[12:19:30 CDT(-0500)] <Bosmon> And as we said earlier, NOONE should be using "direct options" on UIOptions
[12:19:50 CDT(-0500)] <Bosmon> There's no way they could usefully discover this layout anyway, and it is not going to be stable
[12:20:23 CDT(-0500)] <Bosmon> The latest push to FLUID-4428 is really just there to rule out the possibility of some more fundamental bug due to the lack of supporting direct options....
[12:20:44 CDT(-0500)] <Bosmon> There may be some case in which, if munging fails somehow, they might expect to use direct binding as a workaround
[12:24:31 CDT(-0500)] <michelled> Justin_o: I think you can rebuild the nightly and get people testing inline edit again
[12:25:11 CDT(-0500)] <michelled> hi luciantimofte - nice to have you hanging out with us
[12:26:00 CDT(-0500)] <Justin_o> michelled: thanks
[12:32:43 CDT(-0500)] <heidi_> mlam around?
[12:40:20 CDT(-0500)] <heidi_> Justin_o would you say the sliding panel not focusing when opening is a blocker
[12:40:48 CDT(-0500)] <Justin_o> heidi_: i don't think so.. jameswy what do you think
[12:41:18 CDT(-0500)] <jameswy> Justin_o, heidi_: What's it focusing on by default?
[12:41:21 CDT(-0500)] <heidi_> jameswy right now if you are using a screen reader, and use the 'show' button for the UIO panel, you don't get any feedback
[12:41:47 CDT(-0500)] <heidi_> to say the panel is open, and you have to know to go back to access its contents
[12:42:01 CDT(-0500)] <heidi_> so one thought was to refocus to the start of the newly opened panel
[12:43:48 CDT(-0500)] <cindyli> Bosmon: ur commit "a4485f001d9049f8276f7f804e24cf9faf1618a5" for FLUID-4428 seems breaking ie6. i got error "a runtime error has occurred at line 291: expected identifier, string or number"
[12:44:03 CDT(-0500)] <cindyli> might be just a jslint error
[12:59:19 CDT(-0500)] <michelled> cindyli: thanks for catching the ie6 breakage - did Bosmon get back to you on it?
[12:59:51 CDT(-0500)] <cindyli> not yet, michelled
[13:02:19 CDT(-0500)] <Bosmon> I am testing now
[13:02:53 CDT(-0500)] <Bosmon> I tried demo "uiOptionsFullWithoutPreview.html" but it seems to work for me
[13:02:57 CDT(-0500)] <Bosmon> Which file gives the error for you?
[13:18:52 CDT(-0500)] <Bosmon> cindyli, michelled - I have pushed some linting fixes to my branch
[13:19:07 CDT(-0500)] <Bosmon> It seems to work a bit better, although I notice that the sliding panel animation doesn't actually work for me on IE6
[13:20:10 CDT(-0500)] <cindyli> thanks, Bosmon
[13:21:46 CDT(-0500)] <michelled> Bosmon: which branch? do we need another pull request?
[13:23:48 CDT(-0500)] <Bosmon> Same branch
[13:24:02 CDT(-0500)] <Bosmon> It's odd that one can't reopen requests by pushing to branches....
[13:24:08 CDT(-0500)] <Bosmon> Or indeed, reopen them at all
[13:24:11 CDT(-0500)] <Bosmon> At least, I couldn't see a way
[13:33:38 CDT(-0500)] <michelled> Justin_o: dev meeting today?
[13:35:02 CDT(-0500)] <michelled> fluid-everyone: I was thinking that since we've got some momentum on the release we should do a quick dev meeting here
[13:35:55 CDT(-0500)] <yura> sounds good michelled'
[13:36:50 CDT(-0500)] <michelled> we've got 2 blockers left
[13:37:03 CDT(-0500)] <michelled> cindyli: can you give us a quick update on 4427
[13:38:34 CDT(-0500)] <cindyli> i've adjusted the change according to Bosmon's feedback. tested in firefox, works well, i'm waiting on Bosmon's fix on ie6 breakage to get into project repo so i can proceed with ie6 testing
[13:39:25 CDT(-0500)] <michelled> ok, that's waiting on me
[13:40:08 CDT(-0500)] <michelled> and the final blocker is 4362 which Justin_o is reviewing but waiting on 4427 to get in first
[13:40:32 CDT(-0500)] <michelled> all of these blockers are all related to UIO
[13:40:43 CDT(-0500)] <michelled> and QA has started
[13:41:09 CDT(-0500)] <michelled> mlam: do you have the link for QA testing tasks?
[13:41:33 CDT(-0500)] <heidi_> i think i made a new blocker FLUID-4433
[13:41:41 CDT(-0500)] <heidi_> for uploader - can't delete a file w keyboard
[13:41:42 CDT(-0500)] <mlam> yeah, i do
[13:41:50 CDT(-0500)] <mlam> http://wiki.fluidproject.org/display/fluid/Release+Testing+Tasks
[13:41:51 CDT(-0500)] <heidi_> mlam maybe you want to check out that one
[13:42:16 CDT(-0500)] <mlam> Ok, heidi_ I can check it out
[13:42:32 CDT(-0500)] <michelled> heidi_: did Justin_o sign off on that as a blocker?
[13:42:51 CDT(-0500)] <michelled> (it sounds like a blocker to me but he's the king after all)
[13:42:52 CDT(-0500)] <heidi_> michelled nope i just made it - what do you think Justin_o?
[13:44:33 CDT(-0500)] <michelled> looks like Justin_o stepped away from his desk
[13:45:17 CDT(-0500)] <heidi_> mlam let me know if you find the same issue
[13:45:25 CDT(-0500)] <mlam> ok
[13:45:28 CDT(-0500)] <mlam> FF6?
[13:45:32 CDT(-0500)] <heidi_> i mean if you can replicate
[13:45:39 CDT(-0500)] <heidi_> FF6, safari 5, ff on win, IE
[13:45:44 CDT(-0500)] <heidi_> (everything i tried)
[13:46:25 CDT(-0500)] <mlam> heidi, i'm deleting files in ubuntu
[13:46:36 CDT(-0500)] <mlam> with the delete key
[13:46:38 CDT(-0500)] <heidi_> mlam are you just using the 'delete' key or is it a special combo
[13:46:55 CDT(-0500)] <heidi_> ok, let me know about the others
[13:48:03 CDT(-0500)] <heidi_> mlam michelled Justin_o the other kind ugly one is that in IE7 the uploader is scrollbarry - i think an fl-fix issue FLUID-4432
[13:48:09 CDT(-0500)] <mlam> just the delete key heidi_
[13:48:20 CDT(-0500)] <heidi_> mlam do you have access to mac or win?
[13:48:32 CDT(-0500)] <mlam> i'm on a mac now
[13:48:37 CDT(-0500)] <mlam> i'll give it a shot
[13:51:36 CDT(-0500)] <michelled> fluid-everyone: anything else you want to talk about when we are all paying attention?
[13:52:04 CDT(-0500)] <Justin_o> heidi_: did you try fn+del
[13:52:20 CDT(-0500)] <Justin_o> what kind of computer are you using
[13:52:22 CDT(-0500)] <Justin_o> ?
[13:52:22 CDT(-0500)] <michelled> anastasiac: perhaps you want to remind us about tomorrow and Friday?
[13:52:31 CDT(-0500)] <heidi_> Justin_o fn+del works
[13:52:40 CDT(-0500)] <anastasiac> michelled and Justin_o, I guess one question is: what's the process for if/when we find bugs while testing? who makes the call as to what's a blocker? how does that person find out about the bug in the first place?
[13:52:53 CDT(-0500)] <mlam> fn+del!
[13:53:00 CDT(-0500)] <michelled> anastasiac: the KING makes the call
[13:53:06 CDT(-0500)] <heidi_> Justin_o oh is that what i'm supposed to use?
[13:53:09 CDT(-0500)] <heidi_> mlam ?
[13:53:10 CDT(-0500)] <michelled> if you find a bug that you think is a blocker talk about it here
[13:53:14 CDT(-0500)] <mlam> i'm not sure about htat
[13:53:23 CDT(-0500)] <anastasiac> ok, sounds good
[13:53:24 CDT(-0500)] <michelled> and if Justin_o signs off we can add it to bug parade
[13:53:28 CDT(-0500)] <mlam> heidi_: , i'm not sure
[13:53:30 CDT(-0500)] <heidi_> the instructions say "press delete"
[13:53:37 CDT(-0500)] <heidi_> when i hover
[13:53:58 CDT(-0500)] <anastasiac> an yes, fluid-everyone: Tomorrow and Friday we'll be having a docs sprint. Topics and suggested authors are posted on the wiki at http://wiki.fluidproject.org/display/fluid/Docs+Sprint+September+2011
[13:54:05 CDT(-0500)] <anastasiac> we'll coordinate here in the channel
[13:54:07 CDT(-0500)] <heidi_> mlam does delete work for you, or only fn+del ?
[13:54:15 CDT(-0500)] <mlam> only fn+del
[13:54:20 CDT(-0500)] <Justin_o> heidi_: yes.. it's because of your keyboard
[13:54:21 CDT(-0500)] <michelled> he's on a mac too
[13:54:29 CDT(-0500)] <Justin_o> delete on make is === to backspace
[13:54:30 CDT(-0500)] <mlam> the delete key keeps going to my previous browser page
[13:54:35 CDT(-0500)] <Justin_o> which is back in the browser
[13:54:45 CDT(-0500)] <Justin_o> so fn+del makes it a real del key
[13:54:51 CDT(-0500)] <heidi_> Justin_o is that common knowledge or am i a dummy ha
[13:55:03 CDT(-0500)] <heidi_> should we state that somewhere for users?
[13:55:54 CDT(-0500)] <Justin_o> heidi_: sure... could you add a comment to the test plan to indicate that on mac laptop style keyboards to use fn+del
[13:56:09 CDT(-0500)] <Justin_o> i'll close the jira
[13:56:09 CDT(-0500)] <heidi_> Justin_o i don't mean test plan, i mean for users of the uploader
[13:56:30 CDT(-0500)] <Justin_o> anastasiac: is the keyboard deletion documented
[13:56:36 CDT(-0500)] <Justin_o> if so we could add it there heidi
[13:56:42 CDT(-0500)] <Justin_o> heidi_: ^
[13:57:12 CDT(-0500)] <heidi_> Justin_o it's just confusing that the hover says 'Press Delete key to remove file'
[13:57:17 CDT(-0500)] <anastasiac> Justin_o, you raise a good point: as far as I know, we don't have "user" documentation for any of our components
[13:57:30 CDT(-0500)] <heidi_> instead of 'Press fn+del on mac, and del on windows' or whatever
[13:58:10 CDT(-0500)] <Bosmon> fn + del
[13:58:12 CDT(-0500)] <Bosmon> So offensive
[13:58:13 CDT(-0500)] <Justin_o> heidi_: yah.. it is a bit confusing, the tooltip should probably have said "del" key
[13:58:20 CDT(-0500)] <Bosmon> These Mac "keyboards" don't even have a page up key
[13:58:35 CDT(-0500)] <Justin_o> Bosmon: you can make them do page up
[13:58:43 CDT(-0500)] <Bosmon> They clearly watched one Star Trek episode too many!
[13:58:45 CDT(-0500)] <Justin_o> but i pretty much never use it anways..
[13:59:00 CDT(-0500)] <Justin_o> i'll have to track down jameswy to spread some apple propaganda
[13:59:11 CDT(-0500)] <Bosmon> Did you hear that Samsung's countersuit against Apple, who claimed that they stole iPad ideas, is that Apple ripped off their designs from episodes of Star Trek....
[13:59:30 CDT(-0500)] <Bosmon> I hope this works in court
[14:00:11 CDT(-0500)] <michelled> heidi_: the trouble with documenting that is that the larger format mac keyboards have both delete and backspace
[14:00:19 CDT(-0500)] <michelled> so the documentation would not be correct
[14:01:23 CDT(-0500)] <heidi_> Justin_o did jameswy make a call about the blockerness of the inaccessibility of the sliding panel?
[14:01:59 CDT(-0500)] <Justin_o> heidi_: not that I know of.. he's in a meeting at the moment too
[14:02:17 CDT(-0500)] <heidi_> michelled so are users just supposed to know the fn+del trick? tricky one.... i'm a pretty experienced user that didn't know :|
[14:02:40 CDT(-0500)] <Justin_o> Bosmon: they already tried to say that Apple ripped off the iPad from 2001: A Space Odessy so i think they're just grasping for things now
[14:02:46 CDT(-0500)] <heidi_> and it made the keyboard experience not really useable
[14:02:50 CDT(-0500)] <heidi_> for me
[14:03:46 CDT(-0500)] <michelled> Justin_o: maybe we should respond to both delete and backspace?
[14:03:49 CDT(-0500)] <Justin_o> heidi_: i'll defer to anastasiac about documentation related concerns
[14:04:08 CDT(-0500)] <heidi_> Justin_o i'm not convinced it's documentation issue - users are going to do what the hover says
[14:04:11 CDT(-0500)] <Justin_o> michelled: i don't think we should respond to backspace since it's convention to use that to do a browser back
[14:04:38 CDT(-0500)] <Justin_o> heidi_: yes.. i think the tooltip would be better suited to say "del" in that case
[14:04:39 CDT(-0500)] <heidi_> which on a mac laptop, sends them back in their browser, which isn't acceptable i don't think?
[14:04:53 CDT(-0500)] <Justin_o> although make keyboards do use delete for both
[14:04:54 CDT(-0500)] <heidi_> Justin_o what does "del" mean
[14:04:59 CDT(-0500)] <anastasiac> I agree with heidi_ on this: even if we document this on our website, atutor users (for example) won't be looking at our website when they use the uploader
[14:05:02 CDT(-0500)] <Justin_o> we could say forward delete
[14:05:35 CDT(-0500)] <heidi_> Justin_o what does forward delete mean
[14:05:50 CDT(-0500)] <anastasiac> it's a bit like the image reorderer issue: we had to find a way to display instructions. we need to find a short, succinct way to communicate this in the hover
[14:06:16 CDT(-0500)] <Justin_o> heidi_: forward delete is the key that we actually look for
[14:06:22 CDT(-0500)] <michelled> well, this sounds like a mac bug to me - keyboards that are mis-labeled
[14:06:24 CDT(-0500)] <Justin_o> on pc keyboards that's called "del" usually
[14:06:28 CDT(-0500)] <heidi_> are there any other components that 'rely' on the delete key
[14:07:03 CDT(-0500)] <Justin_o> on mac keyboards it is delete with the symbol by it, or fn+delete for laptop style keyboards
[14:07:41 CDT(-0500)] <heidi_> Justin_o i think we need to communicate that in the instructions somehow
[14:07:54 CDT(-0500)] <heidi_> to make the component accessible and inclusive
[14:08:43 CDT(-0500)] <heidi_> i.e. i'm not comfortable with this issue being closed... we should try to think of a solution
[14:08:46 CDT(-0500)] <Justin_o> heidi_: yes.. i think we could make the tooltip a bit more understandable... so my suggestion would be to change the text to say "forward delete" or something, we can get james' input on that
[14:09:28 CDT(-0500)] <michelled> Justin_o: has this behaviour changed since 1.3?
[14:09:32 CDT(-0500)] <Justin_o> so i don't think it's a blocker though.. it's not new functionality at all...
[14:09:49 CDT(-0500)] <Justin_o> yes.. uploader has always worked like that for keyboard deletion
[14:10:12 CDT(-0500)] <michelled> ok, sounds like we need to get jameswy to think about this and we can put in his solution for 1.5
[14:10:13 CDT(-0500)] <heidi_> yeah not a blocker, but still a bug
[14:10:52 CDT(-0500)] <Justin_o> heidi_, michelled: agreed
[14:11:00 CDT(-0500)] <michelled> heidi_: do you want to update that jira and punt to 1.5?
[14:11:06 CDT(-0500)] <heidi_> sure yep
[14:11:20 CDT(-0500)] <heidi_> there was another Justin_o , re: IE7 and uploader.
[14:11:24 CDT(-0500)] <heidi_> it's all scrollbarry
[14:11:26 CDT(-0500)] <Justin_o> heidi_: i closed the one you had filed.. you can either re-open or file a new one
[14:11:47 CDT(-0500)] <Justin_o> heidi_: hmm.. what was that one?
[14:12:58 CDT(-0500)] <heidi_> Justin_o FLUID-4432
[14:13:05 CDT(-0500)] <heidi_> i can't upload a screenshot to jira tho
[14:14:48 CDT(-0500)] <michelled_> heidi_: I found that I could upload a file
[14:15:09 CDT(-0500)] <heidi_> i get Cannot attach file Screen shot 2011-09-07 at 3.12.13 PM.png: Unknown server error (201).
[14:15:35 CDT(-0500)] <michelled_> yes, then I clicked something like 'attach' and it gave me the option
[14:15:39 CDT(-0500)] <Justin_o> heidi_: use this url to access jira issues.fluidproject.org:18080
[14:16:12 CDT(-0500)] <heidi_> Justin_o thanks that did it
[14:16:15 CDT(-0500)] <heidi_> screen shot up
[14:16:28 CDT(-0500)] <heidi_> http://issues.fluidproject.org:18080/browse/FLUID-4432
[14:17:53 CDT(-0500)] <heidi_> Justin_o fl-fix has to be changed to fl-clearfix .... i think that's it!
[14:18:58 CDT(-0500)] <Justin_o> heidi_: okay.. can you quickly test that out and if it works i think we'll make it a blocker
[14:19:44 CDT(-0500)] <heidi_> Justin_o i just have fakey IE7 - do you have a real one?
[14:20:10 CDT(-0500)] <Justin_o> heidi_: yep..
[14:20:17 CDT(-0500)] <michelled_> cindyli: I just pushed Bosmon's linting - can you test your fix?
[14:20:21 CDT(-0500)] <Justin_o> can you send me the file and line to change
[14:20:39 CDT(-0500)] <cindyli> sure, michelled_
[14:21:12 CDT(-0500)] <heidi_> Justin_o components/uploader/html/Uploader.html line 90
[14:21:28 CDT(-0500)] <michelled_> Bosmon: are you looking at the 4354 tests?
[14:25:31 CDT(-0500)] <heidi_> mlam there's also this one i found: http://issues.fluidproject.org:18080/browse/FLUID-4431
[14:25:53 CDT(-0500)] <heidi_> def not a blocker but maybe fixable?
[14:26:01 CDT(-0500)] <anastasiac> Justin_o, I notice that all the mac testing tasks require 10.6 - what of those of us who are using the new systems, with 10.7?
[14:26:07 CDT(-0500)] <mlam> not for 1.4 though, right heidi_?
[14:26:38 CDT(-0500)] <heidi_> i guess not?
[14:26:42 CDT(-0500)] <Justin_o> anastasiac: hmm...
[14:26:47 CDT(-0500)] <Justin_o> that would be none of us
[14:26:52 CDT(-0500)] <heidi_> me
[14:27:04 CDT(-0500)] <Justin_o> it would only be our old machines..so my mini and your imac
[14:27:07 CDT(-0500)] <heidi_> i'll just do all the testing, you guys relax
[14:27:10 CDT(-0500)] <heidi_>
[14:27:18 CDT(-0500)] <Justin_o> heidi_: +1
[14:27:18 CDT(-0500)] <anastasiac> I was about to mention the iMac
[14:27:30 CDT(-0500)] <anastasiac> are we not interested in testing on 10.7?
[14:28:01 CDT(-0500)] <Justin_o> anastasiac: it's a grey area
[14:28:07 CDT(-0500)] <Justin_o> we can discuss it
[14:28:21 CDT(-0500)] <Justin_o> we were just unlucky to have a lot of updates land in the middle of our release
[14:31:40 CDT(-0500)] <cindyli> Bosmon: I sent a pull request for 4427 (https://github.com/fluid-project/infusion/pull/160), can you review the code? thanks.
[14:34:40 CDT(-0500)] <heidi_> mlam did you write http://wiki.fluidproject.org/display/fluid/Uploader+QA+Test+Plan i have a q
[14:34:54 CDT(-0500)] <heidi_> for Test 3: Add Files, Files Already in the File Queue
[14:35:02 CDT(-0500)] <heidi_> what's supposed to happen? it's not clear
[14:36:09 CDT(-0500)] <heidi_> mlam n/m i misunderstood - i guess i'm wondering what happens when the same file is added to the queue
[14:36:20 CDT(-0500)] <heidi_> right now it just creates another entry for the same file
[14:36:48 CDT(-0500)] <Justin_o> heidi_: looks like clearfix does work and jameswy does think it's a blocker
[14:36:59 CDT(-0500)] <Justin_o> did you make a jira for that?
[14:37:07 CDT(-0500)] <heidi_> Justin_o cool. yep it's 4432
[14:39:46 CDT(-0500)] <Justin_o> heidi_: i made that a blocker did you want to make a pull request
[14:40:10 CDT(-0500)] <heidi_> Justin_o sure
[14:40:59 CDT(-0500)] <heidi_> Justin_o all you did was change line 90 right?
[14:41:54 CDT(-0500)] <Bosmon> Hi, cindyli - this impl for 4427 looks good
[14:42:12 CDT(-0500)] <Bosmon> On line 338 you probably want to surface the fact that 1em = 16px is not universally true
[14:42:33 CDT(-0500)] <Bosmon> For a start, a comment referring to its different value on different platforms, and this should probably even be a component configuration option
[14:43:00 CDT(-0500)] <Justin_o> heidi_: yep
[14:43:20 CDT(-0500)] <Bosmon> A 3rd arg of "pixels per em" here (or "ppem") and another option for UIEnhancer + boiling
[14:43:39 CDT(-0500)] <Bosmon> Does it work on IE6 now?
[14:43:43 CDT(-0500)] <Bosmon> How about the sliding panel?
[14:44:54 CDT(-0500)] <heidi_> Justin_o sent
[14:45:44 CDT(-0500)] <cindyli> Bosmon: i see your point. I will make the change
[14:45:59 CDT(-0500)] <cindyli> yes, Bosmon, it works on IE6 now. the sliding panel looks good
[14:46:29 CDT(-0500)] <cindyli> do you still not see the animation with sliding panel? looks fine to me
[14:59:47 CDT(-0500)] <cindyli> Bosmon: pushed. can you have a look? thanks
[15:20:36 CDT(-0500)] <michelled> heidi_: you tested the fix for 4432, right?
[15:20:40 CDT(-0500)] <michelled> I'm going to push it
[15:23:50 CDT(-0500)] <heidi_> michelled Justin_o tested it on his real IE7
[15:23:55 CDT(-0500)] <michelled> fluid-everyone: given the 4432 blocker I think uploader testing will need to be restarted
[15:24:09 CDT(-0500)] <michelled> I'm about to push it and anastasiac will kick the build
[15:25:04 CDT(-0500)] <michelled> anastasiac: it's pushed
[15:25:37 CDT(-0500)] <anastasiac> kicked. let's see if it can get up again...
[15:32:09 CDT(-0500)] <anastasiac> fluid-everyone: the build is up again
[15:42:04 CDT(-0500)] <heidi_> Justin_o how do we add the max-file restriction to uploader? can we do that on the build site to test it?
[15:53:08 CDT(-0500)] <anastasiac> michelled: http://issues.fluidproject.org/browse/FLUID-4434
[15:54:06 CDT(-0500)] <michelled> thx