[Lazarus] Sorting StringGrid by column click

Juha (gmail) juha.manninen62 at gmail.com
Fri May 13 23:55:19 CEST 2011


Jesus Reyes kirjoitti perjantai, 13. toukokuuta 2011 21:28:00:
> I'm sorry this patch must be reverted, there is no reason to send SortOrder
> as parameter when SortOrder is a grid's public property.

Your wiki documentation said SortOrder is not supported when using custom 
sort, so I figured there is some situation when the user's code can't read a 
grid's data. (I have changed the wiki page since that).

> And as you said it unnecessarily breaks code, for example semaphore grid
> which is included with Lazarus which by the way has exteneded sorting
> capabilities.

I will revert it. I also don't see any use for it, now that I think of it.

> I'm sorry I still have to review the code been too busy lately, but I was
> thinking that maybe we don't need default icons for ordering direction,
> first we need to check if the lcl do not use some similar resources, 

Maybe arrow_down.png and arrow_up.png images/action dir.
Are they always present when LCL is built or should they be copied.

> if it doesn't, I think as a default we could just draw it ourselves.

Yes, draw or copy a free image from somewhere.

Juha




More information about the Lazarus mailing list