react-paper-bindings icon indicating copy to clipboard operation
react-paper-bindings copied to clipboard

Group and Path should have 'position' prop

Open kota65535 opened this issue 7 years ago • 2 comments

I found some issues for moving position of Group and Path.

Problem of center and point prop on Group and Path component

both prop perfectly work for shaped paths, like Circle, Rectangle ...etc. On the other hand, Path and Group does not reflect its initial position by these props. For example, this below does not set its center position to [100, 100].

        <Path
          segments={[[20, 20], [80, 80], [140, 20]]}
          center={[100,100]}
          fillColor={'black'}
        />

This also does not work.

        <Path
          segments={[[20, 20], [80, 80], [140, 20]]}
          point={[100,100]}
          fillColor={'black'}
        />

introducing position prop

position prop works for not only shaped paths but also all Item based objects, Path, Group. Layers ...etc. But currently changing these props does not move its actual position. It would be better if this prop is available, like below.

        <Path
          segments={[[20, 20], [80, 80], [140, 20]]}
          position={new Point(100,100)}
          fillColor={'black'}
        />

kota65535 avatar Feb 24 '18 08:02 kota65535

Hey @kota65535. It's been some time, since I last worked on this library, but I think that vanilla PaperJS does not support changing center/point props on Path or Group. As you have already figured out, it works for Circle for example.

I think something like position prop would be perfect for this use case. I was trying to keep the custom props down to the minimum, but for some cases it just makes sense. You can make a PR if you wish.

HriBB avatar Feb 25 '18 20:02 HriBB

Hi @HriBB, I already have some working codes and test cases that go along with #14. I will brush up them and make PR this week.

kota65535 avatar Feb 26 '18 07:02 kota65535