jQuery Grid Plugin - jqGrid - Topic: jQuery.css returns a string

tony on jQuery.css returns a string

Fri, 21 Aug 2009 01:35:54 +0300

Hello Mark,

Thanks for this. I do not know about this bug. Will try to set the css at end of the expressions, where possible, but I think that this will be fixed in the next release of jQuery.

Best Regards

Tony

markw65 on jQuery.css returns a string

Wed, 19 Aug 2009 15:30:04 +0300

Sorry, you're right... except that there appears to be a minor bug in jQuery.

If you're setting height or width, and you pass in a -ve value, (eg $("#foo").css("width", "-4px"), then it goes through the "undefined" case in the above, which then /returns/ the width, rather than setting the width, and returning a jQuery.

That's what was happening, and I just jumped to the conclusion that css always returned a string.

So now I need to track down how colModel[i].width gets to be < 0. I think it has something to do with autowidth.

In any case, its probably my bug (in that it doesnt show up in firefox) - but I'd still say that its a good idea not to chain the css method when setting width/height by name unless you're certain the width/height is >= 0.

Mark

tony on jQuery.css returns a string

Wed, 19 Aug 2009 11:55:32 +0300

Hello,

Sorry, but I think that this is not true.

Here is the jQuery css:

    css: function( key, value ) {
        // ignore negative width and height values
        if ( (key == 'width' || key == 'height') && parseFloat(value) < 0 )
            value = undefined;
        return this.attr( key, value, "curCSS" );
    },

And here the attr

attr: function( name, value, type ) {
var options = name;

        // Look for the case where we're accessing a style value
        if ( typeof name === "string" )
            if ( value === undefined )
                return this[0] && jQuery[ type || "attr" ]( this[0], name );

            else {
                options = {};
                options[ name ] = value;
            }

        // Check to see if we're setting style values
        return this.each(function(i){
            // Set all the styles
            for ( name in options )
                jQuery.attr(
                    type ?
                        this.style :
                        this,
                    name, jQuery.prop( this, options[ name ], type, i, name )
                );
        });
    },

So if we set value the css return this.

Regards

Tony

markw65 on jQuery.css returns a string

Wed, 19 Aug 2009 11:08:22 +0300

There are a few cases where the result of jQuery.css is being used as if it were of type jQuery. IE8 (and probably others) doesnt like that:

grid.base.js:1086

$(eg).css("width",grid.width+"px").append("<div class='ui-jqgrid-resize-mark' id='rs_m"+ts.p.id+"'> </div>");

similarly line 1106.

Mark