Luci URL changes (token removal) break Material theme about luci-theme-material

[hnyman]

The issue is apparently using the existence of ";" in the URL to sniff if the user is logged in or still at the login screen. In the new scheme there is no ";" in the URL, so Material gets it wrong.
https://github.com/LuttyYang/luci-theme-material/blob/master/files/htdocs/js/script.js#L147

I made a small patch in a live router to /www/luci-static/material/js/script.js . I eliminated the sniffing for ";". This works for me otherwise ok, but there is an empty left menu visible at the login screen:

root@OpenWrt:/usr/lib/lua/luci# diff /rom/www/luci-static/material/js/script.js
/www/luci-static/material/js/script.js
--- /rom/www/luci-static/material/js/script.js
+++ /www/luci-static/material/js/script.js
@@ -145,9 +145,6 @@
         var ret = false;
         var getUrlNode = function (href){
             var linkPos = href.indexOf(";");
-            if (linkPos == -1){
-                return "login";
-            }else{
                 linkPos = href.indexOf("/", linkPos);
                 if (linkPos == -1){
                     return "overview";
@@ -158,7 +155,6 @@
                     else
                         return link;
                 }
-            }
         };

         var currentNode = getUrlNode(window.location.pathname);

[hnyman]

Jow implemented a fix at the Openwrt Luci repo:
openwrt/luci@f9e590a

[LuttyYang]

I'm busy these days, forgive me for not fixing it in time.

I will merger it latter, thanks

[LuttyYang]

I fix it in version 0.2.13, I created a PR to fix it.