Thanks, I applied the patch with some modifications in r10989. > * Clicking outside the cell range (grid background) doesn't cause > Click to fire (delphi compatible) > * BUG fix: clicking outside the cell range doesn't cause the cell > focus to change (delphi compatible) .... > * Setting Row or Col out of range raises an "Grid index out of > range." > error and doesn't change the cell focus. (delphi compatible). See > note below!
I don't see this as a bug, but as an enhacement so for users that want to improve delphi compatibility I added a public property AllowOutboundEvents. Set this to false and it will be more strict, delphi compatible. Btw I did a little edit in grids wiki page about improving delphi compatibilty: http://wiki.lazarus.freepascal.org/Grids_Reference_Page#Adjustments_for_improving_Delphi_grids_compatibility > * MouseToGridZone - added Begin..End pairs to improve clarity to > you > can actually see what if/else fits where. > Sorry, I removed some begin..end as I really don't see any clarity improvement. Though I don't have anything serious against "begin" in new line (Oh well I do but it's not serious, I think that waste alot of space, yet I even use it sometimes) as for the coding convention in grids most code is written with "if stuff then begin" so at the moment, this is prefered. Maybe in future all Lazarus code will adopt a single Code convention?, don't know. > > Note: > Setting the Row or Col out of range. I believe Delphi has a bug > here. > In delphi you can set the Row or Col to a fixed row or column, > which > then causes a Click event, but then the focus is screwed. We can > decide at a later date what to do in the LCL - for now Click events > will fire, but the focus cell will be the first non-fixed cell. > > > -- > Graeme Geldenhuys > > There's no place like S34° 03.168' E018° 49.342' Jesus Reyes A. __________________________________________________ Correo Yahoo! Espacio para todos tus mensajes, antivirus y antispam ¡gratis! Regístrate ya - http://correo.yahoo.com.mx/ _________________________________________________________________ To unsubscribe: mail [EMAIL PROTECTED] with "unsubscribe" as the Subject archives at http://www.lazarus.freepascal.org/mailarchives
