eeschema: edit graphic line style on drawing

[Eldar Khayrullin]

On drawing graphic line if try to edit (push 'E' key) style of line (to solid for example) don't edit style of second line.

Application: Eeschema
Version: 6.0.0-unknown-a0ed408~100~ubuntu19.04.1, release build
Libraries:
    wxWidgets 3.0.4
    libcurl/7.64.0 OpenSSL/1.1.1b zlib/1.2.11 libidn2/2.0.5 libpsl/0.20.2 (+libidn2/2.0.5) libssh/0.8.6/openssl/zlib nghttp2/1.36.0 librtmp/2.3
Platform: Linux 5.0.0-25-generic x86_64, 64 bit, Little endian, wxGTK
Build Info:
    wxWidgets: 3.0.4 (wchar_t,wx containers,compatible with 2.8) GTK+ 3.24
    Boost: 1.67.0
    OpenCASCADE Community Edition: 6.9.1
    Curl: 7.64.0
    Compiler: GCC 8.3.0 with C++ ABI 1013

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

[Michael Kavanagh (michaelkavanagh)]

There is something strange here, it only seems to affect the line style before the first vertex? And the new line style is not applied to subsequent lines.

[Wayne Stambaugh (stambaughw)]

Why would you expect property changes to apply to subsequent lines? In Eeschema, all lines are unique elements so you can have lines with different settings. Once the new property inspector code is implemented, you will be able to select multiple lines to set properties.

[Michael Kavanagh (michaelkavanagh)]

It feels wrong because you can draw two line segments at once in "Restrict buses and wires to H and V orientation" mode. Turning this off makes the behaviour feel more natural, but changing the current line style does sometimes change the previously drawn line segments as well.

[Wayne Stambaugh (stambaughw)]

@Michael, it may feel wrong but it is currently how lines are stored and drawn. I'm not sure making all of the lines have the same properties is going to feel any less wrong. Since they are stored as separate lines, there is no way to tell if they were originally drawn as a single polygon or as a single line. We can change that with the new file format but not it the current file format.

[Wayne Stambaugh (stambaughw)]

On 8/21/19 1:00 PM, Wayne Stambaugh wrote:
> @Michael, it may feel wrong but it is currently how lines are stored and
> drawn. I'm not sure making all of the lines have the same properties is
> going to feel any less wrong. Since they are stored as separate lines,
> there is no way to tell if they were originally drawn as a single
> polygon or as a single line. We can change that with the new file
               ^^ multiple single lines." ^^
> format but not it the current file format.
>

[KiCad Janitor (kicad-janitor)]

KiCad bug tracker has moved to Gitlab. This report is now available here: https://gitlab.com/kicad/code/kicad/-/issues/3427