Problem when using with jQuery 1.9.1

[jteich]

While using this plugin with jQuery 1.9.1, I found a problem where the code was using selectors that ended commas when it would select multiple rows as in: Uncaught Error: Syntax error, unrecognized expression: .tj_row_1,.tj_row_2,
I was able to resolve this issue by modifying the code to remove trailing commas whenever it would build the currentRows and nextRows selector strings.

[douglasmiranda]

I don't get it. Why did you just use a selector ending with a comma?

Why did you not use: .tj_row_1, .tj_row_2 instead .tj_row_1, .tj_row_2,?

Or am I missing something?

[jteich]

Let me clarify. The problem is that the code in gridnav will try to use these selectors that end with a comma.
For example, in jquery.gridnav.js, you have a lot of lines that look similar to:
nextRows += '.tj_row_' + (config.currentRow + opts.rows + i) + ','
which leaves a trailing comma in "nextRows". The newer versions of jQuery don't like the trailing comma and cause the code to fail. as a fix, I added lines of code like so: nextRows = nextRows.substring(0, nextRows.length - 1);