Skip to content

Conversation

@romansp
Copy link

@romansp romansp commented Jan 31, 2018

So basically this fixes #18. If you can review this, this will be great.

ElementTreeItemWrapper was required, because ElementTreeItem is rendered recursively.

This is 100% going to be rebased I think, as PR also includes change to webpack.skpm.config.js that was required to get dev tools running on older mac, but most probably you want this to be reverted.

…lows running sketch-dev-tools on an older mac. this might gonna be reverted
- show `QuickLook` when `selectedElement` equals to current element id
@mathieudutour
Copy link
Member

mathieudutour commented Jan 31, 2018

haven't reviewed yet but there were some discussion about transpiling cocoascript-class, I don't think it's as simple as running babel on it: airbnb/react-sketchapp#106 (comment)

@romansp
Copy link
Author

romansp commented Jan 31, 2018

Yeah, saw that, but at least I can run it instead of erroring during start.

In my own plugin with webview I had to use a safer version https://github.com/atlassian/cocoascript-class for proper transpiling.

It's totally fine to exclude webpack.skpm.config.js change from final PR. I will rebase as soon as you are satisfied with actual PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No Scroll on Elements Panel when QuickLook is opened

2 participants