-
-
Notifications
You must be signed in to change notification settings - Fork 348
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
feat(edgeless): component toolbar #1703
feat(edgeless): component toolbar #1703
Conversation
Open in CodeSandbox Web Editor | VS Code | VS Code Insiders |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
ac9af0d
to
3a907ee
Compare
aed279b
to
499214a
Compare
@@ -0,0 +1,11 @@ | |||
export function simplePick<T extends object, K extends keyof T>( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Admittedly couldn't find a better place for this
this.surface.removeElement(element.id); | ||
} | ||
}); | ||
this.page.captureSync(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we should do captureSync()
twice. Having it before operations should be adequate.
CI only fails at uploading Codecov, good to merge. |
fix #1701
fix #1733
2023-03-16.19.26.02.mov