jQuery Grid Plugin - jqGrid

It is very good new, Tony!

To help you to improve the new code I used JSlint and found following small errors or suggestions:

line 67 ended with ',' before ']' in the next line. One should remove the ','

in line 145,154,296,462,493,519,523,527,531,534 one should replace ',' with ';' at the end of line.

in line 309 will be defined unused variables o and df.

In line 516 the code '+ ""' (at the end of line) should be removed

in lines 82,119,164,184,354,377 one should add ';' at the end of lines

in line 219,354,433,468,490,492,511-514,516,517,545,547,557,559,568,570,626 insert {}

in lines 32,166,183,197,218,232,255,282,327,338,344,370,395,410,415,449,489,498,505,511-514,516,544,556,567,576 replace "==" to "==="
in lines 249,269,287,466,477,481,541,553 replace "!=" to "!=="

if would be better to move this.getStringForGroup from the line 464 after the line 494 where
the function this.getStringForRule will be defined because in lines 472,482
the function this.getStringForRule will be used inside of this.getStringForGroup

In the same way the function getStringForRule defined in the line 572 will be better to move before the line 538
where the function getStringForGroup defined which use it

I used the code loaded today so I hope the line numbers stay unchanged in your code.

I next days I want to look the new module more carefully and to test it.

Best regards
Oleg

Hi Tony!

Just one idea. It would be nice to implement some kind of "filter collection": the list of some predefined filters with the name and description which could be easy loaded or saved (first in the internal parameter of jqGrid, next in qookie and which can be easy exported).

Having the possibility to define a complex query in a nice feature (what one has in the new filter module), but it could be too complex for standard users. The usage of predefined "filter collection" could simplify the work and improve users satisfaction. One could allow to switch from "standard mode" of choosing only predefined queries to "advanced mode" where full posibilities of the new filter module could be used.

Best regards
Oleg