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
[13:03:23 CST(-0600)] <michelled> fair enough
[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