fluid-work IRC Logs-2012-11-13

[00:02:15 CST(-0600)] <thealphanerd> sorry

[00:02:18 CST(-0600)] <thealphanerd> did it again

[00:02:25 CST(-0600)] <thealphanerd> I've been listening to music all night

[00:02:26 CST(-0600)] <thealphanerd> http://soundcloud.com/ohsnapmusic/oh-snap-trapstorm

[00:02:59 CST(-0600)] <thealphanerd> but I was wondering if you had done much touch stuf in the browser?

[08:13:59 CST(-0600)] <clown> identify P3PP1F00

[08:14:09 CST(-0600)] * clown wrong room...

[10:22:41 CST(-0600)] <alexn> michelled: ayt ?

[10:23:06 CST(-0600)] <michelled> yep - in a call but you can type and I'll respond when I get a chance

[10:26:22 CST(-0600)] <alexn> great! I just wanted to ask you about code readability. I have two code extracts which have the same functionality and do the same thing but one of them is more "dehydrated". I just wanted to know your opinion on the code readability. The first extract http://pastie.org/5372287 and the second one http://pastie.org/5372323

[10:31:36 CST(-0600)] <alexn> michelled: oh sorry wrong links. These are 2 proper ones -> http://pastie.org/5372287 and http://pastie.org/5372409

[11:19:52 CST(-0600)] <michelled> alexn: I took a look at your two code examples

[11:20:09 CST(-0600)] <michelled> I prefer the second one, but I feel it is still a little hard to read

[11:20:48 CST(-0600)] <michelled> I think I would prefer it if you replaced options with just the disabled flag and changed the code accordingly

[11:21:09 CST(-0600)] <michelled> for example "tabindex": options.tabIndex would change to

[11:21:29 CST(-0600)] <michelled> "tabindex": disabled ? "-1" : null

[11:21:39 CST(-0600)] <alexn> michelled: great point

[11:21:46 CST(-0600)] <michelled> which makes the code a little more readable

[11:23:06 CST(-0600)] <alexn> true. I did not think about this one. I like the idea, let me change it accordingly. Thanks a lot, I knew that you will have some good ideas.

[11:23:24 CST(-0600)] <michelled> np alexn - glad I could help

[12:27:30 CST(-0600)] <michelled> anastasiac: can I get you to do one more thing before switching tasks? Can you flesh out the manual test plan?

[12:27:54 CST(-0600)] <michelled> you don't need to do both keyboard and mouse, just make sure all the features are fully covered

[12:28:03 CST(-0600)] <anastasiac> yes, of course, michelled

[12:28:08 CST(-0600)] <michelled> thanks!

[13:02:40 CST(-0600)] <anastasiac> michelled, I believe that all the mouse functionality testing is described on the QA page except the full-screen stuff. It would be good for you and Heidi to double-check it to see if I've missed anything, or gotten anything wrong

[13:03:08 CST(-0600)] <michelled> thanks anastasiac - I'll check it the next time I run through it all

[13:03:08 CST(-0600)] <anastasiac> michelled, regarding full-screen: I'm not sure what you're implementing, so I don't now what to put in the test plan (smile)

[13:03:23 CST(-0600)] <michelled> fair enough (smile)

[13:03:49 CST(-0600)] <anastasiac> I could write tests according to the designs, michelled; is that what you're working from?

[13:04:14 CST(-0600)] <michelled> yes - I'm putting our controls into action in full screen mode

[13:05:08 CST(-0600)] <anastasiac> and are you moving the scrub-bar down in-line with the other controls?

[13:05:52 CST(-0600)] <anastasiac> and what will happen to the "mouse-over-to-show-controls" in full-screen mode, michelled - will controls just always be visible?

[13:06:50 CST(-0600)] <michelled> anastasiac: my plan is not to have the visible - they would show for a length of time after mouse move or key press and then disappear

[13:07:02 CST(-0600)] <anastasiac> ok, cool.

[13:07:03 CST(-0600)] <michelled> but I'm not sure what I'll manage to get done this week.

[13:12:12 CST(-0600)] <anastasiac> michelled, I've updated the full-screen testing protocol, but we should revisit it once the code is in master.

[13:12:22 CST(-0600)] <michelled> sounds good

[13:12:24 CST(-0600)] <michelled> thx anastasiac

[14:02:36 CST(-0600)] <cindyli> jhung, justin, yura1, I pushed up some changes for dewarp and calibrate server to my branch - https://bitbucket.org/cindyli/decapod-0.7-server-iteration4, to adjust the location of the executables for dewarping and calibration. You may want to merge.

[14:03:49 CST(-0600)] <jhung> ok thanks cindyli

[14:03:56 CST(-0600)] <cindyli> np

[15:18:37 CST(-0600)] <anastasiac> hey, michelled, do you have a few minutes to talk about OER Commons Video Player integration, perhaps on Skype?

[15:25:44 CST(-0600)] <michelled> anastasiac: sure