Re: [Libreoffice] [PATCH] Display resolution in PPI with the original size on Picture/Crop (in Writer)

Hi László,

2011/10/28 Németh László <nemeth@numbertext.org>:
> Hi,
>
> This patch adds PPI resolution to the default image size data in the
> Picture dialog (only for bitmap images), see the attached screenshot.
>
> Explanation: LibreOffice supports explicite resolutions of JPEG/PNG
> picture formats to set default image size, but this important data is
> missing from the UI, so we cannot verify the picture quality.

Thanks for this patch, it looks a useful feature to me. You hard coded
a string into the code (e.g. "PPI"), this should be localized. I know
it is an international abbreviation, but some languages transcript
everything. Also, you assumed that you can replace the measurement
units, this is not the case when those are transcripted or translated.
If you don't have time to figure how to make this patch
localization-friendly, then I will work on it in soon. Please let me
know.

Thanks,
Andras