jamescryer/grumble.js — Issue #5: Blows up on RequireJS (possibly on all other AMD loaders) with "{code here} has no 'extend' method"
Author: dvdotsenko
Created: 2011-12-17 00:16:19+00:00
State: open

This happens because content is eval'ed in some context that does not provide GrumbleBubble from window.

Suggest changing both, Bubble AND jQuery-Grumble codes to be wrapped as such:

For Bubble:

(function(window){
var $ = window.jQuery
...
})(window)

for jquery-grumble:

(function (window) {
var $ = window.jQuery
, Bubble = window.GrumbleBubble
...
})(window)

and the code stops blowing when $obj.grumble( is called.

This, of course does not make the code AMD-compatible in pure sense as it still does not have a define( , but it really does not need to be since it just decorates jQuery and all AMD loaders can just require that and get back $ object.

By the way. Plug in if fricken awesome! Females here (at office) especially love the bubbles, as error messages in bubbles are no longer bad-looking :)