Copy/paste PCB text broken in several ways

Bug #1786727 reported by Electro Twelve
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
KiCad
Fix Released
High
Jeff Young

Bug Description

In version 4.0.7, you could duplicate text items on the F/B.Silk layers with the hotkey C. In this version with the Legacy toolkit enabled, this does not work. You have to right-click the item, say Copy and then it works. Also in the Modern toolset (Accelerated) this hotkey activates Zone Fill option.

Version specifics:

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 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:
    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

Running KiCad on Windows 10, build 1803.

Tags: pcbnew
Changed in kicad:
milestone: none → 5.0.1
tags: added: pcbnew
Revision history for this message
Jeff Young (jeyjey) wrote :

I can confirm that 'C' doesn't do a copy with either Toolset (the text layer isn't material).

I can't reproduce the "fills zones" issue in the Modern Toolset. You might need to reset your hotkeys.

With the Modern Toolset there are additional issues:

Select a PCB text item.
Right-click > Copy
Move mouse somewhere else and click.
Ctrl-V
Don't move mouse: can't see pasted text item.
Move mouse: pasted text item appears.

Select a different PCB text item.
Right-click > Copy
Move mouse somewhere else but do NOT click.
Ctrl-V
Move mouse and note that wrong text was pasted (it's the first one)

Select a different PCB text item.
Right-click > Copy
Move mouse somewhere else and CLICK.
Ctrl-V
Move mouse and note that correct text was pasted.

Application: kicad
Version: (6.0.0-rc1-dev-354-gddbf8e3ff-dirty), debug build
Libraries:
    wxWidgets 3.0.4
    libcurl/7.54.0 LibreSSL/2.0.20 zlib/1.2.11 nghttp2/1.24.0
Platform: Mac OS X (Darwin 17.7.0 x86_64), 64 bit, Little endian, wxMac
Build Info:
    wxWidgets: 3.0.4 (UTF-8,STL containers,compatible with 2.8)
    Boost: 1.66.0
    Curl: 7.57.0
    Compiler: Clang 9.1.0 with C++ ABI 1002

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

summary: - Copy text items with the hotkey "C" on F/B.Silk layer does not work
+ Copy/paste PCB text broken in several ways
Changed in kicad:
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Jeff Young (jeyjey) wrote :

The click/not click issue is because the EDIT_TOOL goes into "pick reference point for block" mode, but this isn't terribly obvious if you're not looking at the status bar.

I don't think reference-point-copy should be the default action. Perhaps a modifier-key option?

And for modes like this, I think the Length-Tuning-Status-Box might be a better way to communicate the mode.

There's also no context menu for the mode, which means there's no visual way to cancel it (esc does work).

Revision history for this message
Jeff Young (jeyjey) wrote :

Ha! The not-drawing-till-you-move thing turned out to be my bug -- the ClientSelectionFilter was inadvertently clearing the selection reference, so it was getting drawn the first time at (0, 0).

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

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

Changed in kicad:
status: Confirmed → Fix Committed
assignee: nobody → Jeff Young (jeyjey)
Revision history for this message
Jeff Young (jeyjey) wrote :

(Note that the above bug didn't exist in 5.0, so there's nothing to merge on that front for 5.0.1.)

Changed in kicad:
status: Fix Committed → Confirmed
Revision history for this message
Jeff Young (jeyjey) wrote :

I've also fixed the lack of a Context Menu when running Select Anchor.

Revision history for this message
Jeff Young (jeyjey) wrote :

... and changed the status message to a popup (both for this and for Position Relative To's anchor picker).

This one's probably carries a bit too much risk and not enough reward for 5.0.1 so I'll leave it 5.1-only.

Revision history for this message
Jeff Young (jeyjey) wrote :

The remaining part appears to be due to the fix for https://bugs.launchpad.net/kicad/+bug/1749549, so I'm going to reopen that one and let this one close.

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

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

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

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.