From fd7d73e8a74002ebab844df6efb8a845338ea288 Mon Sep 17 00:00:00 2001
From: Stephan Wehner <stephan@buckmaster.ca>
Date: Sat, 5 May 2012 12:33:41 -0700
Subject: [PATCH 1/5] Add env-var FRANK_REFRESH_DIRS

---
 lib/frank/base.rb | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/lib/frank/base.rb b/lib/frank/base.rb
index 1e7deb6..994a3f1 100644
--- a/lib/frank/base.rb
+++ b/lib/frank/base.rb
@@ -269,7 +269,13 @@ def self.new(&block)
 
     builder = Rack::Builder.new do
       use Frank::Middleware::Statik, :root => Frank.static_folder
-      use Frank::Middleware::Refresh, :watch => [ Frank.dynamic_folder, Frank.static_folder, Frank.layouts_folder ]
+      watch_dirs = [ Frank.dynamic_folder, Frank.static_folder, Frank.layouts_folder ]
+      unless ENV['FRANK_REFRESH_DIRS'].nil?
+        watch_dirs.concat ENV['FRANK_REFRESH_DIRS'].split(/,/) unless ENV['FRANK_REFRESH_DIRS'].nil?
+        watch_dirs.uniq!
+        puts "Watching directories #{watch_dirs.join(', ')}"
+      end
+      use Frank::Middleware::Refresh, :watch => watch_dirs
       run base
     end
 

From c863c0689fe90fe9cd336bb01df45571bb4f3b26 Mon Sep 17 00:00:00 2001
From: Stephan Wehner <stephan@buckmaster.ca>
Date: Sat, 5 May 2012 12:34:42 -0700
Subject: [PATCH 2/5] javascript helper refresh: add loading indicator, only
 reload once

---
 lib/frank/template_helpers.rb | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/lib/frank/template_helpers.rb b/lib/frank/template_helpers.rb
index 99a0519..966f750 100644
--- a/lib/frank/template_helpers.rb
+++ b/lib/frank/template_helpers.rb
@@ -49,10 +49,22 @@ def refresh
           <script type="text/javascript">
           (function(){
             var when = #{Time.now.to_i};
+            var loading = false
+            function addLoadingIndicator() {
+              var body = document.getElementsByTagName("body")[0]
+              var div = document.createElement('div')
+              div.innerHTML = 'Change detected. Reloading. Please wait ...'
+              div.setAttribute('style', 'z-index: 1000; position: fixed; top: 0; right: 40%; background: #901010; font-weight: bold; color: white; font-size: 16px; text-align: center; padding-top: 10px; width: 400px; font-family: sans; height: 30px; opacity: 0.8; border: 1px solid #601010')
+              body.appendChild(div)
+            }
 
             function process( raw ){
               if( parseInt(raw) > when ) {
-                window.location.reload();
+                if (!loading) {
+                  loading = true
+                  addLoadingIndicator();
+                  window.location.reload();
+                }
               }
             }
             
@@ -65,7 +77,8 @@ def refresh
               };
               req.open('GET', '/__refresh__', true);
               req.send( null );
-              setTimeout( arguments.callee, 1000 );
+              if (!loading)
+                setTimeout( arguments.callee, 1000 );
             })();
 
           })();

From 27bea22b24b0c96b3722702893b7c529fa600f0f Mon Sep 17 00:00:00 2001
From: Stephan Wehner <stephan@buckmaster.ca>
Date: Sat, 5 May 2012 17:43:59 -0700
Subject: [PATCH 3/5] Loading indicator shows the time of reload, and has a
 Close button

---
 lib/frank/template_helpers.rb | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/lib/frank/template_helpers.rb b/lib/frank/template_helpers.rb
