From 9209ab63a0db9df9dafd33f638a61b4dca44ec7c Mon Sep 17 00:00:00 2001
From: Akira Saito <akrstjp@users.sourceforge.jp>
Date: Mon, 23 Mar 2015 01:40:21 +0900
Subject: [PATCH] implement(in progress): WorkTime plug-in extension. refs
 #1362 (http://www.r-labs.org/issues/1362)

---
 app/views/hooks/_quick_view_issues_footer.html.erb | 30 +++++-----
 app/views/hooks/_quick_view_issues_header.html.erb | 29 ++++++----
 .../quick_view_worktime_tooltip.html.erb           | 64 +++++++++++++++++-----
 3 files changed, 84 insertions(+), 39 deletions(-)

diff --git a/app/views/hooks/_quick_view_issues_footer.html.erb b/app/views/hooks/_quick_view_issues_footer.html.erb
index 11bde42..33fbab5 100644
--- a/app/views/hooks/_quick_view_issues_footer.html.erb
+++ b/app/views/hooks/_quick_view_issues_footer.html.erb
@@ -1,5 +1,4 @@
 <div id="quick_view_dialogs"></div>
-<div id="quick_view_tooltip" style="background-color: white; border: solid 1px gray;"></div>
 
 <script>
 //<![CDATA[
@@ -12,13 +11,15 @@ $("tr.issue").dblclick(quick_view_issue_dblclick);
 <% if Setting.plugin_quick_view['worktime_ext_enabled'] %>
 if ($("body").hasClass("controller-work_time")) {
   quick_view_work_time_extension_init();
+  $(document).ajaxComplete(quick_view_work_time_extension_init);
 
   <% if Setting.plugin_quick_view['worktime_ext_tooltip_enabled'] %>
   quick_view_work_time_extension_tooltip_init();
+  $(document).ajaxComplete(quick_view_work_time_extension_tooltip_init);
   <% end %>
 }
 
