Reviewed and accepted #273 (keyboard events forever trapped by the viewer)
Reviewed and accepted the canvas refactor on #239 (resizing improvements)
Reviewed and accepted some CSS improvements by Pau
Added new events for #54 ("use this file" link click and image view)
Fixed #145 (write tests for prev/next)
Fixed #224 (closing the oldschool "use this file" dialog would close the lightbox)
Fixed #255 (clicks on site links were needlessly delayed by 500ms for logging)
Explored TMH (primarily the JS side) to get a better technical understanding for the next time we talk to Kaltura about the player upgrade. Wasn't as bad as I thought, the bulk of the code is the player and associated code that Kaltura is responsible for. The JS in TMH itself is nasty but it's only 600 lines of code. I did notice that some bugfixes that happened on our copy of the player haven't been ported over to the original. The whole setup with mwEmbed is questionable, but seems to still be used by Kaltura, so moving to ResourceLoader would probably be difficult.
Asked Mark to share his EventLogging -> Limn secrets, so that anyone can work on the related tickets (#54 & #198)
Today
Start working on #198 and #54 if Mark hasn't started and if he's had time to document the process
Otherwise, pick up #274 (tests to prevent future event handling issues like #273)
2014-02-27
Yesterday
Reviewed and approved the last changeset on #118 (permissions)
Finished polishing on #146 (blurry placeholder + progress bar) which was accepted
Addressed Gergo's concerns on #202 (CommonsMetadata hcard handling)
Reviewed Gergo's dotdotdot improvement for permissions' "view more"
Reviewed the new changeset on #239 (resizing issues), left a few comments about taking it further
Today
Weekly meeting
Start working on #198 (stats visualization) if it hasn't been picked yet
2014-02-26
Yesterday
Reviewed and accepted 2 changesets on #118 (permissions), raised concerns on another one
Reviewed and accepted the main fix on #239 (resize bugs), recorded a video of the remaining issue
Wrote follow-up bugfix and polishing on #146 (blurry placeholder + progress bar)
Wrote a fix for #242 (clicking next/prev rapidly can leave the lightbox in the wrong state) bundled with the #146 bugfix
Wrote a fix for #236 (left/right arrow should only work with no modifier key)
Wrote small polishing for #217 ("Uploaded by")
Today
Address Gergo's concerns on #202 (CommonsMetadata hcard handling)
Start working on #198 (stats visualization) if it hasn't been picked yet
2014-02-25
Yesterday
Addressed Aaron's and Gergo's follow-up concerns on #146 (show blurry placeholder & progress bar)
Reviewed and approved #177 (consolidate multilightbox) and #253 (wrong repo icon)
Re-tested #118 (show permissions) and found issues with it
Cleaned up Pau's changeset for the new proposed metadata panel animation
Wrote a solution for #202 on the CommonsMetadata side
Today
1-on-1 with Rob
Start working on #198 (stats visualization) if it hasn't been picked yet
2014-02-24
Yesterday
IRC office hour
Re-reviewed and approved #234 (getting rid of data URIs)
Addressed Gergo's concerns on #254 (mmv might interfere with anchor link clicks)
Addressed Aaron's and Gergo's concerns on #146 (show blurry placeholder & progress bar)
Wrote a fix for #258 (lightbox could look broken/image could be too small when opening the viewer)
Today
Design meeting
Solve #202 on the CommonsMetadata side
Start working on #198 (stats visualization) if it hasn't been picked yet
2014-02-21
Yesterday
Weekly meeting
Implemented #254 (mmv interfering with anchor clicks/hash)
Finished writing the truckload of tests for #146 (blurred placeholder + progress bar), up for review
Today
IRC office hour
Review everything in the queue
Solve #202 on the CommonsMetadata side
2014-02-20
Yesterday
Call with Kaltura
Reviewed and accepted a bunch of changesets
Cleaned up everything in Bugzilla and filed relevant cards
Worked out a solution to the Data URI issue with Gergo
Today
Weekly meeting
Write tests for #146 (Blurry thumbnail + progress bar)
Solve #202 on the CommonsMetadata side
2014-02-19
Yesterday
1-on-1 with Rob
Attended the scrum of scrums
Addressed Mark's concerns on #210 (hide mouse cursor when in fullscreen)
Addressed Gergo's concerns on #146 (Blurry thumbnail + progress bar) Tests remain to be written
Today
Review everything in the queue
Write tests for #146
Call with Kaltura
2014-02-18
Yesterday
Implemented #210 (hide mouse cursor when in fullscreen)
Implemented #213 (make metadata animation more subtle and only happen when the user is new to mmv)
Fixed #202 (wikidata author vcard was creating a lot of noise in the metadata panel header)
Implemented #214 (make image and controls unselectable)
Completed all the features of #146 (Blurry thumbnail + progress bar) Tests remain to be written
Today
1-on-1 with Rob
Try to attend the scrum of scrums
Review everything in the queue
Address all concerns on ongoing cards (7 cards for review right now...)
Write tests for #146
2014-02-17
Yesterday
Improved upon Gergo's follow-up work on #211 (Record detailed network performance of all API calls)
Wrote #136 (Lazy-loading mmv) and cleaned up a lot of stuff while in there
Today
Review everything in the queue
Finish #146
2014-02-14
Yesterday
Weekly team meeting
Set up the "Points" custom field in phabricator
Made good progress on #146 (blurry thumbnail placeholder). I now have the blurry thumbnail appearing and animating into focus on browsers that can do so. Left to do: progress bar and tests.
Addressed concerns on #207 (data URIs)
Wrote #211 (detailed network performance on all API calls)
Today
Review everything in the queue
Finish #146
Pick up another card
2014-02-13
Yesterday
Reviewed and accepted remaining changesets on #155 (preload next and prev image)
Fixed the double image loading issue #207, pending review
Fixed the mmv.performance missing dependency, pending review
Started working on #146 (show blurred thumbnail + progress bar while loading)
Spoke to Evan Priestley from Phabricator about replacing Mingle with Phabricator, he pointed me to their beta board page. It seems like the only missing element is the ability to sum story points, which I've filed. We'll probably need to implement it ourselves (shouldn't be hard).
Today
Weekly team meeting
Review everything in the queue
Finish #146
2014-02-12
Day off
2014-02-11
Yesterday
Attended the design meeting
Reviewed 3 changesets for #155 (preload before/after). 1 approved, 2 still have small issues
Reviewed this RfC and commented on it, providing alternative ideas
Implemented #176 (getting rid of lightboxHooks)
Continued Aaron's work on #191 (blurry image + progress bar prototype), recorded this video of it running, for Fabrice to review
The day after tomorrow (I'm off tomorrow)
Wrap up as many remaining cards as possible to finish the iteration
2014-02-10
Yesterday
Approved and tested #196 (bucketing refactor)
Approved and tested #187 (button UI code refactor) and fixed a few merging issues (icons visually gone)
Tested #186 (panel UI code refactor)
Tested #183 (refactored API calls)
Spent a while figuring out how I can grep live EventLogging data, because I couldn't compile Ori's zpubsub. After attempting to install alternatives in several languages unsuccessfully, ended up with a Go port of zmqc, used like this.
Realized that the sampling factor wasn't part of the labs/live config, so there's no data collected yet, sent the appropriate diff to ops, moved the card (#126) to blocked
Reviewed the UI WIP cards sent by Fabrice
Reviewed and left concerns on #155 (preload before/after), re-iterated request to add test coverage to a specific function + minor concern about gender validation
Today
Review everything in the queue
Design meeting
Do #176 (lightboxHooks replacement)
2014-02-07
Yesterday
Watched the metrics meeting recording
Reviewed and accepted #151 (fullscreen metadata)
Tried testing #142 (jsduck as part of vagrant role) but couldn't make it work
Tried completing #126 (gather detailed statistics) but couldn't reproduce nor blind-fix the bug Gergo ran into
Approved #196 (resize calculation refactor) but hasn't merged yet since it has dependencies
Accepted 2 changsets out of 3 on #155 (prev/next preloading), left concerns on the remaining one
Reviewed and made a bunch of additions to #186 (refactor panel UI code)
Today
Review everything in the queue
Review the UI WIP cards sent by Fabrice
Do #176 (lightboxHooks replacement)
2014-02-06
Yesterday
Reviewed #155, 2 changesets out of 3 (next/prev preloading) left concerns about cachebusting due to pixel ratios + where code lives
Rebased #151 (fullscreen metadata reveal) and saw that it still doesn't work in Chrome/OS X