Skip to content
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

Fixed scrolling when QuickLook is opened in Elements panel #22

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

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