index 966f750..6087404 100644
--- a/lib/frank/template_helpers.rb
+++ b/lib/frank/template_helpers.rb
@@ -53,8 +53,16 @@ def refresh
             function addLoadingIndicator() {
               var body = document.getElementsByTagName("body")[0]
               var div = document.createElement('div')
-              div.innerHTML = 'Change detected. Reloading. Please wait ...'
-              div.setAttribute('style', 'z-index: 1000; position: fixed; top: 0; right: 40%; background: #901010; font-weight: bold; color: white; font-size: 16px; text-align: center; padding-top: 10px; width: 400px; font-family: sans; height: 30px; opacity: 0.8; border: 1px solid #601010')
+              div.innerHTML = '<p style="margin: 2px 0; padding: 2px 0"><div style="cursor: pointer; background: #f00; color: #faa;' + 
+                                             'float: right; margin-right: 5px;width: 1em; height; 1ex"' + 
+                                      'onclick="this.parentNode.style.display=\\'none\\'; return false">x</div>' + 
+                                  'Change detected at ' + new Date() +
+                              '</p><p style="margin: 2px 0; padding: 2px 0">Reloading. Please wait ...</p>'
+              div.setAttribute('style', 'z-index: 1000; position: fixed; top: 0; right: 40%;width: 400px; height: 100px;' +
+                                        'background: #308030; color: white; opacity: 0.8; ' +
+                                        'font-family: sans; font-weight: bold; font-size: 16px;' +
+                                        'text-align: center; padding: 2px 5px 5px 5px' +
+                                        'border: 1px solid #104010')
               body.appendChild(div)
             }
 

From e554b46c8c6b982e1b06aea39dfc860607e3a08b Mon Sep 17 00:00:00 2001
From: Stephan Wehner <stephan@buckmaster.ca>
Date: Tue, 8 May 2012 17:50:59 -0700
Subject: [PATCH 4/5] Add hook to disable refresh check (set
 window.stop_refresh_check=false)

---
 lib/frank/template_helpers.rb | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/lib/frank/template_helpers.rb b/lib/frank/template_helpers.rb
index 6087404..9995813 100644
--- a/lib/frank/template_helpers.rb
+++ b/lib/frank/template_helpers.rb
@@ -56,13 +56,13 @@ def refresh
               div.innerHTML = '<p style="margin: 2px 0; padding: 2px 0"><div style="cursor: pointer; background: #f00; color: #faa;' + 
                                              'float: right; margin-right: 5px;width: 1em; height; 1ex"' + 
                                       'onclick="this.parentNode.style.display=\\'none\\'; return false">x</div>' + 
-                                  'Change detected at ' + new Date() +
+                                  'Frank has detected changes at ' + new Date() +
                               '</p><p style="margin: 2px 0; padding: 2px 0">Reloading. Please wait ...</p>'
-              div.setAttribute('style', 'z-index: 1000; position: fixed; top: 0; right: 40%;width: 400px; height: 100px;' +
+              div.setAttribute('style', 'z-index: 1000; position: fixed; top: 0; right: 40%;width: 460px; height: 110px;' +
                                         'background: #308030; color: white; opacity: 0.8; ' +
                                         'font-family: sans; font-weight: bold; font-size: 16px;' +
                                         'text-align: center; padding: 2px 5px 5px 5px' +
-                                        'border: 1px solid #104010')
+                                        'border: 10px solid #104010')
               body.appendChild(div)
             }
 
@@ -85,7 +85,7 @@ def refresh
               };
               req.open('GET', '/__refresh__', true);
               req.send( null );
-              if (!loading)
+              if (!loading && !window.stop_refresh_check)
                 setTimeout( arguments.callee, 1000 );
             })();
 

From 0449f9e2fba0548537f4e7bf28cd7a41a2854ef0 Mon Sep 17 00:00:00 2001
From: Stephan Wehner <stephan@buckmaster.ca>
Date: Tue, 8 May 2012 21:47:26 -0700
Subject: [PATCH 5/5] Add specs for refresh javascript / loading indicator /
 stop automatic refresh

---
 spec/template_helpers_spec.rb | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/spec/template_helpers_spec.rb b/spec/template_helpers_spec.rb
index c21e460..570c5d4 100644
--- a/spec/template_helpers_spec.rb
+++ b/spec/template_helpers_spec.rb
@@ -27,6 +27,16 @@ def app
     template = @app.render('refresh.haml')
     template.should include("<script type=\"text/javascript\">\n            (function(){")
   end
+
+  it 'should have provision for a loading indicator in the refresh javascript' do
+    template = @app.render('refresh.haml')
+    template.should include("addLoadingIndicator()")
+  end
+
+  it 'should have provision for stopping automatic refresh in the refresh javascript' do
+    template = @app.render('refresh.haml')
+    template.should include("!window.stop_refresh_check")
+  end
   
   it 'renders content_for haml in the layout' do
     template = @app.render('content_for_haml.haml')