emyller / mandoo Goto Github PK

rename the Animation class to a smaller name is a really small improvement, but still is one.

u.require would be more useful if it could "import" stuff into an object. (one could then use it to load one's own modules instead of Mandoo plugins only).

Let's start to follow some of the commonjs.org ideas -- at least apply what we can to work with front-end JS.

line 266 or so:
req = new u.Request(url, options, data);

but u.request does not accept data.
Also the .onfinish is not the correct syntax

Fix:
options.method = method;
options.data = data;
req = new u.Request(url, options, data);
fn && req.on('finish',(fn));

Whole code block once complete:
$common: function (method, url, options, data) {
var fn, req;
if (typeof options === 'function') {
fn = options;
options = {}; }
else
if (typeof options === 'boolean') {
options = { async: options }; }
options = options || {};
options.method = method;
options.data = data;
req = new u.Request(url, options, data);
fn && req.on('finish',(fn));
return req; },

Is there going to be a set format for tagging versions... For example, will it continue to follow the following format: x.xx?

Any reason why we cannot use the following:
u.error = function (msg) {
try
{
throw "Mandoo: " + msg;
} catch ( e ) {
alert(e);
}
};

Of course this can be compressed to a shorter form to save space :)

Check handling of special attributes, something is really wrong there.

today, .hover doesn't animate if there's another animation in progress on the related element, due to animation conflicts. the conflict checking is not necessary.

now that the project changed its name, some parts of the code and file names need to be updated. all the modules that also need to use 'mandoo' instead of 'utm'.

put modules in other repositories to keep the main project one cleaner.

Got a really weird error message ("Member not found") when trying to get/set opacity on created (not yet appended) elements.
tested on IE8.

Currently, commands like .pos('left center') acts only based on the whole page size. It would be better if the method could set elements positions based on other elements' size and position.

Toggle could use an option to allow show(force) to be used. Currently Toggle will only work with items that are not set to display: none in firefox 3.5.x

Updated Code:
toggle: function (force) {
for (var i = -1; this[++i];)
u(this[i])u(this[i]).css('display') === 'none' ? 'show' : 'hide';
return this; }

implement a cookie helper in the project core.

u(element).size() returns width and height as 0.

clean .addClass and .rmClass. If possible, get rid of them, and do something more extensible (to handle other space-separated value attributes).