In case anyone tries to use it, the return condition should be:

if (!ptr.length || (td.tagName == 'INPUT' && !scb) || td.tagName == 'A') {
return true;
}

Mark

So following up to my own post… after studying the grid code I found a way to do it – although its something of a hack, and rather fragile.

If beforeSelectRow could be passed the event object, and the column ix, I think I could do everything I want without any trickery.

For anyone else interested in a grid that supports shift-click to select ranges, ctrl-click to toggle rows, and handles clicks on editable fields smartly, the code follows. To use it set “multiselect:true,beforeSelectRow:function(){return false;}” in the grid options, and then add the handler as follows:

$(mygrid).bind(“click”,$(mygrid),multiSelectHandler);

where multiselectHandler is:

function multiSelectHandler(e) {
var grid = e.data;
var ts = grid[0], td = e.target;
var scb = $(td).hasClass(“cbox”);
var ptr = $(td).parents(“tr.jqgrow”);
if (!ptr.length || td.tagName == 'INPUT' || td.tagName == 'A') {
return true;
}
var sel = grid.getGridParam('selarrrow');
var sid = ptr[0].id;
var selected = $.inArray(sid, sel) >= 0;
if (e.ctrlKey || (scb && (selected || !e.shiftKey))) {
grid.setSelection(false,true,ptr);
} else {
if (e.shiftKey) {
var six = grid.getInd( sid);
var min = six, max = six;
$.each(sel, function() {
var ix = grid.getInd( this);
if (ix < min) min = ix;
if (ix > max) max = ix;
});
while (min <= max) {
var row = ts.rows[min++];
var rid = row.id;
if (rid != sid && $.inArray(rid, sel)<0) {
grid.setSelection( false, false, $(row));
}
}
} else if (!selected) {
grid.resetSelection();
}
if (!selected) {
grid.setSelection( false, true, ptr);
} else {
var osr = grid.getGridParam('onSelectRow');
if ($.isFunction(osr)) {
osr(sid, true);
}
}
}
}

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.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