fluid-work IRC Logs-2010-12-03
[08:50:46 CST(-0600)] <heidi> hey jhung, not sure clear on what's happening for http://issues.fluidproject.org/browse/FLUID-3869
[08:50:51 CST(-0600)] <heidi> can you skype/screen share and show me?
[08:51:14 CST(-0600)] <jamon> Justin_o: so, a little mailing list action, what should i be looking for?
[08:52:49 CST(-0600)] <Justin_o> jamon: hello, so the commits list doesn't seem to be getting e-mails about recent commits
[08:52:59 CST(-0600)] <Justin_o> basically after each commit to svn it should send out an e-mail
[08:53:22 CST(-0600)] <Justin_o> colinclark: did you commit anything yesterday and if so do you know when?
[08:53:51 CST(-0600)] <jamon> this started yesterday? i see stuff from dec. looks ok
[08:53:56 CST(-0600)] <jamon> er, dec. 1
[08:53:57 CST(-0600)] <colinclark> Yes, I committed several things yesterday. None of the emails arrived
[08:54:06 CST(-0600)] <colinclark> Mail hit the list on Dec 1 but not at all on Dec 2
[08:54:23 CST(-0600)] <Justin_o> the last commit e-mail i got was on Nov 30
[08:55:01 CST(-0600)] <colinclark> Yes, that's right. There was a uPortal build success notice on Dec 1
[08:55:16 CST(-0600)] <jamon> Dec 2 02:01:57 giga postfix/pipe[2977]: C46FF20BE3: to=<commits@fluidproject.org>, relay=mailman, delay=0.46, delays=0.06/0.05/0/0.36, dsn=2.0.0, status=sent (delivered via mailman service)
[08:55:20 CST(-0600)] <jamon> that's from dec.2
[08:55:58 CST(-0600)] <jamon> someone committing at 02:00? late night
[08:56:41 CST(-0600)] <Justin_o> jamon: we'll have to check with Bosmon
[08:56:55 CST(-0600)] <jamon> strange though, nothing after that
[08:57:36 CST(-0600)] <jamon> actually, i see more again from 02:00 this morning
[09:03:35 CST(-0600)] <jamon> colinclark: r10328 is the last commit correct?
[09:03:58 CST(-0600)] <colinclark> r10328 is the latest revision, yes
[09:04:39 CST(-0600)] <colinclark> Not all of our committers are in Eastern time zone, among other things
[09:09:27 CST(-0600)] <jamon> Justin_o, colinclark, that message came through now? subject svn commit: r10328
[09:09:42 CST(-0600)] <heidi> jhung are the hidden screen reader commands on all the reorderer demos?
[09:10:10 CST(-0600)] <heidi> " Screen reader users turn off virtual cursor (insert + z)." - is that general enough for all?
[09:10:35 CST(-0600)] <jhung> heidi: that's not what I meant
[09:10:37 CST(-0600)] <jhung> lol
[09:10:53 CST(-0600)] <heidi> jhung do we need specific screen reader commands?
[09:10:59 CST(-0600)] <colinclark> jamon: Not so far
[09:11:10 CST(-0600)] <jhung> heidi: I'm talking about the "Ctrl+arrow keys" instructions at the bottom, like in Layout Reorderer.
[09:11:28 CST(-0600)] <jhung> I should have been more clear.
[09:11:31 CST(-0600)] <jhung> Sorry
[09:11:32 CST(-0600)] <heidi> jhung yeah, i was just looking at the grid html and this stuff is there
[09:11:41 CST(-0600)] <heidi> just wondering what it means and if we need it
[09:12:20 CST(-0600)] <jhung> heidi: That's old and hard to read / understand. I think the new instructions are a bit better.
[09:12:46 CST(-0600)] <heidi> jhung agreed. there are two sets of instructions currently, one hidden for screen reader, and one keyboard. i will erase the former and clarify the latter!
[09:12:57 CST(-0600)] <jhung> kk
[09:18:02 CST(-0600)] <heidi> jhung do you want the keyboard instrc in the orange box still?
[09:24:11 CST(-0600)] <jhung> No. Plain text on white is good.
[09:24:17 CST(-0600)] <jhung> heidi ^
[09:24:32 CST(-0600)] <heidi> k
[09:27:30 CST(-0600)] <heidi> jhung are you using a consistent doctype for demos
[09:29:09 CST(-0600)] <heidi> n/m had to add stuff to html tag to get proper validation
[09:29:25 CST(-0600)] <jhung> cool
[09:30:42 CST(-0600)] <heidi> jhung one mor thing i find on safari i sometimes have to using the arrow keys twice to move things. same for you?
[09:35:40 CST(-0600)] <jhung> heidi: I can't do keyboard navigation easily on the ipad (because it doesn't have cursor keys!),
[09:35:52 CST(-0600)] <jhung> heidi: does this only happen for the grid reorderer?
[09:36:06 CST(-0600)] <heidi> ill check the others
[09:36:10 CST(-0600)] <jhung> k
[09:39:41 CST(-0600)] <heidi> jhung all demos
[09:41:24 CST(-0600)] <jhung> justin_o, jameswy, anastasiac, michelled, and other Mac users - can one of you help test any of the reorderer demos in Safari? Try reordering one of the items by keyboard.
[09:43:30 CST(-0600)] <colinclark> what should we be seeing?
[09:43:38 CST(-0600)] <colinclark> Grid seems to be working nicely for me
[09:43:45 CST(-0600)] <colinclark> Safari 5
[09:43:47 CST(-0600)] <heidi> colin is it taking two presses of the arrow key to move things
[09:43:54 CST(-0600)] <heidi> instead of one
[09:44:30 CST(-0600)] <heidi> it seems one is used to make the selection moveable, and one more to actually move it
[09:46:24 CST(-0600)] <colinclark> I'm not seeing that
[09:46:47 CST(-0600)] <colinclark> ah wait
[09:46:53 CST(-0600)] <heidi> it's not consistent
[09:46:55 CST(-0600)] <colinclark> On second move
[09:46:57 CST(-0600)] <colinclark> It is consistent
[09:47:05 CST(-0600)] <colinclark> It's just that it's the second time you move the same thing.
[09:47:18 CST(-0600)] <heidi> ah
[09:48:05 CST(-0600)] <heidi> should i file a jira for this?
[09:48:31 CST(-0600)] <colinclark> Undoubtedly, yes
[09:48:41 CST(-0600)] <heidi> k
[09:49:23 CST(-0600)] <heidi> also jhung have you looked at the layout reorderer (within the demo, as a frame) on a smallre screen? some txt goes outside the boxes
[09:52:19 CST(-0600)] <jhung> heidi: let me see...
[09:53:10 CST(-0600)] <jhung> heidi: ah yes, the word "Advertisement" spills out.
[09:54:30 CST(-0600)] <heidi> jhung i submitted a grid reorderer patch - have a peek when yr free
[09:54:42 CST(-0600)] <jhung> heidi: thanks.
[09:54:53 CST(-0600)] <jhung> heidi: I think we should just use the word "Ad" then.
[09:55:28 CST(-0600)] <heidi> jhung it happens with news too
[09:59:58 CST(-0600)] <jhung> heidi: news looks fine for me.
[10:00:43 CST(-0600)] <jhung> heidi: funny, all text looks fine in IE8. "Advertisement" overflows in FF3.6.
[10:01:48 CST(-0600)] <heidi> jhung its when i make the window small
[10:02:23 CST(-0600)] <heidi> on http://build.fluidproject.org/infusion/demos/reorderer/layoutReorderer/demo.html
[10:02:36 CST(-0600)] <jhung> yes. I get it now.
[10:02:56 CST(-0600)] <jhung> We should set a min-width for the demo container then.
[10:03:13 CST(-0600)] <heidi> something ya
[10:03:34 CST(-0600)] <heidi> it'd apply to every demo then ya
[10:03:40 CST(-0600)] <jhung> exactly.
[10:03:54 CST(-0600)] <jhung> Maybe a broader jira listing all the demos so we can do a checklist approach.
[10:04:55 CST(-0600)] <heidi> jhung yeah it might affect others
[10:05:07 CST(-0600)] <heidi> to not have a min width set
[10:05:33 CST(-0600)] <colinclark> heidi: Did you file a ticket for that Reorderer issue?
[10:05:40 CST(-0600)] <colinclark> Seems like it's a pretty massive blocker
[10:05:48 CST(-0600)] <colinclark> Can you share the link and ping the king about it
[10:05:49 CST(-0600)] <colinclark> ?
[10:08:48 CST(-0600)] <heidi> colinclark http://issues.fluidproject.org/browse/FLUID-3876
[10:09:04 CST(-0600)] <heidi> ill ping king too.
[10:10:15 CST(-0600)] <heidi> justin_o thinks it already exists somewhere... he's looking
[10:15:14 CST(-0600)] <Justin_o> colinclark, heidi: http://issues.fluidproject.org/browse/FLUID-3288
[10:15:46 CST(-0600)] <colinclark> So this is a bug since 1.1?
[10:15:56 CST(-0600)] <colinclark> Is this not a blocker, Justin_o?
[10:16:22 CST(-0600)] <Justin_o> colinclark: i'm not sure it's a blocker... it's definitely and inconvenience
[10:16:44 CST(-0600)] <Justin_o> but i could be convinced either way i think
[10:20:14 CST(-0600)] <jhung> colinclark, heidi, justin_o: we're chatting offline about this with Jameswy
[10:21:24 CST(-0600)] <jameswy> The interaction is faulty, but one can still interact with it (poorly). I'd consider it to be borderline blocker.
[10:24:47 CST(-0600)] <colinclark> lol
[10:25:04 CST(-0600)] <colinclark> anyone have any idea what causes it?
[10:27:43 CST(-0600)] <Justin_o> colinclark: nope... we had the same problem in firefox on Ubuntu for decapod
[10:28:03 CST(-0600)] <Justin_o> i can try to debug it this afternoon though
[10:28:37 CST(-0600)] <colinclark> It strikes me that we should take a look at it, and make a call for this release
[10:28:53 CST(-0600)] <colinclark> it's the kind of bug that we don't want to have lingering for three or four releases without being fixed
[10:29:05 CST(-0600)] <colinclark> but on the other hand, we have no idea what fixing it will entail
[10:29:47 CST(-0600)] <Justin_o> colinclark: yes... so i'll take a look at it this afternoon... and hopefully we'll get a better idea
[10:30:56 CST(-0600)] <colinclark> thanks, king
[10:33:03 CST(-0600)] <jhung1> standuppery.
[10:33:28 CST(-0600)] <colinclark> jess and i are both in a call at the moment, so we won't be able to attend
[10:58:39 CST(-0600)] <michelled> Justin_o: I know that we no longer support FF 2 but I thought you'd be interested to know that inline edit doesn't work in FF2. I can only type a single letter into the edit field and then I'm stuck.
[10:59:01 CST(-0600)] <Justin_o> michelled: really... interesting..
[10:59:26 CST(-0600)] <Justin_o> is it because of that line of code that was removed? or something else entirely?
[11:00:09 CST(-0600)] <michelled> something else - it's broken on the daily build
[11:01:27 CST(-0600)] <Justin_o> michelled: okay...
[11:02:19 CST(-0600)] <Justin_o> I guess we should file a jira for it.. i'm not too worried since we haven't supported FF2 for a while now.. but it would be nice if it still worked
[11:03:48 CST(-0600)] <michelled> ok, I can file it
[11:11:51 CST(-0600)] <michelled> Justin_o: FLUID-3877
[11:15:04 CST(-0600)] <michelled> Justin_o: FLUID-3861 is now closed
[11:23:14 CST(-0600)] <colinclark> hey heidi
[11:23:18 CST(-0600)] <heidi> hi
[11:23:34 CST(-0600)] <colinclark> One of the things I'd love to be able to show Mark Surman today is UI Options in action
[11:23:44 CST(-0600)] <colinclark> Do we still have a dev version of the FLOE site that has UI Options in it?
[11:23:56 CST(-0600)] <colinclark> I guess it's probably pretty ugly, but could we show it transforming the styling of the page?
[11:24:07 CST(-0600)] <heidi> yep it's on dev.inclusivedesign.ca
[11:24:21 CST(-0600)] <colinclark> jameswy: We could also show the recent UI Options wireframes that JenC has been working on to go with it, right?
[11:24:33 CST(-0600)] <colinclark> heidi: Cool
[11:24:54 CST(-0600)] <heidi> colinclark it ain't pretty but functional
[11:25:09 CST(-0600)] <heidi> and yes jen's wireframes would be nice to show! ill find link
[11:25:45 CST(-0600)] <colinclark> cool
[11:26:35 CST(-0600)] <heidi> colinclark http://wiki.fluidproject.org/display/fluid/UI+Options+Redesign+Mockups
[11:26:46 CST(-0600)] <colinclark> Thanks heidi
[11:26:53 CST(-0600)] <heidi> np
[11:46:32 CST(-0600)] <heidi> jhung do you have NVDA
[11:47:02 CST(-0600)] <heidi> reorder doesn't seem to work with it... is there a way around it? ctrl is a used key by NVDA
[11:47:46 CST(-0600)] <jhung> Weird. I've never had issues like that before. Let me see.
[11:50:31 CST(-0600)] <heidi> also the letter keys don't work to move, or the up/down arrows... they're also used by nvda
[11:50:47 CST(-0600)] <heidi> i can only use left/right arrows, but then can't move items
[11:51:45 CST(-0600)] <jhung> heidi: you need to be in application mode in NVDA. Press Insert + Space to switch modes.
[11:52:06 CST(-0600)] <heidi> oyy ok. does it work then?
[11:52:12 CST(-0600)] <jhung> It should.
[11:52:42 CST(-0600)] <jhung> I think there's a Jira for that. I think all our demos are to be set to application mode.
[11:54:43 CST(-0600)] <jhung> So I'm looking into http://issues.fluidproject.org/browse/FLUID-3753
[11:55:07 CST(-0600)] <jhung> Thinking of better label for the link that switches the Uploader from Flash version to the standard uploader.
[11:55:28 CST(-0600)] <jhung> How does the following sound:
[11:55:44 CST(-0600)] <jhung> "Simple (non-Flash) version available." and will appear as a link above the Uploader.
[11:59:26 CST(-0600)] <heidi> jhung that sounds good, or maybe just "non flash version available"
[11:59:57 CST(-0600)] <jhung> heidi: thought of that too, but some people may not understand what Flash vs. non-Flash would mean.
[12:00:10 CST(-0600)] <heidi> true. yeah that sounds good then
[12:00:51 CST(-0600)] <jhung> Heidi: thanks!
[12:13:49 CST(-0600)] <heidi> reorderer with voice over pretty awful. too bad... trying to figure out why
[12:24:02 CST(-0600)] <colinclark> anastasiac: Just doing a quick read through of documentatoin
[12:24:10 CST(-0600)] <colinclark> On this page: http://wiki.fluidproject.org/display/docs/ProtoComponent+Types
[12:24:29 CST(-0600)] <colinclark> Is the format cell for the Link protocomponent correct?
[12:24:35 CST(-0600)] <colinclark> it looks the same as Selection above it
[12:34:48 CST(-0600)] <anastasiac> colinclark, thanks for catching that
[13:17:21 CST(-0600)] <jamon> colinclark, Justin_o, commits list looks ok to me now, check design-commits though
[13:20:14 CST(-0600)] <Justin_o> jamon: thanks for fixing that... jhung, jameswy, have you guys committed anything to the design repo lately?
[13:21:10 CST(-0600)] <jameswy> Justin_o, jamon: Nothing's been committed to the design repo lately, but I should update it with some of the latest goods. Will let you know if design-commits is broken.
[13:27:43 CST(-0600)] <jhung> I haven't put anything into the design repo recently.
[13:54:36 CST(-0600)] <colinclark> Justin_o: You had mentioned the Safari issue with Reorderer
[13:54:48 CST(-0600)] <colinclark> Remind me again of what you found out while triaging it?
[13:55:05 CST(-0600)] <Justin_o> colinclark: so it seems it is related to the the blur event handler
[13:55:38 CST(-0600)] <Justin_o> for some reason in safari, after you move it triggers the blur event, which removes the drag styling... this styling is checked before it allows another move to occure
[13:55:40 CST(-0600)] <colinclark> ok
[13:55:57 CST(-0600)] <Justin_o> so the reason for the double tap is that it first has to put the styling back on
[13:57:21 CST(-0600)] <Justin_o> I commented out the line of code that attaches the blur event handler and the movement works fine, however the styling doesn't get removed when you tab out of the reorderer
[14:20:04 CST(-0600)] <Justin_o> colinclark: so i did some more digging... in thatReorderer.requestMovement there is a line of code that calls focus again on the element because during a move operation focus gets put onto the body. This is probably why the blur event is getting fired in safari
[14:20:36 CST(-0600)] <Justin_o> i think i can get it to work if I unbind the blur event on the element that is going to be moved, and then rebind it after the move event.
[14:20:45 CST(-0600)] <colinclark> Give it a shot and see how it goes, Justin_o
[14:20:53 CST(-0600)] <colinclark> Assuming it won't take too long
[14:21:00 CST(-0600)] <colinclark> Can we get any tests to fail in Safari, by chance?
[14:21:19 CST(-0600)] <Justin_o> you mean before i try to fix it?
[14:21:41 CST(-0600)] <Justin_o> all the tests are currently passing
[14:23:57 CST(-0600)] <Justin_o> hmm.. okay.. so i don't have to rebind the blur because i think there is a call to refresh that will end up doing that... but now i'm finding that i have to hit tab twice to get out of the reorderer
[14:23:59 CST(-0600)] <Justin_o> after a move
[14:24:02 CST(-0600)] <Justin_o> colinclark: ^
[14:24:14 CST(-0600)] <colinclark> odd
[14:37:46 CST(-0600)] <Justin_o> colinclark: okay... seems like you always had to double tab to get our in safari
[15:08:34 CST(-0600)] <Justin_o> colinclark, michelled: I've uploaded a patch for FLUID-3288. Please take a look and let me know what you think. I haven't added this jira to bug parade yet.. i have to run though.. but if you leave a comment on the jira or sending something to the list i can pick it up on monday