Just added the suggestion. Thanks

Tony

I want remind you about one part from the old suggestion.

I answerd last time many questions on the stackoverflow where the reason of posted problem was usage of id of grid having spaces. So I would like remind you about the suggestion to modify the line of code of jqID

and include additional space in the list of characters which need be escaped:

I am very careful now with the editor used in the forum if one use backslashes in the code. So I repeat additionally in words, that I suggest just add ' ' (space character) in the mask. I think that this simple change will save many time for some developers who uses jqGrid.

Oleg

I examined all places of the code of jqGrid where id selectors like $("#" + any text) are used and made correction.

I hate GitHub web interface, "Git Gui" and "Git Bash" (or msysGit) because I couldn't found simple way to post some selected commits as the new pull request. In the pull request are merged many changes even the changes which I pooled from the main code of jqGrid (your code).

Nevetheless the GitHub can be used to documentate all the changes which I suggested:

• first of all I suggest to include " " (blank or space) to the list of the charackters which will be escaped (see here). CSS allows to include any escaped characters in the selectors (see here). jQuery UI just escapes any character which is not digit, underscore (_), hyphen (-) or english letter (A-Za-z) (see the line). I think that it's better to include explicitly all the characters which we need to escape.
• the current code of jqGrid stay the question about escaping of p.pager and p.toppager (see here). After analysing of the code which uses p.pager and p.toppager I can suggest to hold the values of the parameters in the form of ID selector which is escaped to be used directly in jQuery. If somewhere one need to get unescaped id one can use constructions like $(ts.p.pager).attr("id") or $(ts.p.pager)[0].id. It makes the usage p.pager and p.toppager simple enough and clear. So I suggest to make the following change: see here.
• The full changes in the code to use consequently $.jgrid.jqID devided in the changes in every module. The links below describes the changes for every module: grid.base.js, grid.common.js, grid.custom.js, grid.formedit.js and this one fix,  grid.grouping.js, grid.inlinedit.js, grid.jqueryui.js, grid.subgrid.js and grid.treegrid.js.

I hope that I found all the places of the usage of id selectors in the jqGrid code. After the includeing of the above changes I think one will be able to use any characters as ids or name property of colModel.

Best regards
Oleg