From 06e33799b778c15f99e7aff65375cc7e6dcecee8 Mon Sep 17 00:00:00 2001
From: Alon Levy <alon@pobox.com>
Date: Wed, 11 Feb 2015 18:47:34 +0200
Subject: client/view/completer: hide on click outside. this is complex! fixes
 #200

specifically:
1. take care to remove handler on any hide condition, not click as well
2. previous patch to avoid needlessly hiding completer on showing it
after clicking (we could change that behaviour but not as a unexpected
consequence).
---
 src/client/view/completer.js | 20 +++++++++++++++++---
 1 file changed, 17 insertions(+), 3 deletions(-)

(limited to 'src/client')

diff --git a/src/client/view/completer.js b/src/client/view/completer.js
index c75ee93e..66b94ed4 100644
--- a/src/client/view/completer.js
+++ b/src/client/view/completer.js
@@ -1,6 +1,6 @@
 define(
-['jquery', 'Bacon'],
-function($, Bacon) {
+['jquery', 'Bacon', 'util'],
+function($, Bacon,   util) {
 
 function unquoted(name)
 {
@@ -27,6 +27,7 @@ function setCaret(e, num)
 var completer = (function (input_element, dropdown, base_config) {
     var config = get_config(base_config),
         dropdown_raw = dropdown[0],
+        dropdown_visible = false,
         options_bus = new Bacon.Bus(),
         options = [],
         selected_index = -1,
@@ -35,6 +36,9 @@ var completer = (function (input_element, dropdown, base_config) {
         completion_end = 0,
         minimum_length = 1;
 
+    // we need an identifier to remove callbacks without affecting other completers
+    util.assert(input_element_raw.id !== '');
+
     // turn off the browser's autocomplete
     input_element.attr('autocomplete', 'off');
 
@@ -98,14 +102,24 @@ var completer = (function (input_element, dropdown, base_config) {
         return ret;
     }
 
+    var click_event = 'click.completer.' + input_element_raw.id;
+    function hide_on_click(e) {
+        console.log('good night');
+        hide();
+    }
+
     function show() {
-        if (dropdown.children().length > 0) {
+        if (dropdown.children().length > 0 && !dropdown_visible) {
+            dropdown_visible = true;
             dropdown.show();
+            $(document).on(click_event, "body", hide_on_click);
         }
     }
     function hide()
     {
         dropdown.hide();
+        dropdown_visible = false;
+        $(document).off(click_event, "body", hide_on_click);
     }
 
     /***
-- 
cgit v1.3.1