Letting ContactDetails handle the "tap outside to close"

etiennesegonzac · Mar 4, 2012 · a74a188 · a74a188
1 parent 0ca400a
commit a74a188
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/apps/dialer/js/contacts.js b/apps/dialer/js/contacts.js
@@ -229,6 +229,14 @@ var ContactDetails = {
   _editing: false,
   setup: function cd_setup() {
     window.addEventListener('keyup', this, true);
+
+    // click outside details container to close
+    this.overlay.addEventListener('click', function(evt) {
+      ContactDetails.hide();
+    });
+    this.container.addEventListener('click', function(evt) {
+      evt.stopPropagation();
+    });
   },
   get overlay() {
     delete this.overlay;
@@ -473,14 +481,6 @@ window.addEventListener('load', function contactSetup(evt) {
   ShortcutsHandler.setup();
   ContactDetails.setup();
 
-  // click outside details container to close
-  document.getElementById('contacts-overlay').addEventListener('click', function(evt) {
-    ContactDetails.hide();
-  });
-  document.getElementById('contact-details-container').addEventListener('click', function(evt) {
-    evt.stopPropagation();
-  });
-
   // add scrollwheel fallback for browser users
   var scrollNode = Contacts.view;
   scrollNode.addEventListener('DOMMouseScroll', wheel, false);