Possible bug with listenTo and "change:position"

[pfalcon64]

Introduction

I've got some working code where some shapes need to respond to changes to their environment (browser size, and the location of other elements).

// create a shape, it is "this". (code not shown)
// I want to monitor the position of this shape, so i can reposition _ChildCell to match.
// (note: using layers mean that embed isn't effective, but thats not the issue here)

const groupView1 = paper.getLayerView(_ChildCell.Layer);
        groupView1.listenTo(this, "change:position", () => {
            _ChildCell.translate(this.GetInnerPosition().x - _ChildCell.position().x, this.GetInnerPosition().y - _ChildCell.position().y);
            // Now start calling the content chain:
            _ChildCell.ChangePosition(graph, paper, _ChildCell);
        });

later on, when i do this:

_Panel.position(_NewLocation.position.x, _NewLocation.position.y);

the above listenTo event code is executed, as expected.

BUT, if I modify the position using this:

_Panel.position(_NewPositionObject);

then the listenTo event is NOT executed?

I've patched my code to use the process that works for now, but thought I'd mention this. I'm using the current release (4.1.1)

Steps to reproduce

No response

Restrictions & Constraints

No response

Does your question relate to JointJS or JointJS+. Select both if applicable.

JointJS, JointJS+

[MartinKanera]

Hello, thank you for the question. When calling Element.position({}) with a single argument, it acts as a getter with the single argument being options:

class Element<A extends ObjectHash = Element.Attributes, S extends mvc.ModelSetOptions = dia.ModelSetOptions> extends Cell<A, S> {

   // ...

    position(opt?: Element.PositionOptions): g.Point;
    position(x: number, y: number, opt?: Element.PositionOptions): this;

   // ...
}

No position is updated so no change:position event gets fired.

[pfalcon64]

Ahh, thanks, I expecting type checking to pick up a bad param, but that won't work where the Options values are concerned...