← Back to team overview

kicad-developers team mailing list archive

Re: KiCad new look - Moveable button bars


On 8/29/2011 5:44 AM, hauptmech wrote:
> I have a good grip on the wxAui toolbar api now. The toolbar setup code
> for pcbnew was making some erroneous assumptions about what the api was
> doing and it's amazing it produced working results. 
> I pushed a branch called 'moveable_panels' that I need some feedback on
> before I procede. I'm moving the panel initialization code to a central
> location so all the apps can benifit and since I'm not a wizard with C++
> idioms I'd like a second opinion.


I'm all for giving the user the flexibility to rearrange their UI however they
see fit.  That is one of the main advantages of using wxAUI.  However, if the
application does not preserve it's last state, then this feature is of limited
benefit.  Most users will not find it useful if the have to rearrange things
every time they run a Kicad application.  I mentioned this when I responded to
your previous merge request.  Have you implemented the save and restore session
settings as part of this branch?  If so, then I am very much interested.

> The branch should compile a pcbnew with tearable, moveable panels.
> If it looks good I'll propagate it to all the apps with comments and
> request a merge.

I would prefer this as a patch that I can apply against my working branch
rather than download an entire new branch of Kicad and rebuild the entire
source tree just to test a relatively small change.



> regards,
> Hauptmech
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp