Remove hidden columns from column width calculation

[ps2goat]

Consider removing hidden columns from the column width calculation. I did this by modifying the following function in jquery.tablesorter.js:

function fixColumnWidth(table) {
    if (table.config.widthFixed && $(table).find('colgroup').length === 0) {
        var colgroup = $('<colgroup>'),
            overallWidth = $(table).width();
        /*  removed hidden elements from the code that determines the width of columns.  added 'remove(":hidden")' below */
        $(table.tBodies[0]).find("tr:first").children("td").remove(":hidden").each(function() {
            colgroup.append($('<col>').css('width', parseInt(($(this).width()/overallWidth)*1000, 10)/10 + '%'));
        });
        $(table).prepend(colgroup);
    }
}

[thezoggy]

Using the visible selector may be a more elegant solution: http://api.jquery.com/visible-selector/

Elements with visibility: hidden or opacity: 0 are considered visible, since they still consume space in the layout.

[ps2goat]

Good point. I've been busy working other areas of the tablesorter and the filter plugin to also support this with ajax loads, as the code above only works with static versions.

[Mottie]

Hi @ps2goat!

Sorry for the delay.

I'll have this fixed in the next update.