5365: add back data-toggle along with link disabled for Log tab.

[arvados.git] / apps / workbench / app / views / application / _content.html.erb

diff --git a/apps/workbench/app/views/application/_content.html.erb b/apps/workbench/app/views/application/_content.html.erb
index 99fbfaa2fd2087516266aa023c2939754b297f41..5e6211f4623fbedfcfe72539fc1beb4c56b3540c 100644 (file)
--- a/apps/workbench/app/views/application/_content.html.erb
+++ b/apps/workbench/app/views/application/_content.html.erb
@@ -6,6 +6,7 @@
     <% pane_list.each_with_index do |pane, i| %>
       <% pane_name = (pane.is_a?(Hash) ? pane[:name] : pane) %>
 
+      <% data_toggle = "tab" %>
       <% tab_tooltip = "" %>
       <% link_disabled = "" %>
 
@@ -14,6 +15,7 @@
             if @object.log and !@object.log.empty?
               logCollection = Collection.find? @object.log
               if !logCollection
+                data_toggle = "disabled"
                 tab_tooltip = "Log data is not available"
                 link_disabled = "disabled"
               end
@@ -21,6 +23,7 @@
           elsif controller.model_class.to_s == 'PipelineInstance'
             log_uuids = [@object.uuid] + pipeline_jobs(@object).collect{|x|x[:job].andand[:uuid]}.compact
             if stderr_log_history(log_uuids, 1).empty?
+              data_toggle = "disabled"
               tab_tooltip = "Log data is not available"
               link_disabled = "disabled"
             end
@@ -31,7 +34,7 @@
       <li class="<%= 'active' if i==0 %> <%= link_disabled %>" data-toggle="tooltip" data-placement="top" title="<%=tab_tooltip%>">
         <a href="#<%= pane_name %>"
            id="<%= pane_name %>-tab"
-           data-toggle="tab"
+           data-toggle="<%= data_toggle %>"
            data-tab-history=true
            data-tab-history-update-url=true
            >