>- adds soVisibleOnly to SaveOptions
>- with soVisibleOnly saves only visible columns
-> without (default) it works as before


I think using some SaveOptions options only for XML and some only for CSV might 
cause confusion.

The problem with SaveOptions is that it's supposed to apply only to 
Save/LoadFromFile (I wish to have named SaveToCSVFile as ExportToCSVFile 
instead) visibility on CustomColumns should be preserved on saving/loading 
to/from a XML file (that by the way is missing), but it's ok to export only 
visble columns to  CSV file. So what about adding yet another parameter to CSV 
functions?,

Even, if more features continue to be added to CSV support, it may be better to 
move the whole CSV support out of grids unit and create a component or 
something.

What you think.

Jesus Reyes A.

--
_______________________________________________
Lazarus mailing list
[email protected]
http://lists.lazarus.freepascal.org/mailman/listinfo/lazarus

Reply via email to