From 98de8afef63192ca2282702506c058ac612c333f Mon Sep 17 00:00:00 2001
From: Pavol Rusnak <stick@gk2.sk>
Date: Mon, 18 Jul 2011 17:40:45 +0200
Subject: [PATCH] use the whole section line for showing/hiding content

---
 app/controllers/download_controller.rb |  5 -----
 app/views/download/html.erb            | 27 +++++++++++++++++----------
 app/views/download/js.erb              |  3 ---
 config/routes.rb                       |  1 -
 4 files changed, 17 insertions(+), 19 deletions(-)
 delete mode 100644 app/views/download/js.erb

diff --git a/app/controllers/download_controller.rb b/app/controllers/download_controller.rb
index acd4e2a..f4d6d37 100644
--- a/app/controllers/download_controller.rb
+++ b/app/controllers/download_controller.rb
@@ -61,11 +61,6 @@ class DownloadController < ApplicationController
     render :html, :layout => false
   end
 
-  # /download.js?prj=name&pkg=name
-  def js
-    render :js, :layout => false
-  end
-
   # /download.json?prj=name&pkg=name
   def json
     # needed for rails < 3.0 to support JSONP
diff --git a/app/views/download/html.erb b/app/views/download/html.erb
index 47e2049..ae48ebc 100644
--- a/app/views/download/html.erb
+++ b/app/views/download/html.erb
@@ -23,10 +23,10 @@ body {
   padding-top: 10px;
   font-size: 1.1em;
   display: none;
+  cursor: pointer;
 }
 .soo_line span {
   float: right;
-  cursor: pointer;
 }
 .soo_line_visible {
   display: block;
@@ -51,7 +51,6 @@ body {
   margin: 0;
 }
 .soo_ymplink, .soo_pkglink {
-  font-size: 0.85em;
   color: white;
   display: block;
   margin: 6px;
@@ -70,11 +69,13 @@ body {
 <%= gradient('#9D0','#7B0','#580') %>
   border: 1px solid #490;
   min-width: 112px;
+  font-size: 0.85em;
 }
 .soo_pkglink {
 <%= gradient('#09D','#07B','#058') %>
   border: 1px solid #049;
   min-width: 64px;
+  font-size: 0.75em;
 }
 pre {
   background: #EEE;
@@ -104,14 +105,20 @@ $(function(){
     $('.soo_distro_' + distro).show();
     if (distro == 'openSUSE' || distro == 'SLE') {
       $('#soo_ymp').show();
+      $('#soo_section_ymp').show();  $('#soo_section_toggle_ymp span').text('(hide)');
+      $('#soo_section_repo').hide(); $('#soo_section_toggle_repo span').text('(show)');
+      $('#soo_section_pkg').hide();  $('#soo_section_toggle_pkg span').text('(show)');
     } else {
       $('#soo_ymp').hide();
+      $('#soo_section_ymp').hide();  $('#soo_section_toggle_ymp span').text('(show)');
+      $('#soo_section_repo').show(); $('#soo_section_toggle_repo span').text('(hide)');
+      $('#soo_section_pkg').hide();  $('#soo_section_toggle_pkg span').text('(show)');
     }
   });
 
-  $('.soo_line span').click(function(){
+  $('.soo_line').click(function(){
       $('#' + $(this).attr('id').replace('toggle_', '')).toggle();
-      $(this).text( $(this).text() == 'V' ? '>' : 'V');
+      $(this).children('span').text($(this).children('span').text() == '(hide)' ? '(show)' : '(hide)');
   });
 
 // try to detect distro via user agent
@@ -137,19 +144,19 @@ $(function(){
 </div>
 
 <div id="soo_ymp" class="soo_box">
-  <p class="soo_line">Install using One Click Install<span id="soo_section_toggle_ymp">V</span></p>
+  <p class="soo_line" id="soo_section_toggle_ymp">Install using One Click Install<span></span></p>
   <div id="soo_section_ymp">
   <% @data.select {|k,v| v.has_key?(:ymp)}.each do |k,v| %>
-  <a class="soo_ymplink soo_distro soo_distro_<%= v[:flavor] %>" href="<%= v[:ymp] %>"><%= k.gsub('_', '&nbsp;') %></a>
+  <a class="soo_ymplink soo_distro soo_distro_<%= v[:flavor] %> soo_distro_<%= k %>" href="<%= v[:ymp] %>"><%= k.gsub('_', '&nbsp;') %></a>
   <% end %>
   </div>
 </div>
 
 <div id="soo_repo" class="soo_box">
-  <p class="soo_line">Add repository and install manually<span id="soo_section_toggle_repo">V</span></p>
+  <p class="soo_line" id="soo_section_toggle_repo">Add repository and install manually<span></span></p>
   <div id="soo_section_repo">
   <% @data.select {|k,v| v.has_key?(:repo)}.each do |k,v| %>
-  <div class="soo_repoinfo soo_distro soo_distro_<%= v[:flavor] %>">
+  <div class="soo_repoinfo soo_distro soo_distro_<%= v[:flavor] %> soo_distro_<%= k %>">
   <p>For <strong><%= k.gsub('_', '&nbsp;') %></strong> run the following as <strong>root</strong>:</p>
   <pre><%=
     case v[:flavor]
@@ -171,11 +178,11 @@ $(function(){
 </div>
 
 <div id="soo_pkg" class="soo_box">
-  <p class="soo_line">Grab binary packages directly<span id="soo_section_toggle_pkg">V</span></p>
+  <p class="soo_line" id="soo_section_toggle_pkg">Grab binary packages directly<span></span></p>
   <div id="soo_section_pkg">
   <table>
   <% @data.select {|k,v| v.has_key?(:pkg)}.each do |k,v| %>
-  <tr class="soo_pkginfo soo_distro soo_distro_<%= v[:flavor] %>"><td>Packages for <strong><%= k.gsub('_', '&nbsp;') %></strong>:</td>
+  <tr class="soo_pkginfo soo_distro soo_distro_<%= v[:flavor] %> soo_distro_<%= k %>"><td>Packages for <strong><%= k.gsub('_', '&nbsp;') %></strong>:</td>
   <td>
   <% v[:pkg].each do |k,v| %>
   <a class="soo_pkglink" href="<%= v %>"><%= k %></a>
diff --git a/app/views/download/js.erb b/app/views/download/js.erb
deleted file mode 100644
index 62adb9a..0000000
--- a/app/views/download/js.erb
+++ /dev/null
@@ -1,3 +0,0 @@
-js
-<%= @data %>
-/js
diff --git a/config/routes.rb b/config/routes.rb
index b808862..cd0301b 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -40,7 +40,6 @@ ActionController::Routing::Routes.draw do |map|
   map.connect '/codecs', :controller => 'codecs', :action => 'index'
 
   map.connect '/download.html', :controller => 'download', :action => 'html'
-  map.connect '/download.js',   :controller => 'download', :action => 'js'
   map.connect '/download.json', :controller => 'download', :action => 'json'
 
   # Install the default route as the lowest priority.
-- 
2.1.4