← Back to team overview

kicad-developers team mailing list archive

Re: PATCH: Unify mirror and copy block functions between schematic editor and library aditor

 

Hi,

Sorry about that, I'll update the patch to comply to the coding policy.
I'm currently investigating the rotate block function. It seems to be
working only is the selection rectangle lands on a pin location but doesn't
work otherwise. I'm trying to figure out why this appears only for the
rotate block function and not the others.

Thanks for reviewing it.

Cheers,
Mikael

On Tue, Mar 1, 2016 at 6:56 AM, Wayne Stambaugh <stambaughw@xxxxxxxxx>
wrote:

> Mikael,
>
> I just tested your patches and they do unify the block mirror behavior
> but the rotate hot key (R) does not rotate a block in component library
> editor.  I would prefer that you fix that before I commit your patches.
>  Also, you have some copy policy violations (trailing white space, curly
> bracket placement, and missing spaces between text and braces) in your
> patch.  Since it's your first patch, I will cut you some slack but in
> the future please submit patches that to adhere to the coding policy[1].
>
> Cheers,
>
> Wayne
>
> [1]:
>
> http://ci.kicad-pcb.org/job/kicad-doxygen/ws/Documentation/doxygen/html/md_Documentation_development_coding-style-policy.html
>
> On 2/24/2016 12:43 PM, Mikael Arguedas wrote:
> > Hi guys,
> >
> > Here are two patches:
> > unifyCopyBlock.patch: makes the CopyBlock function in library editor
> > match the behaviour of the one in schematic editor
> > addMirrorHotkeys.patch:
> > - adds the HkMirrorX and HkMirrorY hotkeys to library editor
> > - allor the HkRotate hotkey to rotate blocks and not only items
> >
> > Regards,
> > Mikael
> >
> >
> > _______________________________________________
> > 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
> >
>
> _______________________________________________
> 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
>

Follow ups

References