"Choose Footprint" does not use anti-aliasing settings specified

Bug #1813973 reported by Thomas Pointhuber on 2019-01-30
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Undecided
Jeff Young

Bug Description

The dialog "Choose Footprint" shows aliasing artefacts which are not present in pcbnew (Grid dots of different size).

The "Library Browser" does not seem to have such grid artefacts, but it does not use the settings specified for the pcbnew/footprint editor as well.

# Version information

Application: kicad
Version: (6.0.0-rc1-dev-1645-gb3319d29c), debug build
Libraries:
    wxWidgets 3.1.1
    libcurl/7.63.0 OpenSSL/1.1.1a zlib/1.2.11 libidn2/2.1.0 libpsl/0.20.2 (+libidn2/2.1.0) libssh2/1.8.0 nghttp2/1.35.1
Platform: Linux 4.20.4-arch1-1-ARCH x86_64, 64 bit, Little endian, wxGTK
Build Info:
    wxWidgets: 3.1.1 (wchar_t,wx containers) GTK+ 3.24
    Boost: 1.69.0
    OpenCASCADE Community Edition: 6.9.1
    Curl: 7.63.0
    Compiler: Clang 7.0.1 with C++ ABI 1002

Build settings:
    USE_WX_GRAPHICS_CONTEXT=OFF
    USE_WX_OVERLAY=OFF
    KICAD_SCRIPTING=ON
    KICAD_SCRIPTING_MODULES=ON
    KICAD_SCRIPTING_PYTHON3=ON
    KICAD_SCRIPTING_WXPYTHON=ON
    KICAD_SCRIPTING_WXPYTHON_PHOENIX=ON
    KICAD_SCRIPTING_ACTION_MENU=ON
    BUILD_GITHUB_PLUGIN=ON
    KICAD_USE_OCE=ON
    KICAD_USE_OCC=OFF
    KICAD_SPICE=ON

KiCad Janitor (kicad-janitor) wrote :

Fixed in revision ba7b970817a284d7266d868de9b3bcd66a45e634
https://git.launchpad.net/kicad/patch/?id=ba7b970817a284d7266d868de9b3bcd66a45e634

Changed in kicad:
status: New → Fix Committed
assignee: nobody → Jeff Young (jeyjey)
Thomas Pointhuber (pointhi) wrote :

@Jeff,

does not seem to be fixed on my system. I still see the obvious differences in the grid. I think the viewer is using cairo while I'm using OpenGL in the footprint editor. Nevertheless, the cairo Anti-aliasing settings are not used in the viewer as well.

Jeff Young (jeyjey) wrote :

Hmm... that last commit was for a different bug. Must have fat-fingered the copy/paste.

In any case, this is because the canvas is hard-coded for the Footprint Viewer. We can probably do better now that we have Orson's EDA_DRAW_FRAME::LoadCanvasTypeSetting() stuff....

Changed in kicad:
status: Fix Committed → In Progress
milestone: none → 5.1.0
Jeff Young (jeyjey) wrote :

@Thomas, it attempts to read the anti-aliasing settings from PCBNew if it's running, otherwise from the config. Does it fail for you in both cases, or only one of them?

KiCad Janitor (kicad-janitor) wrote :

Fixed in revision b6a06aa1291b4cb22c1f54ed8e382a0d69de5bc1
https://git.launchpad.net/kicad/patch/?id=b6a06aa1291b4cb22c1f54ed8e382a0d69de5bc1

Changed in kicad:
status: In Progress → Fix Committed
Thomas Pointhuber (pointhi) wrote :

@Jeff,

seems to work fine in the windows which are using the settings.

But I have to note the settings do not affect all windows. Some of them always use the grid style and ignore user settings:

* Footprint Library Browser
* CvPcb

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