[FIX] web_tip: better event handling

* Override message_fetch from mail.Thread in a dedicated limk module (mail_tip)
and fire an event when the messages are loaded. This is necessary for tips on
the chatter.
* Override render_value form web.form.FieldStatus: Call reposition() when a
FieldStatus object is done rendering.
* Tip.reposition() method
* Handle web bus resize event

1 files changed, 38 insertions, 9 deletions

diff --git a/addons/web_tip/static/src/js/tip.js b/addons/web_tip/static/src/js/tip.js
index da2ac7f655e..6ad94dd998f 100644
--- a/addons/web_tip/static/src/js/tip.js
+++ b/addons/web_tip/static/src/js/tip.js
@@ -93,7 +93,13 @@
             formView.on('view_content_has_changed', self, function() {
                 self.eval_tip(null, model, mode, type);
             });
-            self.eval_tip(null, model, mode, type);
+            if ($('.oe_chatter').length > 0) {
+                instance.web.bus.on('chatter_messages_fetched', this, function() {
+                    self.eval_tip(null, model, mode, type);
+                });
+            } else {
+                self.eval_tip(null, model, mode, type);
+            }
         },
 
         // stub
@@ -183,16 +189,9 @@
                     $(scroll).scrollTo(self.$element);
                 }
 
-                var _top = self.$element.offset().top -5;
-                var _left = self.$element.offset().left -5;
-                var _width = self.$element.outerWidth() + 10;
-                var _height = self.$element.outerHeight() + 10;
-
                 self.$helper = $("<div>", { class: 'oe_tip_helper' });
                 self.$element.after(self.$helper);
-                self.$helper.offset({top: _top , left: _left});
-                self.$helper.width(_width);
-                self.$helper.height(_height);
+                self._set_helper_position();
 
                 self.$overlay = $("<div>", { class: 'oe_tip_overlay' });
                 $('body').append(self.$overlay);
@@ -247,6 +246,11 @@
                         def.resolve();
                     }
                 });
+
+                // resize
+                instance.web.bus.on('resize', this, function() {
+                    self.reposition();
+                });
             } else {
                def.reject();
             }
@@ -266,6 +270,24 @@
             $(document).off('keyup.web_tip');
             Tips.call('consume', [tip.id], {});
             tip.is_consumed = true;
+        },
+
+        reposition: function() {
+            var self = this;
+            if (self.tip_mutex.def.state() === 'pending') {
+                self._set_helper_position();
+            }
+        },
+
+        _set_helper_position : function() {
+            var offset = this.$element.offset();
+            var _top = offset.top - 5;
+            var _left = offset.left - 5;
+            var _width = this.$element.outerWidth() + 10;
+            var _height = this.$element.outerHeight() + 10;
+            this.$helper.offset({top: _top , left: _left});
+            this.$helper.width(_width);
+            this.$helper.height(_height);
         }
     });
 
@@ -275,4 +297,11 @@
             this.tip_handler = new instance.web.Tip();
         }
     });
+
+    instance.web.form.FieldStatus = instance.web.form.FieldStatus.extend({
+        render_value: function() {
+            this._super();
+            instance.webclient.tip_handler.reposition();
+        }
+    });
 })();