-function quick_view_work_time_extension_init(){
+function quick_view_work_time_extension_init() {
   var links = $(".wt_iss_link");
   links.each(function(){
      var link = $(this);
@@ -33,24 +34,27 @@ function quick_view_work_time_extension_init(){
      $(anchor_html).insertAfter(insertTo);
      link.addClass("has_quick_view_anchor");
   });
-  $(document).ajaxComplete(quick_view_work_time_extension_init);
 }
 
 function quick_view_work_time_extension_tooltip_init() {
   var links = $(".wt_iss_link");
-  links.mouseover(function(){
+  links.each(function(){
      var link = $(this);
      if (link.hasClass("has_tooltip")) return;
-  
-     var issue_id = link.data("issue");
-     var url = "<%= quick_view_worktime_issue_path(0) %>".replace(/0$/, link.data("issue"));
-     $.ajax( { url: url, dataType: 'html', success: function (html){
-       link.addClass("has_tooltip");
-       link.tooltip({tooltipClass:"quick_view_tooltip", track: true, create:function(){link.attr("title",html);}});
-       link.tooltip("open");
-     }});
+
+     link.addClass("has_tooltip"); //need before ajax call.
+     link.mouseenter(function() { 
+       if (link.hasClass("has_tooltip_content")) return;
+
+       var issue_id = link.data("issue");
+       var url = "<%= quick_view_worktime_issue_path(0) %>".replace(/0$/, link.data("issue"));
+       $.ajax( { url: url, dataType: 'html', success: function (html){
+         link.addClass("has_tooltip_content"); //mark of tooltip content created.
+         link.tooltip({tooltipClass:"quick_view_tooltip", track: true, create:function(){link.attr("title",html);}});
+         link.tooltip("open");
+       }});
+     });
   });
-  $(document).ajaxComplete(quick_view_work_time_extension_tooltip_init);
 }
 <% end %>
 
diff --git a/app/views/hooks/_quick_view_issues_header.html.erb b/app/views/hooks/_quick_view_issues_header.html.erb
index 84f51df..a5dd568 100644
--- a/app/views/hooks/_quick_view_issues_header.html.erb
+++ b/app/views/hooks/_quick_view_issues_header.html.erb
@@ -1,29 +1,36 @@
 <style>
- .quick_view_tooltip {
-   background: white none;
-   font-size: 0.9em;
+.quick_view_tooltip {
+  background: white none;
+  max-width: 500px;
+  font-size: 0.8em;
+}
+
+.quick_view_tooltip > .ui-tooltip-content img.gravatar {
+  float: left;
+  margin-right: 3px;
+  margin-bottom: 3px;
 }
 
 .quick_view_tooltip .subject p{
-//  font-size: 0.8em;
-   margin-top: 0.2em;
-   margin-bottom: 0.2em;
+  font-size: 0.9em;
+  margin-top: 0.2em;
+  margin-bottom: 0.2em;
 }
 
 .quick_view_tooltip .subject h3{
-   font-size: 1.25em;
-   margin-left: 0px;
+  font-size: 1.25em;
+  margin-left: 0px;
 }
 
 .quick_view_tooltip .subject div{
-   margin-left: 1em;
+  margin-left: 1em;
 }
 
 .quick_view_tooltip div.subject{
-   margin-left: -1em;
+  margin-left: -1em;
 }
 
 .quick_view_tooltip_attributes th {
-   text-align: left;
+  text-align: left;
 }
 </style>
diff --git a/app/views/quick_view_worktime_issues/quick_view_worktime_tooltip.html.erb b/app/views/quick_view_worktime_issues/quick_view_worktime_tooltip.html.erb
index ae67556..562ac0d 100644
--- a/app/views/quick_view_worktime_issues/quick_view_worktime_tooltip.html.erb
+++ b/app/views/quick_view_worktime_issues/quick_view_worktime_tooltip.html.erb
@@ -6,28 +6,63 @@
   extend IssuesHelper
   extend AttachmentsHelper
   extend CustomFieldsHelper
-#  extend RepositoriesHelper
-#  extend JournalsHelper
 
   issue = @issue
   project = @issue.project
 %>
-     <div>
+      <div>
+        <%= avatar(@issue.author, :size => "50") %>
         <div class="subject">
-        <%= render_issue_subject_with_tree(issue) %>
+          <%= render_issue_subject_with_tree(issue) %>
         </div>
-        <hr>
+      </div>
+        <hr style="clear: both;">
         <table class="quick_view_tooltip_attributes">
-          <tr><th class="status"><%=l(:field_status) %></th><td class="status" ><%= h(issue.status.name)%></td></tr>
-          <tr><th class="priority"><%=l(:field_priority) %></th><td class="priority"><%= h(issue.priority.name)%></td></tr>
-          <tr><th class="fixed-version"><%=l(:field_fixed_version) %></th><td class="fixed-version"><%= issue.fixed_version ? link_to_version(issue.fixed_version) : "-" %></td></tr>
-          <tr><th class="assigned-to"><%=l(:field_assigned_to) %></th><td class="assigned_to"><%= avatar(issue.assigned_to, :size => "14").to_s.html_safe + (issue.assigned_to ? link_to_user(issue.assigned_to) : "-") %></td></tr>
-          <tr><th class="start-date"><%=l(:field_start_date) %></th><td class="start-date"><%= format_date(issue.start_date)%></td></tr>
-          <tr><th class="due-date"><%=l(:field_due_date) %></th><td class="due-date"><%= format_date(issue.due_date)%></td></tr>
+          <tr>
+            <th class="status"><%=l(:field_status) %></th>
+            <td class="status" ><%= h(issue.status.name)%></td>
+          </tr>
+          <tr>
+            <th class="priority"><%=l(:field_priority) %></th>
+            <td class="priority"><%= h(issue.priority.name)%></td>
+          </tr>
+          <% unless issue.disabled_core_fields.include?('category_id') %>
+          <tr>
+            <th class="category"><%=l(:field_category) %></th>
+            <td class="category"><%= h(issue.category ? issue.category.name : "-") %></td>
+          </tr>
+          <% end %>
+          <% unless issue.disabled_core_fields.include?('fixed_version_id') %>
+          <tr>
+            <th class="fixed-version"><%=l(:field_fixed_version) %></th>
+            <td class="fixed-version"><%= issue.fixed_version ? link_to_version(issue.fixed_version) : "-" %></td>
+          </tr>
+          <% end %>
+          <% unless issue.disabled_core_fields.include?('assigned_to_id') %>
+          <tr>
+            <th class="assigned-to"><%=l(:field_assigned_to) %></th>
+            <td class="assigned_to"><%= avatar(issue.assigned_to, :size => "14").to_s.html_safe + (issue.assigned_to ? link_to_user(issue.assigned_to) : "-") %></td>
+          </tr>
+          <% end %>
+          <% unless issue.disabled_core_fields.include?('start_date') %>
+          <tr>
+            <th class="start-date"><%=l(:field_start_date) %></th>
+            <td class="start-date"><%= format_date(issue.start_date)%></td>
+          </tr>
+          <% end %>
+          <% unless issue.disabled_core_fields.include?('due_date') %>
+          <tr>
+            <th class="due-date"><%=l(:field_due_date) %></th>
+            <td class="due-date"><%= format_date(issue.due_date)%></td>
+          </tr>
+          <% end %>
           <% unless issue.disabled_core_fields.include?('estimated_hours') %>
-          <%  unless issue.estimated_hours.nil? %>
-          <tr><th class="estimated-hours"><%=l(:field_estimated_hours) %></th><td class="estimated-hours"><%= format_date(issue.estimated_hours)%></td></tr>
-          <%  end %>
+          <%   unless issue.estimated_hours.nil? %>
+          <tr>
+            <th class="estimated-hours"><%=l(:field_estimated_hours) %></th>
+            <td class="estimated-hours"><%= format_date(issue.estimated_hours)%></td>
+          </tr>
+          <%   end %>
           <% end%>
           <% if User.current.allowed_to?(:view_time_entries, project) %>
           <tr>
@@ -46,4 +81,3 @@
         <% if issue.created_on != issue.updated_on %>
         <%= l(:label_updated_time, time_tag(issue.updated_on)).html_safe %>.
         <% end %>
-     </div>
-- 
2.11.0