From 0a7c4e83e823e2a8d74c7ac352ae69bfbecfe434 Mon Sep 17 00:00:00 2001
From: Martin Markech <martin.markech@matho.sk>
Date: Thu, 27 Sep 2012 18:22:25 +0200
Subject: add page tree + jquery tristate checkbox plugin

---
 .../refinery/banners/admin/banners/_form.html.erb  | 153 +++++++++++----------
 .../refinery/banners/admin/banners/_page.html.erb  |  19 +++
 2 files changed, 98 insertions(+), 74 deletions(-)
 create mode 100644 app/views/refinery/banners/admin/banners/_page.html.erb

(limited to 'app/views')

diff --git a/app/views/refinery/banners/admin/banners/_form.html.erb b/app/views/refinery/banners/admin/banners/_form.html.erb
index 1403fc4..b249f09 100644
--- a/app/views/refinery/banners/admin/banners/_form.html.erb
+++ b/app/views/refinery/banners/admin/banners/_form.html.erb
@@ -1,76 +1,81 @@
+<%- content_for(:javascripts) do -%>
+    <%= javascript_include_tag "banners/jquery.tristate.js", "banners/banners.js" -%>
+<%- end -%>
+
 <%= form_for [refinery, :banners_admin, @banner] do |f| -%>
-  <%= render '/refinery/admin/error_messages',
-              :object => @banner,
-              :include_object_name => true %>
-
-  <%= hidden_field_tag 'banner[page_ids][]' %>
-
-  <div class='field'>
-    <%= f.label :name -%>
-    <%= f.text_field :name, :class => 'larger widest'  -%>
-
-  </div>
-  
-  <div class='field'>
-    <%= f.label :title -%>
-    <%= f.text_field :title  -%>
-
-  </div>
-  
-  <div class='field'>
-    <%= f.label :description -%>
-    <%= f.text_field :description  -%>
-
-  </div>
-  
-  <div class='field'>
-    <%= f.label :image -%>
-    <%= render '/refinery/admin/image_picker',
-               :f => f,
-               :field => :image_id,
-               :image => @banner.image,
-               :toggle_image_display => false,
-               :description => t('refinery.banners.admin.banner')
-    %>
-
-  </div>
-  
-  <div class='field'>
-    <%= f.label :url -%>
-    <%= f.text_field :url  -%>
-
-  </div>
-  
-  <div class='field'>
-    <%= f.label :is_active -%>
-    <%= f.check_box :is_active, :checked => @banner[:is_active]  -%>
-
-  </div>
-  
-  <div class='field'>
-    <%= f.label :start_date -%>
-    <%= f.date_select :start_date  -%>
-
-  </div>
-  
-  <div class='field'>
-    <%= f.label :expiry_date -%>
-    <%= f.date_select :expiry_date  -%>
-
-  </div>
-
-  <div class="field">
-    <%= f.label :pages %>
-    <% Refinery::Page.all.each do |page| %>
-      <div>
-        <%= check_box_tag "banner[page_ids][]", page.id, @banner.pages.include?(page) %>
-        <%= page.title %>
-      </div>
-    <% end %>
-  </div>
-
-  <%= render '/refinery/admin/form_actions', :f => f,
-             :continue_editing => false,
-             :delete_title => t('delete', :scope => 'refinery.banners.admin.banners.banner'),
-             :delete_confirmation => t('message', :scope => 'refinery.admin.delete', :title => @banner.name) %>
+    <%= render '/refinery/admin/error_messages',
+               :object => @banner,
+               :include_object_name => true %>
+
+    <%= hidden_field_tag 'banner[page_ids][]' %>
+
+    <div class='field'>
+      <%= f.label :name -%>
+      <%= f.text_field :name, :class => 'larger widest'  -%>
+
+    </div>
+
+    <div class='field'>
+      <%= f.label :title -%>
+      <%= f.text_field :title  -%>
+
+    </div>
+
+    <div class='field'>
+      <%= f.label :description -%>
+      <%= f.text_field :description  -%>
+
+    </div>
+
+    <div class='field'>
+      <%= f.label :image -%>
+      <%= render '/refinery/admin/image_picker',
+                 :f => f,
+                 :field => :image_id,
+                 :image => @banner.image,
+                 :toggle_image_display => false,
+                 :description => t('refinery.banners.admin.banner')
+      %>
+
+    </div>
+
+    <div class='field'>
+      <%= f.label :url -%>
+      <%= f.text_field :url  -%>
+
+    </div>
+
+    <div class='field'>
+      <%= f.label :is_active -%>
+      <%= f.check_box :is_active, :checked => @banner[:is_active]  -%>
+
+    </div>
+
+    <div class='field'>
+      <%= f.label :start_date -%>
+      <%= f.date_select :start_date  -%>
+
+    </div>
+
+    <div class='field'>
+      <%= f.label :expiry_date -%>
+      <%= f.date_select :expiry_date  -%>
+
+    </div>
+
+    <section id="records" class="tree">
+      <%= f.label :pages %>
+      <ul class="tristate">
+        <%= render :partial => 'refinery/banners/admin/banners/page', :collection => Refinery::Page.order("lft ASC").select{|p| p.parent_id.nil?}  %>
+      </ul>
+    </section>
+
+
+    <div style="clear: both;"></div>
+    <%= render '/refinery/admin/form_actions', :f => f,
+               :continue_editing => false,
+               :delete_title => t('delete', :scope => 'refinery.banners.admin.banners.banner'),
+               :delete_confirmation => t('message', :scope => 'refinery.admin.delete', :title => @banner.name) %>
 <% end -%>
