From 8b6db4461c75fb72a9c64c0bae6105c927b3dc10 Mon Sep 17 00:00:00 2001
From: Piotr Sarnacki <drogus@gmail.com>
Date: Mon, 12 Jan 2015 12:20:19 +0100
Subject: [PATCH] Don't use events to open popup for allowed failures help

---
 assets/scripts/app/templates/jobs/list.hbs | 3 ++-
 assets/scripts/app/views/job.coffee        | 8 ++++++++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/assets/scripts/app/templates/jobs/list.hbs b/assets/scripts/app/templates/jobs/list.hbs
index c122243f..c9aee3f9 100644
--- a/assets/scripts/app/templates/jobs/list.hbs
+++ b/assets/scripts/app/templates/jobs/list.hbs
@@ -7,7 +7,8 @@
   {{else}}
     <caption>
       Allowed Failures
-      <a title="What's this?" class="help open-popup" name="help-allowed_failures" {{action "popup" target=view}}></a>
+      <a title="What's this?" class="help open-popup"
+        name="help-allowed_failures" {{action "openHelpPopup" target=view}}></a>
     </caption>
   {{/if}}
     <thead>
diff --git a/assets/scripts/app/views/job.coffee b/assets/scripts/app/views/job.coffee
index 40d89b14..d87e283d 100644
--- a/assets/scripts/app/views/job.coffee
+++ b/assets/scripts/app/views/job.coffee
@@ -10,6 +10,14 @@ Travis.reopen
         'allowed_failure_jobs'
     )
 
+    actions:
+      popupClose: ->
+        @popupCloseAll()
+
+      openHelpPopup: ->
+        @popupCloseAll()
+        @popup('help-allowed_failures')
+
   JobsItemView: Travis.View.extend
     tagName: 'tr'
     classNameBindings: ['color']