From 16dc26b6e82a26c40afcbb92f4f1bb13f57efa06 Mon Sep 17 00:00:00 2001
From: Jeremy Ruston <jeremy@osmosoft.com>
Date: Tue, 19 Jun 2012 18:21:58 +0100
Subject: [PATCH] Fixed the reveal macro so that is marked dependentAll

---
 core/modules/macros/reveal.js | 52 +++++++++++++++++++----------------
 1 file changed, 28 insertions(+), 24 deletions(-)

diff --git a/core/modules/macros/reveal.js b/core/modules/macros/reveal.js
index d32a3ffa3..455c4a985 100644
--- a/core/modules/macros/reveal.js
+++ b/core/modules/macros/reveal.js
@@ -100,30 +100,7 @@ exports.executeMacro = function() {
 	return child;
 };
 
-exports.refreshInDom = function(changes) {
-	var needChildrenRefresh = true, // Avoid refreshing the children nodes if we don't need to
-		t;
-	// Re-read the open state
-	this.readState();
-	// Render the children if we're open and we don't have any children yet
-	if(this.isOpen && this.child.children.length === 0) {
-		// Install the children and execute them
-		this.child.children = this.content;
-		for(t=0; t<this.child.children.length; t++) {
-			this.child.children[t].execute(this.parents,this.tiddlerTitle);
-			this.child.children[t].renderInDom(this.child.domNode);
-		}
-		needChildrenRefresh = false; // Don't refresh the children if we've just created them
-	}
-	// Refresh the children
-	if(needChildrenRefresh) {
-		for(t=0; t<this.child.children.length; t++) {
-			this.child.children[t].refreshInDom(changes);
-		}
-	}
-	// Set the visibility of the children
-	this.child.domNode.style.display = this.isOpen ? "block" : "none";
-	// Position the content if required
+exports.postRenderInDom = function() {
 	switch(this.params.type) {
 		case "popup":
 			if(this.isOpen) {
@@ -152,4 +129,31 @@ exports.refreshInDom = function(changes) {
 	}
 };
 
+exports.refreshInDom = function(changes) {
+	var needChildrenRefresh = true, // Avoid refreshing the children nodes if we don't need to
+		t;
+	// Re-read the open state
+	this.readState();
+	// Render the children if we're open and we don't have any children yet
+	if(this.isOpen && this.child.children.length === 0) {
+		// Install the children and execute them
+		this.child.children = this.content;
+		for(t=0; t<this.child.children.length; t++) {
+			this.child.children[t].execute(this.parents,this.tiddlerTitle);
+			this.child.children[t].renderInDom(this.child.domNode);
+		}
+		needChildrenRefresh = false; // Don't refresh the children if we've just created them
+	}
+	// Refresh the children
+	if(needChildrenRefresh) {
+		for(t=0; t<this.child.children.length; t++) {
+			this.child.children[t].refreshInDom(changes);
+		}
+	}
+	// Set the visibility of the children
+	this.child.domNode.style.display = this.isOpen ? "block" : "none";
+	// Position the content if required
+	this.postRenderInDom();
+};
+
 })();