Adds an helper for creating the context menu.

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3542 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index 593b5a4cc1c4c5015cbb2ff92620ee79dcc29135..ff877f18f404e37c9d2efad174618ddfc15667e8 100644 (file)
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -661,6 +661,17 @@ module ApplicationHelper
       ), :class => 'progress', :style => "width: #{width};") +
       content_tag('p', legend, :class => 'pourcent')
   end
+  
+  def context_menu(url)
+    unless @context_menu_included
+      content_for :header_tags do
+        javascript_include_tag('context_menu') +
+          stylesheet_link_tag('context_menu')
+      end
+      @context_menu_included = true
+    end
+    javascript_tag "new ContextMenu('#{ url_for(url) }')"
+  end
 
   def context_menu_link(name, url, options={})
     options[:class] ||= ''

diff --git a/app/views/issues/index.rhtml b/app/views/issues/index.rhtml
index eac177c896655d1a1abee2815ceafad43b64b344..66bba490d913d0f955f833b8823dbf88fc0428de 100644 (file)
--- a/app/views/issues/index.rhtml
+++ b/app/views/issues/index.rhtml
@@ -78,9 +78,6 @@
 <% content_for :header_tags do %>
     <%= auto_discovery_link_tag(:atom, {:query_id => @query, :format => 'atom', :page => nil, :key => User.current.rss_key}, :title => l(:label_issue_plural)) %>
     <%= auto_discovery_link_tag(:atom, {:action => 'changes', :query_id => @query, :format => 'atom', :page => nil, :key => User.current.rss_key}, :title => l(:label_changes_details)) %>
-    <%= javascript_include_tag 'context_menu' %>
-    <%= stylesheet_link_tag 'context_menu' %>
 <% end %>
 
-<div id="context-menu" style="display: none;"></div>
-<%= javascript_tag "new ContextMenu('#{url_for(:controller => 'issues', :action => 'context_menu')}')" %>
+<%= context_menu :controller => 'issues', :action => 'context_menu' %>

diff --git a/app/views/my/page.rhtml b/app/views/my/page.rhtml
index 4d4c921b6d3cf090f86da486781f4367ce051a1c..608e233260d0e94b00ce4c7306bfed91de567dc5 100644 (file)
--- a/app/views/my/page.rhtml
+++ b/app/views/my/page.rhtml
@@ -31,12 +31,6 @@
        <% end if @blocks['right'] %>
 </div>
 
-<% content_for :header_tags do %>
-    <%= javascript_include_tag 'context_menu' %>
-    <%= stylesheet_link_tag 'context_menu' %>
-<% end %>
-
-<div id="context-menu" style="display: none;"></div>
-<%= javascript_tag "new ContextMenu('#{url_for(:controller => 'issues', :action => 'context_menu')}')" %>
+<%= context_menu :controller => 'issues', :action => 'context_menu' %>
 
 <% html_title(l(:label_my_page)) -%>

diff --git a/public/javascripts/context_menu.js b/public/javascripts/context_menu.js
index dac5076e2ef7bcd59497ec965a31166660a8066d..bd3b72618ab5c7ae436a12bf36035edfc2294540 100644 (file)
--- a/public/javascripts/context_menu.js
+++ b/public/javascripts/context_menu.js
@@ -7,7 +7,8 @@ ContextMenu = Class.create();
 ContextMenu.prototype = {
        initialize: function (url) {
        this.url = url;
-
+       this.createMenu();
+       
        // prevent text selection in the issue list
        var tables = $$('table.issues');
        for (i=0; i<tables.length; i++) {
@@ -95,6 +96,15 @@ ContextMenu.prototype = {
     }
   },
   
+  createMenu: function() {
+    if (!$('context-menu')) {
+      var menu = document.createElement("div");
+      menu.setAttribute("id", "context-menu");
+      menu.setAttribute("style", "display:none;");
+      document.getElementById("content").appendChild(menu);
+    }
+  },
+  
   showMenu: function(e) {
     var mouse_x = Event.pointerX(e);
     var mouse_y = Event.pointerY(e);