From afd9830168b6e2140b418230f13f268599d01b7d Mon Sep 17 00:00:00 2001
From: Nicolas Stepien <mayhemydg@gmail.com>
Date: Sat, 17 Sep 2011 22:31:19 +0200
Subject: [PATCH] Go deeper, shave lines.

---
 4chan_x.user.js | 38 ++++++--------------------------------
 script.coffee   | 15 +++++----------
 2 files changed, 11 insertions(+), 42 deletions(-)

diff --git a/4chan_x.user.js b/4chan_x.user.js
index 366f09911..37c71aea1 100644
--- a/4chan_x.user.js
+++ b/4chan_x.user.js
@@ -199,26 +199,12 @@
   };
   ui = {
     dialog: function(id, position, html) {
-      var el, left, saved, top;
+      var el, saved;
       el = d.createElement('div');
       el.className = 'reply dialog';
       el.innerHTML = html;
       el.id = id;
-      if (saved = localStorage["" + NAMESPACE + id + ".position"]) {
-        el.style.cssText = saved;
-      } else {
-        left = position.left, top = position.top;
-        if (left) {
-          el.style.left = left;
-        } else {
-          el.style.right = 0;
-        }
-        if (top) {
-          el.style.top = top;
-        } else {
-          el.style.bottom = 0;
-        }
-      }
+      el.style.cssText = (saved = localStorage["" + NAMESPACE + id + ".position"]) ? saved : position;
       el.querySelector('div.move').addEventListener('mousedown', ui.dragstart, false);
       return el;
     },
@@ -1363,10 +1349,7 @@
       THREAD_ID = g.THREAD_ID || $.x('ancestor::div[@class="thread"]/div', link).id;
       qr.challenge = $('#recaptcha_challenge_field').value;
       html = "      <a id=close title=close>X</a>      <input type=checkbox id=autohide title=autohide>      <div class=move>        <input class=inputtext type=text name=name value='" + name + "' placeholder=Name form=qr_form>        Quick Reply      </div>      <div class=autohide>        <form name=post action=http://sys.4chan.org/" + g.BOARD + "/post method=POST enctype=multipart/form-data target=iframe id=qr_form>          <input type=hidden name=resto value=" + THREAD_ID + ">          <input type=hidden name=mode value=regist>          <input type=hidden name=recaptcha_challenge_field id=recaptcha_challenge_field>          <input type=hidden name=recaptcha_response_field id=recaptcha_response_field>          <div><input class=inputtext type=text name=email value='" + email + "' placeholder=E-mail>" + qr.spoiler + "</div>          <div><input class=inputtext type=text name=sub placeholder=Subject><input type=submit value=" + submitValue + " id=com_submit " + submitDisabled + "><label><input type=checkbox id=auto>auto</label></div>          <div><textarea class=inputtext name=com placeholder=Comment></textarea></div>          <div><img src=http://www.google.com/recaptcha/api/image?c=" + qr.challenge + "></div>          <div><input class=inputtext type=text autocomplete=off placeholder=Verification id=dummy><span id=captchas>" + ($.get('captchas', []).length) + " captchas</span></div>          <div><input type=file name=upfile accept='" + qr.acceptFiles + "'></div>        </form>        <div id=files></div>        <div><input class=inputtext type=password name=pwd value='" + pwd + "' placeholder=Password form=qr_form maxlength=8><a id=attach>attach another file</a></div>      </div>      <a id=error class=error></a>      ";
-      qr.el = ui.dialog('qr', {
-        top: '0px',
-        left: '0px'
-      }, html);
+      qr.el = ui.dialog('qr', 'top: 0; left: 0;', html);
       $.bind($('input[name=name]', qr.el), 'mousedown', function(e) {
         return e.stopPropagation();
       });
@@ -1744,10 +1727,7 @@
       }
       checked = conf['Auto Update'] ? 'checked' : '';
       html += "      <div><label title='Controls whether *this* thread automatically updates or not'>Auto Update This<input name='Auto Update This' type=checkbox " + checked + "></label></div>      <div><label>Interval (s)<input name=Interval value=" + conf['Interval'] + " type=text></label></div>      <div><input value='Update Now' type=button></div>";
