Add meeting JS to show/hide edit form

app/assets/javascripts/meeting/meeting.js

@@ -0,0 +1,23 @@
+jQuery(function($) {
+  function toggleContentTypeForm(content_type, edit) {
+    jQuery('.edit-' + content_type).toggle(edit);
+    jQuery('.show-' + content_type).toggle(!edit);
+
+    jQuery('.button--edit-agenda').toggleClass('-active', edit);
+    jQuery('.button--edit-agenda').attr('disabled', edit);
+  }
+
+  $('.button--edit-agenda').click(function() {
+    var content_type = $(this).data('contentType');
+    toggleContentTypeForm(content_type, true);
+
+    return false;
+  });
+
+  $('.button--cancel-agenda').click(function() {
+    var content_type = $(this).data('contentType');
+    toggleContentTypeForm(content_type, false);
+
+    return false;
+  });
+});

app/helpers/meeting_contents_helper.rb

@@ -92,14 +92,11 @@ module MeetingContentsHelper
 
   def meeting_content_edit_link(content_type)
     content_tag :li, '', class: 'toolbar-item' do
+
       content_tag :button,
                   '',
-                  class: 'button button--edit-agenda',
+                  data: { 'content-type': content_type },
-                  onclick: "jQuery('.edit-#{content_type}').show();
+                  class: 'button button--edit-agenda' do
-                            jQuery('.show-#{content_type}').hide();
-                            jQuery('.button--edit-agenda').addClass('-active');
-                            jQuery('.button--edit-agenda').attr('disabled', true);
-                  return false;" do
         link_to l(:button_edit),
                 '',
                 class: 'icon-context icon-edit',

app/views/meeting_contents/_form.html.erb

@@ -29,13 +29,7 @@ See doc/COPYRIGHT.md for more details.
 <p><%= styled_button_tag l(:button_save), class: '-highlight -with-icon icon-checkmark' %>
 <% path = send("preview_#{content_type}_path", content.meeting) %>
 <%= preview_link path, "#{content_type}_form", { class: 'button preview' } %>
-<%= link_to l(:button_cancel), "#",
+<%= link_to l(:button_cancel), "#", data: { 'content-type': content_type }, class: 'button button--cancel-agenda' %>
-      :onclick => "jQuery('.show-#{content_type}').show();
-                   jQuery('.edit-#{content_type}').hide();
-                   jQuery('.button--edit-agenda').removeClass('-active');
-                   jQuery('.button--edit-agenda').removeAttr('disabled');;
-                   return false;",
-      class: 'button' %></p>
 <%= wikitoolbar_for "#{content_type}_text" %>
 <% end %>
 

app/views/meeting_contents/_show.html.erb

@@ -23,6 +23,11 @@ See doc/COPYRIGHT.md for more details.
   tab ||= locals[:tab_contents] if defined? locals
   content, content_type = tab[:content], tab[:content_type] if tab && tab.present?
 -%>
+
+<% content_for :header_tags do %>
+  <%= javascript_include_tag('meeting/meeting.js') %>
+<% end %>
+
 <div class="meeting_content <%= content_type %>">
   <div>
     <%= toolbar title: l(:"label_#{content_type}") do %>
@@ -31,20 +36,22 @@ See doc/COPYRIGHT.md for more details.
   </div>
 
   <% if can_edit_meeting_content?(content, content_type) -%>
-    <div id="edit-<%= content_type %>" class="edit-<%= content_type %>"<%= " style=\"display: none;\"" unless show_meeting_content_editor?(content, content_type)%>>
+    <div id="edit-<%= content_type %>"
+         class="edit-<%= content_type %>"
+         style="<%= 'display: none' unless show_meeting_content_editor?(content, content_type) %>">
     <%= render(:partial => "meeting_contents/form", :locals => {:content => content, :content_type => content_type}) %>
     </div>
   <% end -%>
 
   <% if saved_meeting_content_text_present?(content) -%>
-    <div id="<%= content_type %>-text" class="wiki show-<%= content_type %>">
+    <div id="<%= content_type %>-text"
+         style="<%= 'display: none' if show_meeting_content_editor?(content, content_type) %>"
+         class="wiki show-<%= content_type %>">
       <%= format_text(content.text, :object => @meeting) %>
     </div>
   <% else -%>
     <%= no_results_box %>
   <% end -%>
-
-  <%= nonced_javascript_tag(show_meeting_content_editor?(content, content_type) ? "jQuery('.show-#{content_type}').hide();" : "jQuery('.edit-#{content_type}').hide();") %>
 </div>
 
 <%= render :partial => 'shared/meeting_header' %>

lib/open_project/meeting/engine.rb

@@ -68,7 +68,7 @@ module OpenProject::Meeting
     patch_with_namespace :BasicData, :SettingSeeder
 
     initializer 'meeting.precompile_assets' do
-      Rails.application.config.assets.precompile += %w(meeting/meeting.css)
+      Rails.application.config.assets.precompile += %w(meeting/meeting.css meeting/meeting.js)
     end
 
     initializer 'meeting.register_hooks' do