EEschema: mirror icons inverted

Bug #1828631 reported by Michele Giorato
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
KiCad
Fix Committed
Low
Michael Kavanagh

Bug Description

I noticed that doing the multiselection and pressing the right mouse button the menu appears with the inverted mirroring icons.

I always keep wondering why don't you put a simple Horizontal and Vertical flip in place of "Mirror around vertical axis" and "Mirror around horizontal axis".

See attached snapshot

Application: kicad
Version: 6.0.0-unknown-r15630-3caa4376, debug build
Libraries:
    wxWidgets 3.0.4
    libcurl/7.64.0 OpenSSL/1.1.1b zlib/1.2.11 brotli/1.0.7 libidn2/2.1.1 libpsl/0.20.2 (+libidn2/2.0.5) libssh/0.8.7/openssl/zlib nghttp2/1.38.0
Platform: Linux 5.0.13-300.fc30.x86_64 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.69.0
    OpenCASCADE Community Edition: 6.9.1
    Curl: 7.64.0
    Compiler: GCC 9.1.1 with C++ ABI 1013

Build settings:
    USE_WX_GRAPHICS_CONTEXT=OFF
    USE_WX_OVERLAY=ON
    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

Tags: eeschema
Revision history for this message
Michele Giorato (mr-miky64) wrote :
Revision history for this message
Jeff Young (jeyjey) wrote : Re: [Bug 1828631] [NEW] EEschema : multiselect mirror inverted

The terminology problem originates with PCBNew where “flip” means move to the other side of the board.

That being said, it’s not clear to me that using the same term for both is worse than the confusion over mirroring.

Changed in kicad:
importance: Undecided → Low
milestone: none → 6.0.0-rc1
status: New → Triaged
tags: added: eeschema
Revision history for this message
Michael Kavanagh (michaelkavanagh) wrote : Re: EEschema : multiselect mirror inverted
summary: - EEschema : multiselect mirror inverted
+ EEschema: mirror icons inverted
Revision history for this message
KiCad Janitor (kicad-janitor) wrote :

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

Changed in kicad:
status: Triaged → Fix Committed
assignee: nobody → Michael Kavanagh (michaelkavanagh)
Revision history for this message
Wayne Stambaugh (stambaughw) wrote :

@Michael, I merged your patch. Thanks.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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