From d26b36480b6908a0bbb2016bf914e91cea648249 Mon Sep 17 00:00:00 2001
From: ggh-openerp <ggh@tinyerp.com>
Date: Wed, 4 Jul 2012 11:21:06 +0530
Subject: [PATCH] [IMP]Improve code for Bounce effect in listview

bzr revid: ggh@tinyerp.com-20120704055106-r3yuhcvadpel9mhy
---
 addons/web/static/src/css/base.css    |  4 ++++
 addons/web/static/src/js/corelib.js   |  7 +++++++
 addons/web/static/src/js/view_list.js | 11 ++++++-----
 3 files changed, 17 insertions(+), 5 deletions(-)

diff --git a/addons/web/static/src/css/base.css b/addons/web/static/src/css/base.css
index f738fe757ac..00ecab65951 100644
--- a/addons/web/static/src/css/base.css
+++ b/addons/web/static/src/css/base.css
@@ -2215,4 +2215,8 @@
 }
 .kitten-mode-activated > * {
   opacity: 0.7;
+}
+.openerp .oe_bounce_button_left {
+  float: left;
+  margin-right: 4px;
 }
\ No newline at end of file
diff --git a/addons/web/static/src/js/corelib.js b/addons/web/static/src/js/corelib.js
index 5af51fa519a..a4b0ca97eaa 100644
--- a/addons/web/static/src/js/corelib.js
+++ b/addons/web/static/src/js/corelib.js
@@ -728,6 +728,12 @@ instance.web.Widget = instance.web.Class.extend(instance.web.WidgetMixin, {
         }
         return false;
     },
+    /*
+    * In button to bounce effect 
+    */
+    do_bounce: function(element) {
+        element.effect("bounce", { times:3}, 500);
+    },
     do_notify: function() {
         if (this.getParent()) {
             return this.getParent().do_notify.apply(this,arguments);
@@ -740,6 +746,7 @@ instance.web.Widget = instance.web.Class.extend(instance.web.WidgetMixin, {
         }
         return false;
     },
+    
     rpc: function(url, data, success, error) {
         var def = $.Deferred().then(success, error);
         var self = this;
diff --git a/addons/web/static/src/js/view_list.js b/addons/web/static/src/js/view_list.js
index 848c2934b17..4c725c157ab 100644
--- a/addons/web/static/src/js/view_list.js
+++ b/addons/web/static/src/js/view_list.js
@@ -285,8 +285,8 @@ instance.web.ListView = instance.web.View.extend( /** @lends instance.web.ListVi
                 this.$element.find('.oe_list_buttons').replaceWith(this.$buttons);
             }
             this.$buttons.find('.oe_list_add')
-            		.wrap("<div></div>")
-        		    .css({"float":"left","margin-right":"4px"})
+                    .wrap("<div></div>")
+                    .addClass('oe_bounce_button_left')
                     .click(this.proxy('do_add_record'))
                     .prop('disabled', grouped && this.options.editable);
             this.$buttons.on('click', '.oe_list_button_import', function() {
@@ -982,9 +982,10 @@ instance.web.ListView.List = instance.web.Class.extend( /** @lends instance.web.
                 }
                 else{
                     if (opts.options.$buttons){
-                        $(opts.options.$buttons.find('.oe_list_add')).effect("bounce", { times:5 }, 300);
-					}
-					}
+                    var a = $(opts.options.$buttons.find('.oe_list_add'));
+                    self.view.do_bounce(a);
+                    }
+                    }
             });
     },
     row_clicked: function (e, view) {