[vcl4php-developers] dbgrid.inc.php

[Gary <z0...@40...>]

 Submission for review:   

 I updated my easygrid templates ( 405hp.net/d4php/easygrid.php )
into the latest version of the vcl this week and I decided to do it in
a 2 step process - The first step was just the enhancements to
dbgrid.inc.php and the second the part that uses the templates.  

 405hp.net/comparevcl/dbgrid.zip It is kind of tough to do as a + -
type of thing so just compare the scripts in something like winmerge
to see what I changed. dbgrid.opt.inc.php is stage 1 dbgrid.inc.php is
the easy grid and dbgrid.std.inc.php is the stock one. To see it in
action so you can compare the generated dhtml. (I didn't bother
formatting the opt and std grids to be pretty as I never use the
column editor but you can get the idea.) Only difference in each of
these scripts is the name of the include file.
405hp.net/comparevcl/compare_easy.php   makes 44.9k dhtml 10k
compressed 405hp.net/comparevcl/compare_opt.php makes 43.1k dhtml
405hp.net/comparevcl/compare_std.php makes 153k dhtml
 16k compressed
I did add 2 properties - column visibility button and status line
visible at the bottom. Other than that it is just code compaction and
rearrangement. Discussion:  

 oData - I do think the browser footprint could be reduced even more
if this was eliminated and I'm all for deleting it unless someone can
come up with a valid reason to keep it. I think it would be easy to
add a next/prev pagination to this with the json rpc (ajax) code if
anyone thought it would be worth while, just need some feedback on
usability. A further wonder - can class CustomDBGrid extends
CustomGrid and use_unit("grids.inc.php"); be removed from this? I
can't find where they are used for anything.