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

Export offscreenCanvasUtils #2231

Merged
merged 1 commit into from
Aug 19, 2021
Merged

Export offscreenCanvasUtils #2231

merged 1 commit into from
Aug 19, 2021

Conversation

cmdcolin
Copy link
Collaborator

@cmdcolin cmdcolin commented Aug 19, 2021

@github-actions github-actions bot added the needs label triage Needs a label to show in changelog (breaking, enhancement, bug, documentation, or internal) label Aug 19, 2021
@cmdcolin cmdcolin added internal and removed needs label triage Needs a label to show in changelog (breaking, enhancement, bug, documentation, or internal) labels Aug 19, 2021
@cmdcolin cmdcolin marked this pull request as draft August 19, 2021 15:20
@cmdcolin cmdcolin marked this pull request as ready for review August 19, 2021 15:31
@cmdcolin
Copy link
Collaborator Author

Tested this and it should work. Code at https://github.com/cmdcolin/jbrowse-plugin-arc-renderer demonstrates usage of renderToAbstractCanvas which would help the bezier curves get used in svg export (may need to add beziers to the translation layer still, but in general, useful to have this)

@cmdcolin
Copy link
Collaborator Author

This does appear to work if I use the link: approach in the jbrowse-plugin-arc-renderer package.json

It would be nice if we could use yarn link btw, it is IMO the most intuitive method for in-progress yarn development

@cmdcolin cmdcolin merged commit 31f2523 into main Aug 19, 2021
@cmdcolin cmdcolin deleted the export_offscreen_canvas_utils branch August 19, 2021 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants