As we discussed on IRC, the page jumps around too much. This is an alternative 
solution to the same problem.

cheers

-- 
Jakub Steiner
http://jimmac.musichall.cz
>From 966e08db20c373c58927dfe578e09660e2380e77 Mon Sep 17 00:00:00 2001
From: Jakub Steiner <[email protected]>
Date: Thu, 7 Oct 2010 16:51:29 +0200
Subject: [PATCH] Give feedback that something is happening while packages are added.

---
 src/app/stylesheets/aggregator.scss |    5 +++++
 src/app/views/templates/new.haml    |   10 ++++++----
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/src/app/stylesheets/aggregator.scss b/src/app/stylesheets/aggregator.scss
index 82698e6..a7775d0 100644
--- a/src/app/stylesheets/aggregator.scss
+++ b/src/app/stylesheets/aggregator.scss
@@ -1067,6 +1067,11 @@ a.iconbutton {
       margin: 0;
     }
   }
+  span.loading {
+    background: url(/images/spinner.gif) no-repeat center left;
+    display: inline-block;
+    padding: 6px 0 6px 24px;
+  }
 
   .packagename {
     opacity: 1;
diff --git a/src/app/views/templates/new.haml b/src/app/views/templates/new.haml
index dcf6e6e..4963d72 100644
--- a/src/app/views/templates/new.haml
+++ b/src/app/views/templates/new.haml
@@ -11,16 +11,18 @@
       $container.load(list_url, list_data, function() {
         $container.removeClass('loading');
         $('#do_add_software').click(function(e) {
-          e.preventDefault();
           var url = '#{url_for :action => 'add_software', :ajax => true, :add_selected => true}';
           var data = {
             'packages[]': $("input:checked[name='packages[]']").map(function() {return $(this).val()}).get(),
             'groups[]': $("input:checked[name='groups[]']").map(function() {return $(this).val()}).get(),
             'template_id': $("input[name='tpl[id]']").val() || ''
           };
-          $('#selected_packages').load(url, data, function() {});
-          $container.hide();
-          $submit.show();
+          e.preventDefault();
+          $(this).replaceWith('<span class="loading grid_2 alpha">Adding Packages</span>');
+          $('#selected_packages').load(url, data, function() {
+            $container.hide();
+            $submit.show();
+          });
         });
         $('#cancel_add_software').click(function(e) {
           e.preventDefault();
-- 
1.7.2.3

_______________________________________________
deltacloud-devel mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/deltacloud-devel

Reply via email to