[01:52:56] tgr: I think apsdehal is seeing something I've seen, but I'm not sure it's actually a bug...the close icon *does* show up [10:31:33] (03PS1) 10Apsdehal: Slideshow Functionality [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/107023 [10:32:40] (03CR) 10jenkins-bot: [V: 04-1] Slideshow Functionality [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/107023 (owner: 10Apsdehal) [10:44:37] (03PS2) 10Apsdehal: Slideshow Functionality [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/107023 [20:46:20] (03CR) 10MarkTraceur: "Patch incoming! :)" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/106846 (owner: 10MarkTraceur) [20:46:31] (03PS4) 10MarkTraceur: Refactor to a data model [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/106846 [20:46:38] Woot, so glad I found time for that [20:46:59] (03CR) 10jenkins-bot: [V: 04-1] Refactor to a data model [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/106846 (owner: 10MarkTraceur) [20:47:03] ...bah [20:47:19] Ah, right [20:47:52] (03PS5) 10MarkTraceur: Refactor to a data model [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/106846 [20:48:19] (03CR) 10jenkins-bot: [V: 04-1] Refactor to a data model [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/106846 (owner: 10MarkTraceur) [20:50:44] (03PS6) 10MarkTraceur: Refactor to a data model [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/106846 [20:52:26] (03PS2) 10MarkTraceur: Adding first pass at jsduck docs [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/106848 [20:52:28] (03CR) 10jenkins-bot: [V: 04-1] Adding first pass at jsduck docs [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/106848 (owner: 10MarkTraceur) [20:52:34] (03PS2) 10MarkTraceur: Add main class to jsduck [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/106849 [20:52:42] (03PS3) 10MarkTraceur: Adding first pass at jsduck docs [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/106848 [20:52:46] (03PS3) 10MarkTraceur: Add main class to jsduck [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/106849 [20:52:52] All the rebase [20:53:02] All rebase, rebase, are belong to us [20:59:13] (03CR) 10MarkTraceur: [C: 04-1] Added test if Lightbox is attached (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/106889 (owner: 10Apsdehal) [21:16:01] (03CR) 10MarkTraceur: [C: 04-1] "Some thoughts" (0310 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/107023 (owner: 10Apsdehal)