[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kopete-devel
Subject:    Re: [kopete-devel] saving photo to disk on each message
From:       Jason Keirstead <jason () keirstead ! org>
Date:       2005-02-17 20:42:28
Message-ID: 200502171642.28324.jason () keirstead ! org
[Download RAW message or body]

On Thursday 17 February 2005 4:31 pm, Olivier Goffart wrote:
> In my case, most photo are from MSN, and then are on the disk.

Yes, but this is not the majority I don't think. Anyone who has photos from 
anywhere else have them stored inside the VCard, because this is where they 
are stored if you set the photo using KAddressbook itself.

For example, if you add the VCard of the Kopete developers Will made as a 
remote resource - all Base64 photos.

> > It would only store one photo per contatct...
>
> Per contact * messages.

> the KTempFile isn't afaik shared between messages.

With my proposal.. it would need to be only one  photo / contact, unless the 
contact's photo changed in the middle of the chat

> I would say that the chat window show the contact, in case the contact
> change his photo, the photo should be updated in the whole chat window.

That isn't what other IM clients do.. the photo in the chat is the photo at 
the time the message was sent.

> 2) the metaContact::photo()  should return something like a KABC::Picture
> which could return both a QString or a QImage.    one could add in this
> Kopete::Picture class a function to convert to a QString, or a QImage.
>      - It require some change in Kopete which is probably not good now
>     + It's the IMO the best way to do.

Fine, we could add this class and do this, but I don't think it is woth it. 
The only case it will change is MSN, and it will only save a few KB at most.

-- 
If you wait by the river long enough, eventually
you will see the bodies of all your enemies float by.
    - Sun Tzu
_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel
[prev in list] [next in list] [prev in thread] [next in thread]