New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add some getParent* methods on Layer #284

Merged
merged 1 commit into from Nov 16, 2018

Conversation

Projects
None yet
3 participants
@mathieudutour
Copy link
Contributor

mathieudutour commented Nov 15, 2018

No description provided.

@bohemian-coding-danger

This comment has been minimized.

Copy link
Collaborator

bohemian-coding-danger commented Nov 15, 2018

Warnings
⚠️

Source/dom/layers/Artboard.js changed, but not its tests. That's OK as long as you're refactoring.

⚠️

Source/dom/layers/Page.js changed, but not its tests. That's OK as long as you're refactoring.

⚠️

Source/dom/wrapNativeObject.js changed, but not its tests. That's OK as long as you're refactoring.

⚠️

Source/dom/layers/Artboard.js changed, but not its doc. That's OK as long as you're refactoring.

⚠️

Source/dom/layers/Page.js changed, but not its doc. That's OK as long as you're refactoring.

Generated by 🚫 dangerJS

layer.getParentShape()
```

In addition to the direct `parent`, you can access a few other entities in the hierarchy of the layer.

This comment has been minimized.

@bomberstudios

bomberstudios Nov 15, 2018

Collaborator

Shouldn't this line be on top of the previous code block, ending in :? 🤔

This comment has been minimized.

@mathieudutour

mathieudutour Nov 15, 2018

Contributor

no the code block is shown on the side

@mathieudutour mathieudutour merged commit cc521ab into develop Nov 16, 2018

1 check passed

ci/circleci Your tests passed on CircleCI!
Details

@mathieudutour mathieudutour deleted the f/parents branch Nov 16, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment