Guriddo » Topic: Support an “All” option for the rowList parameter

Support an “All” option for the rowList parameter

Guriddo
Support an “All” option for the rowList parameter

This topic has 4 replies, 4 voices, and was last updated 14 years, 4 months ago by OlegK.

Creator

Topic
March 3, 2010 at 9:53 am #75640

dw
Participant

I
Creator

Topic

Viewing 4 replies - 1 through 4 (of 4 total)

Author

Replies
September 15, 2011 at 12:34 pm #98801

boogey100
Participant

Perhaps allowing it to be an object rather than an array would do it, so you could have { '10':10,'20':20,'50':50,'All':-1 }. My gues is the only thing that would need changing is the bit that builds the in the pager.

September 15, 2011 at 1:21 pm #98802

OlegK
Participant

Hello!

I agree with you. The same requirement have the most customers. The usage of “-1” is not allowed and can't be used, but one can use some large enough value and

January 24, 2012 at 9:35 am #99406

fabamb
Participant

May I suggest an improvement?
Instead of setting an arbitrary value, which is always wrong, you can set as a sentinel value 0 for 'All' and manage server-side, correctly. On the other hand, often in paging systems, it works without having to do anything.

rowList: [5, 10, 20, 50, 1000, 0], pager: '#pager

$("#pager option[value=0]").text('All');

January 24, 2012 at 12:45 pm #99409

OlegK
Participant

Hello fabamb,

I don't think that one should include in the code of jqGrid any changes which break the current server code. Your suggestion require changes on the server side because the code SELECT TOP(0) will work not good. On the other side if you use “SQL Server Management Studio” for example it uses per default TOP(1000) in all your selects of the tables. It's good because mostly you will not look through all returened rows and examine only some first rows. So I think that the usage of some large enough value like 5000 will be better. One can include the value as a new jqGrid option together with the text 'All'
Author

Replies