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

support duplicating components in editor #150

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

WebFreak001
Copy link

Not too fixed on the current keyboard shortcuts (just pressing D, or Ctrl-D)

I'm not sure if I should be accessing the internals of the widget like I am doing here, or if I should abstract that away into some helper (if so and there is an existing one, I don't know which existing helper to put it in) - I needed to clear the internals like this to make sure that it doesn't have the same widget reference on multiple instances as well as in the saving that it didn't save the same reference twice (breaks rendering otherwise)

For rapidly creating a form with a few custom inputs this is a good time saver for me. Undo/redo support is something I would be quite interested in as well, but the code base didn't seem to be built for that yet and I would need a lot of time getting into the code.

@CLAassistant
Copy link

CLAassistant commented Apr 25, 2024

CLA assistant check
All committers have signed the CLA.

@WebFreak001
Copy link
Author

ok with this code it looks like when duplicating nested structures, the nested things are not being duplicated properly (references stay the same, kinda corrupting the file until manually resolved)

is there any kind of interface I can assume that'd exist on the component type like somehow being able to access its children, so I can recurse through it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants