-
Notifications
You must be signed in to change notification settings - Fork 86
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
Full screen action #132
Comments
I tried to develop this functionality for a project but I am not sure I chose the right solution. I first tried with
Now the figure re-sizes correctly when I double-click but I am not sure at all the solution I propose is correct. Any feedback will be greatly appreciated ! |
My idea was to add another action to the Vega-Embed actions, which is independent of Vega signals. See Line 323 in 1fcb438
|
Closing since noone has asked for this yet. |
Doh; Apparently I missed the boat on this! I think this is a great idea, and would definitely use it! Please consider reopening. Thanks! |
Wanna send a pull request? I'm happy to help with reviews. |
Maybe; LOL. It's not high priority for me right now, but I might consider working on it at some point. How do you feel about reopening so that folks interested in contributing can find it more easily? Thanks as always |
It would be great to have an action to open a chart in full screen.
The text was updated successfully, but these errors were encountered: