← Back to team overview

kicad-developers team mailing list archive

Re: wxAui improvements.

 

Le 09/12/2013 15:06, Wayne Stambaugh a écrit :
> On 12/9/2013 3:49 AM, Maciej Sumiński wrote:
>> On 12/07/2013 01:42 AM, Wayne Stambaugh wrote:
>>> I have just committed a small change to the testing branch (r4529) to
>>> the Pcbnew footprint viewer that will allow the main toolbar to be
>>> dockable.  I would like some feedback about how well saving and loading
>>> the window state between session works on various platforms before I
>>> start to do this to all of the KiCad application main frame windows.
>>> The wxAui stuff was put in place a few years ago with the intent that
>>> all of the fancy layout features would be enabled.  Since that hasn't
>>> happened I thought I would start working on it incrementally rather than
>>> get carried away an create a lot of problems.  When you have some spare
>>> time, please test it and let me know if there are any major issues.  I
>>> don't have wxWidgets 3 on my Linux partition and no access to OSX so
>>> feedback from those users is very important since I cannot test those
>>> platforms.
>>>
>>> Thanks,
>>>
>>> Wayne
>>
>> Hi Wayne,
>>
>> It is a great idea to have dockable toolbars. I tested it with wxWidgets
>> 2.8/Linux and the freshest wxWidgets 3.0 and in general it works fine
>> and the position of toolbar is saved.
>> One major problem I stumbled upon is that while using wx3.0, I somehow
>> lost the toolbar and now I cannot recover it. It could be an issue of my
>> window manager, but anyway - it could be good to have a menu where one
>> can toggle toolbars visibility.
>>
>> Regards,
>> Orson
>>
> 
> Hey Orson,
> 
> You should not be able to close the footprint viewer tool bar.  I
> intentionally have the close button disabled.  If you managed to close
> it, then there is a bug in wxWidgets.  What platform and wxWidgets
> version where you using when this occurred?  If I can duplicate it, I
> might be able to fix it.  As I apply this to the other KiCad apps, I
> will add menu entries to show tool bars that have been closed.
> Unfortunately the footprint viewer does not have a menu so I may have to
> do something about that in the future if we want to be able to hide the
> tool bars.  Thanks for the help with testing.
> 
> Cheers,
> 
> Wayne

I am using wxWidgets 3.0
I do not have this issue.
However I noticed 2 minor issues (both on Windows and Linux):

1 - When the footprint viewer tool bar is stored vertical, when the
footprint viewer is reopened, the tool-bar is displayed horizontally,
inside a large rectangle.
To reproduce this: move the tool-bar tho the left side, to have it
displayed vertically. then move it to the center of the windows. it is
still vertical. Close and reopen the viewer: the toolbar is horizontal,
inside a large square background.

2 - the second issue is relative to the optional tool to the right side
of the toolbar (the tool to import a footprint from the viewer to the
pcb editor).
It is a bit strange:
* if the toolbar is docked vertical at the left side, it is always
correctly displayed.
* if the toolbar is docked horizontally, it is not always correctly
displayed:
if previously the viewer was opened from the pcb editor tool bar, and
therefore with no import tool, when it is reopened from the footprint
load dialog (load from viewer), the import tool is not displayed (it is
still accessible from the overflow arrow)
Slightly moving the aui toolbar restores the correct display.
if you reopen the viewer from the main toolbar, the optional tool is not
displayed (as expected) but its area is visible in the toolbar.
(Slightly moving the aui toolbar restores the correct display again.)

Looks like an incorrect toolbar size was forced when reopen it.

-- 
Jean-Pierre CHARRAS


Follow ups

References