From 3b6810eeec73bc58272f22b12f0a2350544a2f6a Mon Sep 17 00:00:00 2001 From: Timo Tijhof Date: Sat, 27 Jul 2013 03:29:44 +0200 Subject: [PATCH] ce: Don't set backgroundColor transparent for every node in debug mode This only affects debug mode, but things look broken when the background of every single node is set to transparent (e.g. a
 looks weird with a white instead of grey background).

It also leaves the DOM dirty full of inline styles.

Though setting a grey background isn't guaranteed to be visible
either, and all of these redraws and stuff really slow things
down (we should perhaps only start doing these after the initial
document is painted).. the least we can do is undo it and not
leave it there.

Change-Id: I9abfd46765914828ad8618748be5716a8c6b185c
---
 modules/ve/ce/ve.ce.ContentBranchNode.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/ve/ce/ve.ce.ContentBranchNode.js b/modules/ve/ce/ve.ce.ContentBranchNode.js
index 20f6ec7b75..06d864cc07 100644
--- a/modules/ve/ce/ve.ce.ContentBranchNode.js
+++ b/modules/ve/ce/ve.ce.ContentBranchNode.js
@@ -183,7 +183,7 @@ ve.ce.ContentBranchNode.prototype.renderContents = function () {
 	if ( ve.debug ) {
 		this.$.css( 'backgroundColor', '#F6F6F6' );
 		setTimeout( ve.bind( function () {
-			this.$.css( 'backgroundColor', 'transparent' );
+			this.$.css( 'backgroundColor', '' );
 		}, this ), 350 );
 	}
 };