From 3f4b01dd8ba861b3b173de720707b1a7c70e7ee8 Mon Sep 17 00:00:00 2001
From: James Campos <james.r.campos@gmail.com>
Date: Sun, 25 Sep 2011 19:52:03 -0700
Subject: [PATCH] open file dialog when file button clicked on chrom

---
 4chan_x.user.js | 46 ++++++++++++++++++++++++++++++++++------------
 script.coffee   | 34 ++++++++++++++++++++++++----------
 2 files changed, 58 insertions(+), 22 deletions(-)

diff --git a/4chan_x.user.js b/4chan_x.user.js
index 4990a0bf4..b86e3f6fc 100644
--- a/4chan_x.user.js
+++ b/4chan_x.user.js
@@ -1324,18 +1324,17 @@
         });
       }
     },
-    attach: function() {
-      var box, file, files;
+    attach: function(file) {
+      var box, files;
       files = $('#files', QR.qr);
       box = $.el('li', {
-        innerHTML: "<img><input type=file name=upfile accept='" + QR.accept + "'><a class=x>X</a>"
+        innerHTML: "<img><a class=x>X</a>"
       });
-      file = $('input', box);
-      $.bind(file, 'change', QR.change);
       $.bind($('.x', box), 'click', QR.rmThumb);
+      $.add(box, file);
       $.add(files, box);
-      file.click();
-      return QR.stats();
+      QR.stats();
+      return QR.foo();
     },
     rmThumb: function() {
       $.rm(this.parentNode);
@@ -1392,15 +1391,16 @@
       return window.location = 'javascript:Recaptcha.reload()';
     },
     change: function(e) {
-      var file, fr, img, qr;
+      var file, fr, img;
       file = this.files[0];
       if (file.size > QR.MAX_FILE_SIZE) {
         alert('Error: File too large.');
-        $.rm(this.parentNode);
-        QR.attach();
+        QR.foo(this);
         return;
       }
-      qr = QR.qr;
+      if (this.previousSibling.nodeName === 'BUTTON') {
+        QR.attach(this);
+      }
       fr = new FileReader();
       img = $('img', this.parentNode);
       fr.onload = function(e) {
@@ -1437,6 +1437,20 @@
         }
       }
     },
+    foo: function(old) {
+      var input;
+      input = $.el('input', {
+        type: 'file',
+        name: 'upfile',
+        accept: QR.accept
+      });
+      $.bind(input, 'change', QR.change);
+      if (old) {
+        return $.replace(old, file);
+      } else {
+        return $.after($('button', QR.qr), input);
+      }
+    },
     dialog: function(text, tid) {
       var c, l, m, qr, ta;
       if (text == null) {
@@ -1452,7 +1466,10 @@
       if (conf['Cooldown']) {
         QR.cooldown();
       }
-      $.bind($('button', qr), 'click', QR.attach);
+      QR.foo();
+      $.bind($('button', qr), 'click', function() {
+        return this.nextSibling.click();
+      });
       $.bind($('.close', qr), 'click', QR.close);
       $.bind($('form', qr), 'submit', QR.submit);
       $.bind($('#recaptcha_response_field', qr), 'keydown', QR.keydown);
@@ -3261,6 +3278,11 @@
       #qr input[name=resto] {\
         width: 80px;\
       }\
+      #qr button + input {\
+        position: absolute;\
+        opacity: 0;\
+        pointer-events: none;\
+      }\
     '
   };
   if (d.body) {
diff --git a/script.coffee b/script.coffee
index db0563c84..34a629c2b 100644
--- a/script.coffee
+++ b/script.coffee
@@ -976,17 +976,15 @@ QR =
         $('#autohide', QR.qr).checked = true
     if conf['Cooldown']
       $.bind window, 'storage', (e) -> QR.cooldown() if e.key is "#{NAMESPACE}cooldown/#{g.BOARD}"
-  attach: ->
-    #$('#autopost', QR.qr).checked = true
+  attach: (file) ->
     files = $ '#files', QR.qr
     box = $.el 'li',
-      innerHTML: "<img><input type=file name=upfile accept='#{QR.accept}'><a class=x>X</a>"
-    file = $ 'input', box
-    $.bind file, 'change', QR.change
+      innerHTML: "<img><a class=x>X</a>"
     $.bind $('.x', box), 'click', QR.rmThumb
+    $.add box, file
     $.add files, box
-    file.click()
     QR.stats()
+    QR.foo()
   rmThumb: ->
     $.rm @parentNode
     QR.stats()
@@ -1029,10 +1027,10 @@ QR =
     file = @files[0]
     if file.size > QR.MAX_FILE_SIZE
       alert 'Error: File too large.'
-      $.rm @parentNode
-      QR.attach()
+      QR.foo @
       return
-    {qr} = QR
+    if @previousSibling.nodeName is 'BUTTON'
+      QR.attach @
     fr = new FileReader()
     img = $ 'img', @parentNode
     fr.onload = (e) ->
@@ -1057,6 +1055,16 @@ QR =
         textContent: 'Submit'
         disabled: false
       QR.submit() if $('#autopost', QR.qr).checked
+  foo: (old) ->
+    input = $.el 'input',
+      type: 'file'
+      name: 'upfile'
+      accept: QR.accept
+    $.bind input, 'change', QR.change
+    if old
+      $.replace old, file
+    else
+      $.after $('button', QR.qr), input
   dialog: (text='', tid) ->
     tid or= g.THREAD_ID or ''
     QR.qr = qr = ui.dialog 'qr', 'top: 0; right: 0;', "
@@ -1100,7 +1108,8 @@ QR =
     $('[name=pwd]', qr).value   = if m = c.match(/4chan_pass=([^;]+)/)  then decodeURIComponent m[1] else $('input[name=pwd]').value
     $('textarea', qr).value = text
     QR.cooldown() if conf['Cooldown']
-    $.bind $('button', qr), 'click', QR.attach
+    QR.foo()
+    $.bind $('button', qr), 'click', -> @nextSibling.click()
     $.bind $('.close', qr), 'click', QR.close
     $.bind $('form', qr), 'submit', QR.submit
     $.bind $('#recaptcha_response_field', qr), 'keydown', QR.keydown
@@ -2522,6 +2531,11 @@ Main =
       #qr input[name=resto] {
         width: 80px;
       }
+      #qr button + input {
+        position: absolute;
+        opacity: 0;
+        pointer-events: none;
+      }
     '
 
 if d.body