[Lazarus] TFont.Assign not complete

[Vojtěch Čihák]

Hi,
 
IMO it should be OK. PixelsPerInchs has probably meaning when you have two displays with different DPI. Therefore copying the value PixelsPerInchs is wrong because method Assign() cannot know where the font will be displayed (on the same display or the second display with different DPI).
 
V.
 
______________________________________________________________
> Od: Alexey via Lazarus <lazarus at lists.lazarus-ide.org>
> Komu: Lazarus mailing list <lazarus at lists.lazarus-ide.org>
> Datum: 30.05.2017 15:10
> Předmět: [Lazarus] TFont.Assign not complete
>
Not sure it is ok, why not to a) copy Size, b) copy PixelsPerInch. why 
 such strange logic.
 
 
           if TFont(Source).PixelsPerInch <> FPixelsPerInch then
             // use size to convert source height pixels to current 
 resolution
             Size := TFont(Source).Size
           else
             // use height which users could have changed directly
             Height := TFont(Source).Height;
 
 Also good to copy PixelsPerInch to do good copy
 
 -- 
 Regards,
 Alexey
 
 -- 
 _______________________________________________
 Lazarus mailing list
 Lazarus at lists.lazarus-ide.org
 http://lists.lazarus-ide.org/listinfo/lazarus <http://lists.lazarus-ide.org/listinfo/lazarus>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus-ide.org/pipermail/lazarus/attachments/20170530/8d7b6490/attachment.html>