+
+
diff --git a/app/views/refinery/banners/admin/banners/_page.html.erb b/app/views/refinery/banners/admin/banners/_page.html.erb
new file mode 100644
index 0000000..640d378
--- /dev/null
+++ b/app/views/refinery/banners/admin/banners/_page.html.erb
@@ -0,0 +1,19 @@
+<li class='clearfix record icons' id="<%= dom_id(page) -%>" >
+
+    <% if page.children.present? %>
+        <span class="icon toggle expanded" title="<%= t('expand_collapse', :scope => 'refinery.admin.pages') %>"></span>
+    <% else %>
+        <span class="icon"></span>
+    <% end %>
+
+    <%= check_box_tag "banner[page_ids][]", page.id, @banner.pages.include?(page) %>
+
+    <span class='title <%= 'toggle' if page.children.present? %>'>
+      <%= page.title %>
+    </span>
+
+
+  <ul class='nested' >
+    <%= render(:partial => 'refinery/banners/admin/banners/page', :collection => page.children) %>
+  </ul>
+</li>
-- 
cgit v1.2.3


From 5225251f0463662c5ccae4332dbd17f90036a760 Mon Sep 17 00:00:00 2001
From: Martin Markech <martin.markech@matho.sk>
Date: Sun, 30 Sep 2012 19:56:31 +0200
Subject: temp fix for jquery tristate - render only on new action, because on
 edit it has problems to correctly render some checkboxes

---
 app/views/refinery/banners/admin/banners/_form.html.erb | 4 ----
 app/views/refinery/banners/admin/banners/new.html.erb   | 6 ++++++
 2 files changed, 6 insertions(+), 4 deletions(-)

(limited to 'app/views')

diff --git a/app/views/refinery/banners/admin/banners/_form.html.erb b/app/views/refinery/banners/admin/banners/_form.html.erb
index b249f09..3ea7421 100644
--- a/app/views/refinery/banners/admin/banners/_form.html.erb
+++ b/app/views/refinery/banners/admin/banners/_form.html.erb
@@ -1,7 +1,3 @@
-<%- content_for(:javascripts) do -%>
-    <%= javascript_include_tag "banners/jquery.tristate.js", "banners/banners.js" -%>
-<%- end -%>
-
 <%= form_for [refinery, :banners_admin, @banner] do |f| -%>
     <%= render '/refinery/admin/error_messages',
                :object => @banner,
diff --git a/app/views/refinery/banners/admin/banners/new.html.erb b/app/views/refinery/banners/admin/banners/new.html.erb
index 2d34363..967b41e 100644
--- a/app/views/refinery/banners/admin/banners/new.html.erb
+++ b/app/views/refinery/banners/admin/banners/new.html.erb
@@ -1 +1,7 @@
+<%# TODO temp fix - activate tristate only on new. There is bug and inproperly render some checkboxes in edit action %>
+<%- content_for(:javascripts) do -%>
+    <%= javascript_include_tag "banners/jquery.tristate.js", "banners/banners.js" -%>
+<%- end -%>
+
 <%= render 'form' %>
+
-- 
cgit v1.2.3