[lazarus] CVS Committed

[Mattias Gaertner]

Hi all,

I think, the TImageList is a very important class used in many nice
freeware components, and thus should be compatible to Delphi.
AFAIK the corrupted images was/is not really Borlands fault. Some graphic
device drivers had got problems with Bitmaps width greater than a few
thousands.
But I also want a TImageList which can store images with different sizes. I
see two possibilites:
  -A flag in TImageList.
  -A new component. Perhaps named TImageCollection.

Mattias


> >One reason I would personally prefer an implementation like this is to
allow
> >adding images of different sizes simultaneously.  By throwing all of
the
> >images onto a common bitmap (how Delphi does this), you have all kinds
of
> >problems trying to resize the base bitmap or cropping when images don't
> >exactly fit the dimensions you have chosen for the image list.

> 
> This way you loose compatibility with delphi. Now you have made your own

> imagelist (in fact a TList with images) The more I think about it the
less 
> advantages I see in the new solution. When I implemented the original 
> implementation I also tought about storing separate images, but if you
want 
> to be compatible you have to implement all methods and props and
behaviour 
> of the standard imagelist. In that light I didn't see  an advantage of 
> storing  images separate. (The amount of code will be the same ifnot
more)
> 
> 
> >One other problem that I have in general with the Delphi implementation
is
> >that on some occasions the bitmap gets corrupted, requiring me to have
to 
> >load in all of the bitmaps all over again one at a time... and
remembering 
> >what their item indexes were so I don't mess up any controls relying on

> >the image list, but that is beyond the scope of this thread.
> 
> I never had that problem (not in a normal situation, where your app
behaves 
> normally)
> 
> Marc
> 
> _________________________________________________________________
>      To unsubscribe: mail lazarus-request at miraclec.com with
>                 "unsubscribe" as the Subject
>     archives at http://www.miraclec.com/list_archives/lazarus