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@lists.lazarus-ide.org>
Komu: Lazarus mailing list <lazarus@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@lists.lazarus-ide.org
http://lists.lazarus-ide.org/listinfo/lazarus 
<http://lists.lazarus-ide.org/listinfo/lazarus>

-- 
_______________________________________________
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
http://lists.lazarus-ide.org/listinfo/lazarus

Reply via email to