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

[Vis Builder] Restructure so that implementing undo and redo is possible #3102

Open
abbyhu2000 opened this issue Dec 16, 2022 · 2 comments
Open
Labels
vis builder visualizations Issues and PRs related to visualizations

Comments

@abbyhu2000
Copy link
Member

No description provided.

@abbyhu2000 abbyhu2000 changed the title Restructure so that implementing undo and redo is possible too (Can be spun-off into a separate issue too) [Vis Builder] Restructure so that implementing undo and redo is possible Dec 16, 2022
@joshuarrrr
Copy link
Member

Is this just to unblock redo/undo, or to actually implement those? I thought I remembered that @ashwin-pc had an issue for undo/redo in Vis Builder, but I can't find it, so I must have made that up.

@joshuarrrr joshuarrrr added vis builder visualizations Issues and PRs related to visualizations labels Dec 20, 2022
@ashwin-pc
Copy link
Member

Nope, I didnt actually create that issue, although i agree, this issue needs more context. I dont think undo redo needs a separate restructure task by itself. the whole feature is simple enough that it can be implemented at once once we know the UX for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vis builder visualizations Issues and PRs related to visualizations
Projects
None yet
Development

No branches or pull requests

3 participants