From 41d28dddbc85ccb8ca42cb01b0ea34c712d59356 Mon Sep 17 00:00:00 2001
From: "James E. Blair" <jeblair@openstack.org>
Date: Thu, 2 Jan 2014 11:41:14 -0800
Subject: [PATCH] Zuul status: don't toggle on link click

Don't toggle the display of detailed info for a change if the user
clicks on the hyperlink within the header div.

Change-Id: I3f20dfe6a4ee40b79859572017721a45acbc5672
---
 modules/openstack_project/files/zuul/status.js | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/modules/openstack_project/files/zuul/status.js b/modules/openstack_project/files/zuul/status.js
index 6eaf517985..2602dcc2fb 100644
--- a/modules/openstack_project/files/zuul/status.js
+++ b/modules/openstack_project/files/zuul/status.js
@@ -230,7 +230,7 @@ function format_change(change, change_queue) {
 
     html += '<td class="change-container">';
     html += '<div class="change" id="' + safe_id(change['id']) + '">' +
-            '<div class="header" onClick="toggle_display_jobs(this)" ' +
+            '<div class="header" onClick="toggle_display_jobs(event, this)" ' +
             'onmouseover="$(this).addClass(\'hover\')" ' +
             'onmouseout="$(this).removeClass(\'hover\')">';
 
@@ -310,8 +310,15 @@ function format_change(change, change_queue) {
     return html;
 }
 
-function toggle_display_jobs(_header) {
+function toggle_display_jobs(e, _header) {
+    e = e || window.event;  // standards compliant || IE
     var header = $(_header);
+    var target = $(e.target || e.srcElement);
+    var link = header.find("a");
+    if (target.is(link)) {
+        return true;
+    }
+
     content = header.next("div");
     content.slideToggle(100, function () {
         changeid = header.parent().attr('id');