From 7fb910bf216eee45fa7bcada5ab580d39f00bcb8 Mon Sep 17 00:00:00 2001
From: Thomas Gelf <thomas@gelf.net>
Date: Mon, 26 May 2014 13:46:14 +0000
Subject: [PATCH] layout/menu: don't show uncollapsed loading menu

We have a no-JS-friendly CSS showing the menu uncollapsed unless JS
is fully loaded. Even if the page loads pretty fast this looks still
ugly in JS-enabled browsers.

This patch adds a JS-snippet removing the no-js class from the HTML
element before the rendering engine reaches the body. You'll see no
more flickering in your JS-enabled browser.

fixes #6307
---
 application/layouts/scripts/layout.phtml | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/application/layouts/scripts/layout.phtml b/application/layouts/scripts/layout.phtml
index c527edbd5..5c91b4b48 100644
--- a/application/layouts/scripts/layout.phtml
+++ b/application/layouts/scripts/layout.phtml
@@ -30,6 +30,12 @@ $isIframe = isset($_GET['_render']) && $_GET['_render'] === 'iframe';
   <!-- TODO: viewport and scale settings make no sense for us, fix this -->
   <meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no">
   <link rel="stylesheet" href="<?= $this->href($cssfile) ?>" media="screen" type="text/css" />
+  <script type="text/javascript">
+  (function() {
+    var html = document.getElementsByTagName('html')[0];
+    html.className = html.className.replace(/no-js/, 'js');
+  }());
+  </script>
   <? if ($isIframe): ?>
   <base target="_parent"/>
   <? endif ?>