[Merkaartor] Patch: GeoImage displayed with anti-alias

[Timo Schlüßler]

Hello,

I implemented your suggestions and checked them in. See revision 13926 
and the CHANGELOG.
Images are now painted with QPainter's renderHint set to 
QPainter::SmoothPixmapTransform through which your desired anti-aliasing 
is achieved. Also the image isn't always copied when it is painted which 
makes moving and zooming significantly faster.
Further on I changed the zooming implementation. It is now based on a 
percantage basis and when zooming in/out the zoom-value changes similar 
to  zoom-values of other applications (like gimp or okular) and is 
limited to 25600%.
The image filename is stripped to fit the width of GeoImageDock and 
three dots are placed at the beginning when it is trimmed to indicate 
the trimming.

I would be glad if you can test my changes and report errors :)

regards, timo

Yves Goergen wrote:
> >From what I've read, the support for (geo-tagged) photos is fairly new.
> But somehow I've got the impression that it's more of a technology
> preview. Zooming behaves unforeseeable at extreme values and it has no
> limits. The image is drawn in poor quality and the filename doesn't even
> begin to fit in the small dock window. Also, moving the image in the
> view takes 100% CPU because it's recreated in every paint event.
>
> So this is a partial fix to improve the situation. It adds anti-aliasing
> to the display (much better reabability of street name signs now at any
> zoom level) and shortens the filename by cutting off the path. It does
> not address handling or performance flaws. The latter would possibly
> require wide rewriting of the zoom and pan code. (Or maybe not, if I
> only understood what it does.)
>
> The photos support is very important to me because this is the way I'm
> making notes in the field. Without this feature, I'm doing very hard
> knowing what I've seen where. Also, I'm not actually deciding on
> tracktypes when I'm there, but I just take a photograph of it and then
> compare it with the tracktypes reference at the computer.
>
>   
> ------------------------------------------------------------------------
>
> _______________________________________________
> Merkaartor mailing list
> Merkaartor at openstreetmap.org
> http://lists.openstreetmap.org/listinfo/merkaartor
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/merkaartor/attachments/20090228/7a6a01d2/attachment.html>