askupasoftware / mivhak.js

src/js/mivhak.component.js

/**
 * The list of registered components.
 * 
 * @type Array
 */
Mivhak.components = [];

The variable Mivhak seems to be never declared. If this is a global, consider adding a /** global: Mivhak */ comment.

/**
 * Register a new component
 * 
 * @param {string} name The components name
 * @param {Object} options A list of component properties
 */
Mivhak.component = function(name, options)
{
    Mivhak.components[name] = options;

The variable Mivhak seems to be never declared. If this is a global, consider adding a /** global: Mivhak */ comment.

};

/**
 * Render a new component
 * 
 * TODO: move this into a seperate library
 * 
 * @param {string} name The components name
 * @param {Object} props A list of component properties. 
 * This overrides the component's initial property values.
 */
Mivhak.render = function(name, props)
{
    var component = $.extend(true, {}, Mivhak.components[name]);

The variable Mivhak seems to be never declared. If this is a global, consider adding a /** global: Mivhak */ comment.

    var el = {};
    
    // Create the element from the template
    el.$el = $(component.template);
    
    // Create all methods
    $.each(component.methods, function(name, method){
        el[name] = function() {return method.apply(el,arguments);};
    });
    
    // Set properties
    $.each(component.props, function(name, prop){
        el[name] = (typeof props !== 'undefined' && props.hasOwnProperty(name) ? props[name] : prop);
    });
    
    // Bind events
    $.each(component.events, function(name, method){
        el.$el.on(name, function() {return method.apply(el,arguments);});
    });
    
    // Call the 'created' function if exists
    if(component.hasOwnProperty('created')) component.created.call(el);

Curly braces around statements make for more readable code and help prevent bugs when you add further statements.

    
    return el;
};