Changeset 16456 for util

Timestamp:

05/19/07 10:33:17 (18 months ago)

Author:

Aankhen

Message:

util/smartlinks.pl (JS):
* combined a couple conditionals.
* moved the location.hash bit outside the loop.
* fixed the line anchors!

Files:

• util/smartlinks.pl (modified) (3 diffs)

util/smartlinks.pl

@@ -165 +165 @@
   for (var i = 0, j = divs.length; i < j; i++) {
     var curr = divs[i];
-    if (!curr.id) { continue; }
-    if (curr.id.match(/smartlink_(\d+)/)) {
+    if (curr.id && curr.id.match(/smartlink_(\d+)/)) {
       var num = RegExp.$1;
 
@@ -202 +201 @@
       curr.parentNode.insertBefore(p, curr);
       curr.style.display = 'none';
-
-      // explicitly jump to the page anchor (if any) since the code above messes it up
-      if (location.hash.match(/#.+/)) location.hash = RegExp.lastMatch;
-    }
+    }
+
+    // explicitly jump to the page anchor (if any) since the code above messes it up
+    if (location.hash && location.hash.match(/#.+/)) location.hash = RegExp.lastMatch;
   }
 });
@@ -454 +453 @@
     my $html = '';
     for ($curr .. $list->[0] - 1) {
-        $html .= qq{<a id"_line_$_"></a>\n};
+        $html .= qq{<a id="_line_$_"></a>\n};
     }
     $html;