From 5127ff9e9f791d22d44d75aadf799ecdcbfa61b5 Mon Sep 17 00:00:00 2001
From: James Campos <james.r.campos@gmail.com>
Date: Wed, 20 Jul 2011 14:42:12 -0700
Subject: [PATCH] rm ui.el in ui.hoverend, use it

---
 4chan_x.user.js | 74 +++++++++++++++++++++++--------------------------
 script.coffee   | 35 +++++++++++------------
 2 files changed, 50 insertions(+), 59 deletions(-)

diff --git a/4chan_x.user.js b/4chan_x.user.js
index 6238582dc..217bd00f4 100644
--- a/4chan_x.user.js
+++ b/4chan_x.user.js
@@ -268,7 +268,7 @@
       }
     },
     hoverend: function(e) {
-      return ui.el.style.top = '999%';
+      return ui.el.parentNode.removeChild(ui.el);
     }
   };
   /*
@@ -2122,40 +2122,31 @@
   };
   quotePreview = {
     init: function() {
-      var preview;
-      g.callbacks.push(quotePreview.node);
-      preview = $.el('div', {
+      return g.callbacks.push(function(root) {
+        var quote, _i, _len, _ref, _results;
+        _ref = $$('a.quotelink, a.backlink', root);
+        _results = [];
+        for (_i = 0, _len = _ref.length; _i < _len; _i++) {
+          quote = _ref[_i];
+          if (!quote.hash) {
+            continue;
+          }
+          $.bind(quote, 'mouseover', quotePreview.mouseover);
+          $.bind(quote, 'mousemove', ui.hover);
+          _results.push($.bind(quote, 'mouseout', quotePreview.mouseout));
+        }
+        return _results;
+      });
+    },
+    mouseover: function(e) {
+      var el, id, qp, quote, replyID, threadID, _i, _len, _ref, _results;
+      qp = $.el('div', {
         id: 'qp',
         className: 'replyhl'
       });
-      return $.append(d.body, preview);
-    },
-    node: function(root) {
-      var quote, _i, _len, _ref, _results;
-      _ref = $$('a.quotelink, a.backlink', root);
-      _results = [];
-      for (_i = 0, _len = _ref.length; _i < _len; _i++) {
-        quote = _ref[_i];
-        if (!quote.hash) {
-          continue;
-        }
-        $.bind(quote, 'mouseover', quotePreview.mouseover);
-        $.bind(quote, 'mousemove', ui.hover);
-        _results.push($.bind(quote, 'mouseout', quotePreview.mouseout));
-      }
-      return _results;
-    },
-    mouseout: function() {
-      var el;
-      if (el = d.getElementById(this.hash.slice(1))) {
-        $.removeClass(el, 'qphl');
-      }
-      return ui.hoverend();
-    },
-    mouseover: function(e) {
-      var el, id, qp, quote, replyID, threadID, _i, _len, _ref;
+      $.append(d.body, qp);
+      ui.el = qp;
       id = this.hash.slice(1);
-      qp = $('#qp');
       if (el = d.getElementById(id)) {
         qp.innerHTML = el.innerHTML;
         if ($.config('Quote Highlighting')) {
@@ -2164,21 +2155,27 @@
         if (/backlink/.test(this.className)) {
           replyID = $.x('ancestor::*[@id][1]', this).id.match(/\d+/)[0];
           _ref = $$('a.quotelink', qp);
+          _results = [];
           for (_i = 0, _len = _ref.length; _i < _len; _i++) {
             quote = _ref[_i];
-            if (quote.hash.slice(1) === replyID) {
-              quote.className = 'forwardlink';
-            }
+            _results.push(quote.hash.slice(1) === replyID ? quote.className = 'forwardlink' : void 0);
           }
+          return _results;
         }
       } else {
         qp.innerHTML = "Loading " + id + "...";
         threadID = this.pathname.split('/').pop() || $.x('ancestor::div[@class="thread"]/div', this).id;
-        $.cache(this.pathname, (function() {
+        return $.cache(this.pathname, (function() {
           return quotePreview.parse(this, id, threadID);
         }));
       }
-      return ui.el = qp;
+    },
+    mouseout: function() {
+      var el;
+      if (el = d.getElementById(this.hash.slice(1))) {
+        $.removeClass(el, 'qphl');
+      }
+      return ui.hoverend();
     },
     parse: function(req, id, threadID) {
       var body, html, op, qp, reply, _i, _len, _ref;
@@ -2452,7 +2449,7 @@
         }
         $.bind(thumb, 'mouseover', imgHover.mouseover);
         $.bind(thumb, 'mousemove', ui.hover);
-        return $.bind(thumb, 'mouseout', imgHover.mouseout);
+        return $.bind(thumb, 'mouseout', ui.hoverend);
       });
     },
     mouseover: function(e) {
@@ -2463,9 +2460,6 @@
       });
       ui.el = img;
       return $.append(d.body, img);
-    },
-    mouseout: function(e) {
-      return $.rm(ui.el);
     }
   };
   imgPreloading = {
diff --git a/script.coffee b/script.coffee
index f7139490d..38437def7 100644
--- a/script.coffee
+++ b/script.coffee
@@ -182,7 +182,7 @@ ui =
       el.style.right = clientWidth - clientX + 45
 
   hoverend: (e) ->
-    ui.el.style.top = '999%'
+    ui.el.parentNode.removeChild ui.el
 
 ###
 loosely follows the jquery api:
@@ -1651,23 +1651,20 @@ quoteInline =
 
 quotePreview =
   init: ->
-    g.callbacks.push quotePreview.node
-    preview = $.el 'div',
+    g.callbacks.push (root) ->
+      for quote in $$ 'a.quotelink, a.backlink', root
+        continue unless quote.hash
+        $.bind quote, 'mouseover', quotePreview.mouseover
+        $.bind quote, 'mousemove', ui.hover
+        $.bind quote, 'mouseout',  quotePreview.mouseout
+  mouseover: (e) ->
+    qp = $.el 'div',
       id: 'qp'
       className: 'replyhl'
-    $.append d.body, preview
-  node: (root) ->
-    for quote in $$ 'a.quotelink, a.backlink', root
-      continue unless quote.hash
-      $.bind quote, 'mouseover', quotePreview.mouseover
-      $.bind quote, 'mousemove', ui.hover
-      $.bind quote, 'mouseout',  quotePreview.mouseout
-  mouseout: ->
-    $.removeClass el, 'qphl' if el = d.getElementById @hash[1..]
-    ui.hoverend()
-  mouseover: (e) ->
+    $.append d.body, qp
+    ui.el = qp
+
     id = @hash[1..]
-    qp = $ '#qp'
     if el = d.getElementById id
       qp.innerHTML = el.innerHTML
       $.addClass el, 'qphl' if $.config 'Quote Highlighting'
@@ -1680,7 +1677,9 @@ quotePreview =
       qp.innerHTML = "Loading #{id}..."
       threadID = @pathname.split('/').pop() or $.x('ancestor::div[@class="thread"]/div', @).id
       $.cache @pathname, (-> quotePreview.parse @, id, threadID)
-    ui.el = qp
+  mouseout: ->
+    $.removeClass el, 'qphl' if el = d.getElementById @hash[1..]
+    ui.hoverend()
   parse: (req, id, threadID) ->
     qp = $ '#qp'
     return unless qp.innerHTML is "Loading #{id}..."
@@ -1851,15 +1850,13 @@ imgHover =
       return unless thumb = $ 'img[md5]', root
       $.bind thumb, 'mouseover', imgHover.mouseover
       $.bind thumb, 'mousemove', ui.hover
-      $.bind thumb, 'mouseout',  imgHover.mouseout
+      $.bind thumb, 'mouseout',  ui.hoverend
   mouseover: (e) ->
     img = $.el 'img'
       id: 'iHover'
       src: @parentNode.href
     ui.el = img
     $.append d.body, img
-  mouseout: (e) ->
-    $.rm ui.el
 
 imgPreloading =
   init: ->