Page layout bitmap size wrong in modern/GAL canvases

Bug #1798685 reported by Michael Kavanagh
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fix Released
jean-pierre charras

Bug Description

My page layout file has 3 bitmaps (logos). These were scaled down by adjusting the DPI in the page layout editor (weird as I now cant find that option ), since it improved display when zooming in on plotted PDFs.

These display fine in the legacy canvas in Eeschema and Pcbnew but appear massive in the modern canvases.

I can send the files if it'd help but would rather not upload them online.

(Actually running Windows 10 despite what it says below).

Application: kicad
Version: (5.0.1)-3, release build
    wxWidgets 3.0.3
    libcurl/7.54.1 OpenSSL/1.0.2l zlib/1.2.11 libssh2/1.8.0 nghttp2/1.23.1 librtmp/2.3
Platform: Windows 8 (build 9200), 64-bit edition, 64 bit, Little endian, wxMSW
Build Info:
    wxWidgets: 3.0.3 (wchar_t,wx containers,compatible with 2.8)
    Boost: 1.60.0
    OpenCASCADE Community Edition: 6.8.0
    Curl: 7.54.1
    Compiler: GCC 7.1.0 with C++ ABI 1011

Build settings:

tags: added: pcbnew
Changed in kicad:
milestone: none → 5.0.2
Revision history for this message
Maciej Suminski (orsonmmz) wrote :

Hi Michael,

Please send me the files exhibiting the problem. I am in process of fixing bitmap scaling in printouts, so this might be related.

Changed in kicad:
assignee: nobody → Maciej Suminski (orsonmmz)
Revision history for this message
jean-pierre charras (jp-charras) wrote :

Hi Orson,

Bitmap images have a scaling factor (the PPI is set to 300, and the actual drawing size is size_for_300_ppi * scaling factor)
For instance, if in page layout editor you set the ppi to 100, the scaling factor is 3.0

Unfortunately, when I added the draw bitmap function in GAL, I forgot to take in account this scaling factor

I also just saw the recent (5.0) page layout editor the PPI setting is no longer displayed, due to and issue in the .fbp file.
(But works in 4.0 version)

I'll commit a fix very soon for the page layout editor issues.

Revision history for this message
KiCad Janitor (kicad-janitor) wrote :

Fixed in revision dafeb9676535a545352f2411af0acaec36d3d873

Changed in kicad:
status: New → Fix Committed
assignee: Maciej Suminski (orsonmmz) → jean-pierre charras (jp-charras)
Revision history for this message
Maciej Suminski (orsonmmz) wrote :

Thank you Jean-Pierre!

Changed in kicad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers