Sloppy snapping when moving dimension

[Tim N. Tashpulatov]

Not sure how to describe this one properly, here it goes: When moving a Dimension previously placed on some grid, and when trying to select one of Dimension's drag points (which are perfectly on the grid) the selection does not snap precisely but rather shifts a bit.

This does not seem to affect the Dimension after moving (the Length reported is rock solid), it's that it can be somewhat annoying when dealing with fine grids of 0.127mm and lesser.

Application: kicad
Version: (5.0.0), release build
Libraries:
    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 7 (build 7601, Service Pack 1), 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:
    USE_WX_GRAPHICS_CONTEXT=OFF
    USE_WX_OVERLAY=OFF
    KICAD_SCRIPTING=ON
    KICAD_SCRIPTING_MODULES=ON
    KICAD_SCRIPTING_WXPYTHON=ON
    KICAD_SCRIPTING_ACTION_MENU=OFF
    BUILD_GITHUB_PLUGIN=ON
    KICAD_USE_OCE=ON
    KICAD_USE_OCC=OFF
    KICAD_SPICE=ON

[Seth Hillbrand (sethh)]

I suspect that this one has been addressed

Can you try one of the 5.0 nightly builds from:
http://downloads.kicad-pcb.org/windows/testing/5.0/

[Tim N. Tashpulatov (tim-tashpulatov)]

Tried with 5.0.1 build from /testing/5.0, same behavior.

More details on the issue: set grid to 0.127mm (does not matter actually), place a Dimension object on any layer. Move cursor a bit off Dimension's node, press 'm'. The Dimension is now being picked for moving, with no grid snapping.

Please see another screenshot attached.

Application: pcbnew
Version: (5.0.1-dev-148-g1f19903e7), release build
Libraries:
    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 7 (build 7601, Service Pack 1), 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:
    USE_WX_GRAPHICS_CONTEXT=OFF
    USE_WX_OVERLAY=OFF
    KICAD_SCRIPTING=ON
    KICAD_SCRIPTING_MODULES=ON
    KICAD_SCRIPTING_WXPYTHON=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)]

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

[Tim N. Tashpulatov (tim-tashpulatov)]

Great to see the bug fixed so fast! Or at least reported to be fixed :)

However correct snapping happens only once, subsequent snappings do not occur unless pcbnew is restarted.

Tested with fresh kicad-5.0-jenkins-30-x86_64.exe as of 05-Oct-2018 00:15 (maybe a little bit early?)

Application: pcbnew
Version: (5.0.1-dev-160-g5e0ef3671), release build
Libraries:
    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 7 (build 7601, Service Pack 1), 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:
    USE_WX_GRAPHICS_CONTEXT=OFF
    USE_WX_OVERLAY=OFF
    KICAD_SCRIPTING=ON
    KICAD_SCRIPTING_MODULES=ON
    KICAD_SCRIPTING_WXPYTHON=ON
    KICAD_SCRIPTING_ACTION_MENU=ON
    BUILD_GITHUB_PLUGIN=ON
    KICAD_USE_OCE=ON
    KICAD_USE_OCC=OFF
    KICAD_SPICE=ON

[Jeff Young (jeyjey)]

@Tim, "Fix Committed" means the bug fix is available in the development branch (and therefore in the nightly builds). "Fixed Released" would mean it is available in the stable build.

[Tim N. Tashpulatov (tim-tashpulatov)]

@Jeff, thanks for explanations, and sorry for being impatient.