From b0aa337fd537f5ae914299163c517969a71780ea Mon Sep 17 00:00:00 2001 From: Mark Holmquist Date: Fri, 26 Sep 2014 19:03:14 -0700 Subject: [PATCH] Initialise dialogOpen value to false Because it was undefined, the toggleClass method just turned the class on if it wasn't there in the first place. Silly jQuery, Trix are for kids. Change-Id: I4e21e033529834a4f808270306c72f5756b69ab2 Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/921 --- resources/mmv/ui/mmv.ui.canvas.js | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/resources/mmv/ui/mmv.ui.canvas.js b/resources/mmv/ui/mmv.ui.canvas.js index 2a31b7a3d..73dd08d36 100644 --- a/resources/mmv/ui/mmv.ui.canvas.js +++ b/resources/mmv/ui/mmv.ui.canvas.js @@ -33,6 +33,12 @@ function Canvas( $container, $imageWrapper, $mainWrapper ) { mw.mmv.ui.Element.call( this, $container ); + /** + * @property {boolean} + * @private + */ + this.dialogOpen = false; + /** * @property {mw.mmv.ThumbnailWidthCalculator} * @private