← Back to team overview

kicad-developers team mailing list archive

Re: Still banging on about undo....

 

Hi Tom,

I don’t have a pre-conceived notion about what “the rest” would entail.  Certainly not zoom or layer visibilities.  But maybe layer set and layer colours.  Actual list to be determined….

Huh, I had thought Get/SetPosition were in the EDA_ITEM, but you’re right, they’re up in ORIGIN_VIEWITEM.  I’ll look into refactoring as suggested.

Cheers,
Jeff.

> On 23 Jan 2018, at 21:46, Tomasz Wlostowski <tomasz.wlostowski@xxxxxxx> wrote:
> 
> On 23/01/18 22:38, Jeff Young wrote:
>> I’d like to get the Drill & Grid origin implementation in for 5.0, and
>> I’m happy to volunteer to do the rest for 6.0.
> 
> Hi Jeff,
> 
> What do you mean by 'the rest'?
> 
> Personally, I'm not opposed to undoing grid origin changes, but the 'all
> that is in the file' approach you asked about on the forum seems like
> going a bit too far for me - for instance I see no need to save changing
> layer visibilities in the undo buffer.
> 
> I also had a look at the patch - please refactor ORIGIN_VIEWITEM as a
> BOARD_ITEM so that the undo buffer needs not know the internals of the
> origin object.
> 
> Cheers,
> Tom



References