From 72f74b7f6e75418fa86aff64ee4d984bcef5bb8e Mon Sep 17 00:00:00 2001
From: str4d <str4d@mail.i2p>
Date: Tue, 24 Jul 2012 13:15:11 +0000
Subject: [PATCH] Disabled the iframe fallback for the summary bar

HTML compliance wins out over summary bar refresh for users with JS disabled,
for now - get user feedback after release.
---
 apps/routerconsole/jsp/summary.jsi | 6 +++---
 history.txt                        | 2 ++
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/apps/routerconsole/jsp/summary.jsi b/apps/routerconsole/jsp/summary.jsi
index 6a388b8172..d38637d836 100644
--- a/apps/routerconsole/jsp/summary.jsi
+++ b/apps/routerconsole/jsp/summary.jsi
@@ -23,7 +23,7 @@
             // update disable boolean
             intl.setDisableRefresh(d);
         }
-        if (!intl.getDisableRefresh())
+        if (false && !intl.getDisableRefresh())
             out.print("<noscript><iframe src=\"/summaryframe.jsp" + newDelay + "\" height=\"1500\" width=\"200\" scrolling=\"auto\" frameborder=\"0\" title=\"sidepanel\"></noscript>\n");
     }
 %>
@@ -42,9 +42,9 @@
     }
 
     // d and allowIFrame defined above
-    if (!intl.getDisableRefresh()) {
+    if (false && !intl.getDisableRefresh()) {
         out.print("</div><noscript></iframe></noscript>\n");
-    } else if (allowIFrame) {
+    } else if (false && allowIFrame) {
         // since we don't have an iframe this will reload the base page, and
         // the new delay will be passed to the iframe above
         out.print("<noscript><div class=\"refresh\"><form action=\"" + request.getRequestURI() + "\" method=\"POST\">\n" +
diff --git a/history.txt b/history.txt
index 777aab456d..b908f28253 100644
--- a/history.txt
+++ b/history.txt
@@ -1,4 +1,6 @@
 2012-07-24 str4d
+ * Summary bar: disable fallback iframe in summary.jsi (so when JS is disabled
+   the summary bar will not refresh at all)
  * Home page:
    - Replaced 16x16 icons with 32x32 ones
    - Updated icon for git.repo.i2p