-      dialog = ui.dialog('updater', {
-        bottom: '0',
-        right: '0'
-      }, html);
+      dialog = ui.dialog('updater', 'bottom: 0; right: 0;', html);
       updater.count = $('#count', dialog);
       updater.timer = $('#timer', dialog);
       updater.br = $('br[clear]');
@@ -1875,10 +1855,7 @@
     init: function() {
       var favicon, html, input, inputs, _i, _len;
       html = '<div class=move>Thread Watcher</div>';
-      watcher.dialog = ui.dialog('watcher', {
-        top: '50px',
-        left: '0px'
-      }, html);
+      watcher.dialog = ui.dialog('watcher', 'top: 50px; left: 0px;', html);
       $.add(d.body, watcher.dialog);
       inputs = $$('.op input');
       for (_i = 0, _len = inputs.length; _i < _len; _i++) {
@@ -2474,10 +2451,7 @@
       threadStats.posts = 1;
       threadStats.images = $('.op img[md5]') ? 1 : 0;
       html = "<div class=move><span id=postcount>" + threadStats.posts + "</span> / <span id=imagecount>" + threadStats.images + "</span></div>";
-      dialog = ui.dialog('stats', {
-        bottom: '0px',
-        left: '0px'
-      }, html);
+      dialog = ui.dialog('stats', 'bottom: 0; left: 0;', html);
       dialog.className = 'dialog';
       threadStats.postcountEl = $('#postcount', dialog);
       threadStats.imagecountEl = $('#imagecount', dialog);
diff --git a/script.coffee b/script.coffee
index e0ce98f59..1698fe712 100644
--- a/script.coffee
+++ b/script.coffee
@@ -126,12 +126,7 @@ ui =
     el.className = 'reply dialog'
     el.innerHTML = html
     el.id = id
-    if saved = localStorage["#{NAMESPACE}#{id}.position"]
-      el.style.cssText = saved
-    else
-      {left, top} = position
-      if left then el.style.left = left else el.style.right  = 0
-      if top  then el.style.top  = top  else el.style.bottom = 0
+    el.style.cssText = if saved = localStorage["#{NAMESPACE}#{id}.position"] then saved else position
     el.querySelector('div.move').addEventListener 'mousedown', ui.dragstart, false
     el
   dragstart: (e) ->
@@ -1073,7 +1068,7 @@ qr =
       </div>
       <a id=error class=error></a>
       "
-    qr.el = ui.dialog 'qr', top: '0px', left: '0px', html
+    qr.el = ui.dialog 'qr', 'top: 0; left: 0;', html
 
     $.bind $('input[name=name]',   qr.el), 'mousedown', (e) -> e.stopPropagation()
     $.bind $('input[name=upfile]', qr.el), 'change', qr.validateFileSize
@@ -1393,7 +1388,7 @@ updater =
       <div><label>Interval (s)<input name=Interval value=#{conf['Interval']} type=text></label></div>
       <div><input value='Update Now' type=button></div>"
 
-    dialog = ui.dialog 'updater', bottom: '0', right: '0', html
+    dialog = ui.dialog 'updater', 'bottom: 0; right: 0;', html
 
     updater.count = $ '#count', dialog
     updater.timer = $ '#timer', dialog
@@ -1501,7 +1496,7 @@ updater =
 watcher =
   init: ->
     html = '<div class=move>Thread Watcher</div>'
-    watcher.dialog = ui.dialog 'watcher', top: '50px', left: '0px', html
+    watcher.dialog = ui.dialog 'watcher', 'top: 50px; left: 0px;', html
     $.add d.body, watcher.dialog
 
     #add watch buttons
@@ -1885,7 +1880,7 @@ threadStats =
     threadStats.posts = 1
     threadStats.images = if $ '.op img[md5]' then 1 else 0
     html = "<div class=move><span id=postcount>#{threadStats.posts}</span> / <span id=imagecount>#{threadStats.images}</span></div>"
-    dialog = ui.dialog 'stats', bottom: '0px', left: '0px', html
+    dialog = ui.dialog 'stats', 'bottom: 0; left: 0;', html
     dialog.className = 'dialog'
     threadStats.postcountEl  = $ '#postcount',  dialog
     threadStats.imagecountEl = $ '#imagecount', dialog