← Back to team overview

kicad-developers team mailing list archive

Re: Library Editor refactor

 

2017-11-21 10:22 GMT+01:00 Nick Østergaard <oe.nick@xxxxxxxxx>:

> 2017-11-21 <20%2017%2011%2021> 9:43 GMT+01:00 Maciej Sumiński <
> maciej.suminski@xxxxxxx>:
>
>> On 11/20/2017 11:16 PM, Nick Østergaard wrote:
>> > Hello Maciej,
>>
>> Hi Nick,
>>
>> > I have some comments:
>> >
>> > First of all, it looks quite neat.
>>
>> I am glad to hear that. Thank you for the test ride!
>>
>> > Secondly I noticed a small thing. When I open the symbol library editor
>> and
>> > it open in a small window and i resize it (make it bigger). The canvas
>> > contains some resizing artefacts, but if I zoom on step with the scroll
>> > whell and resize the window these aretefacts are not present. I am
>> > guessing: Maybe something needs to be initialized, or maybe it only
>> draws
>> > the viewable area initially, but have a bigger area cached after a zoom
>> > action.
>> >
>> > Thirdly it seems strange to me that when I edit something (move a pin)
>> and
>> > try to close the editor it asks me to selct a library. Hmm, ok? The
>> dialog
>> > is named "Save Libraries", but there is no OK or cancel button. This is
>> > stange and unexpected. This means that I can't really save or reject my
>> > changes.
>>
>> Indeed these two points are unexpected. Do you use an exotic window
>> manager? I tested the new library editor with awm (I consider it
>> somewhat unusual wm) on Linux, xfce on Linux and Windows 7. I could not
>> reproduce the problems on any of the mentioned configurations.
>>
>
> No, normally I use i3, but this time I used a default gnome3 desktop on
> archlinux. Dunno what the window manager is called nowadays.
>
> It is required to start in a floating small window to be able to see the
> effect from point my second point.
>
> So you say that the "Save Libraries" dialog gives you the ability to
> confirm or reject?  I mean I have the window managers close button but, the
> check boxes and the select all and select none buttons. But no Save/OK or
> Cancel button.
>

Intial window:
http://storage7.static.itmages.com/i/17/1121/h_1511280056_3771965_553e8fe830.png
When resizing:
http://storage1.static.itmages.com/i/17/1121/h_1511280096_3285438_9b4d6cf913.png

See the artifacts. They dissapear if I zoom.   This is not a regression of
your libedit branch.

Here the buttons are gone, but I think I can see some pixels of it...
http://storage2.static.itmages.com/i/17/1121/h_1511280153_9351314_c380fb9db0.png
If I resize the dialog the buttons apear fully. So you need to edit
multiple libs and then try to close libedit.


>
>> > Fourthly, when I have multiple libs open, and I seach it seems to alway
>> > unfold the last lib. I would probably have expected all libs with
>> matches
>> > in it being unfolded, such that I can see the entries I am searching
>> for.
>>
>> Agreed, this could be improved. It should not take much time to fix, I
>> will try to add it before merging.
>>
>> Regards,
>> Orson
>>
>> > I tested 0431a6fc6ce128973308638c1011bc95049fb173 from your libedit
>> branch
>> > on archlinux.
>> >
>> > Application: kicad
>> > Version: (2017-11-20 revision 0431a6fc6)-HEAD, debug build
>> > Libraries:
>> >     wxWidgets 3.0.3
>> >     libcurl/7.56.1 OpenSSL/1.1.0g zlib/1.2.11 libpsl/0.18.0
>> (+libicu/59.1)
>> > libssh2/1.8.0 nghttp2/1.27.0
>> > Platform: Linux 4.13.12-1-ARCH x86_64, 64 bit, Little endian, wxGTK
>> > Build Info:
>> >     wxWidgets: 3.0.3 (wchar_t,wx containers,compatible with 2.8) GTK+
>> 2.24
>> >     Boost: 1.65.1
>> >     Curl: 7.56.1
>> >     Compiler: GCC 7.2.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=OFF
>> >     KICAD_SPICE=ON
>> >
>> >
>> > Nick
>>
>>
>

Follow ups

References