← Back to team overview

kicad-developers team mailing list archive

Re: Current state of ActionPlugin

 

Hi Maciej,

> 
> I think your patch will facilitate use of 3rd party python plugins, as
> the current way of executing commands from the Python shell is not quite
> user friendly. I vote for merging the patch, but we need to fix some
> code formatting issues first.
Ok.

> 
> Also, I wonder if it would be the right thing to group menu entries by
> their categories to create submenus. It could be a simple way to
> organize the actions provided by python plugins. Just an idea.
I started without submenu but I can add it.

> 
> Another question is about the following lines:
> +        if( IsGalCanvasActive() )
> +        {
> +            UseGalCanvas( GetGalCanvas() );
> +        }
> 
> What is exactly the goal here? If it is only about refreshing the
> canvas, then a simple Refresh() call should fix the problem.
I fighted many time with refresh and other solution but nothing was
working. The only solution I found was this last one.



Attachment: signature.asc
Description: OpenPGP digital signature


References