setting the frame of a Layer makes Sketch crash about sketchapi

[nichenqin]

@everdrone try this:

copy.frame().setX_(300);
copy.frame().setY_(300);
copy.frame().setHeight_(300);
copy.frame().setWidth_(300);

[mathieudutour]

be careful, new sketch.Rectangle() doesn't do what you expect. What you want is

var Rectangle = sketch.Rectangle
var rect = new Rectangle(0, 0, 100, 200)

[everdrone]

still crashing

here is the code:

var sketch = context.api();

var document = sketch.selectedDocument;
var selection = document.selectedLayers;
var page = document.selectedPage;

if (selection.isEmpty) {
  sketch.message('empty');
} else {
  var original = selection.nativeLayers[0];
  var copy = original.duplicate();
  
  var Rectangle = sketch.Rectangle;
  var rect = new Rectangle(0, 0, 100, 200);
  copy.frame = rect;
}

ran from sketch-dev-tools' playground

[mathieudutour]

you are mixing wrapped objects and native objects. copy is a native sketch layer so its frame expects a CGRect: copy.frame = rect.asCGRect();

This selection is a bit messy since there is no way to access a wrapped object from it :/ Hopefully it will be easier with the new API: https://github.com/BohemianCoding/SketchAPI/blob/feature/15747/docs/Selection.md

[everdrone]

still crashing.

All i want to achieve is duplicate the selected layer and translate/resize it.

is there an easy way to do it with the current API?

[everdrone]

it works! thanks @nichenqin