lor.js: race condition при замене якоря на ссылку

+ попытка оптимизировать загрузку
maxcom · Sep 18, 2023 · 9d8ab06 · 9d8ab06
1 parent d7bb879
commit 9d8ab06
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/main/webapp/WEB-INF/jsp/head.jsp b/src/main/webapp/WEB-INF/jsp/head.jsp
@@ -25,6 +25,7 @@
 <html lang=ru>
 <head>
 <link rel="stylesheet" type="text/css" href="/${template.style}/combined.css?MAVEN_BUILD_TIMESTAMP">
+<link rel="preload" href="/js/lor.js?MAVEN_BUILD_TIMESTAMP">
 
 <link rel="yandex-tableau-widget" href="/manifest.json" />
 <meta name="referrer" content="always">

diff --git a/src/main/webapp/js/lor.js b/src/main/webapp/js/lor.js
@@ -302,8 +302,8 @@ function replace_state() {
         // Yes, we are viewing a comment
 
         // exit if no such target
-        if (document.querySelector('article.msg:target') === null) return;
-        
+        if (document.getElementById(document.location.hash.substring(1)) === null) return;
+
         var hash = document.location.hash.split('-');
         if (parseInt(hash[1]) > 0) {
             // OK, comment ID is valid