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

Add continuous simulation option #20

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

Conversation

l0f3n
Copy link
Contributor

@l0f3n l0f3n commented Nov 11, 2024

This adds a continuous simulation button to the background radial menu. As soon as we press redraw or we get a new node from the extension, this mode is disabled again. Kind of a quick and dirty solution. You want to do something more sophisticated?

@sornas
Copy link
Contributor

sornas commented Nov 12, 2024

i don't know, it really eats CPU even when nothing changes. i think @BeeSneeze is the one to ask about the intent though

@BeeSneeze
Copy link
Contributor

So like, I can get the want to have it if you want to drag stuff around and have a fun woooshing effect, but idk I think it should be very much an optional thing for the right click, maybe add an "EXPERIMENTAL" or something. It's not something I feel is super neccesary, but it is fun and silly, so I'm not against adding it

@l0f3n
Copy link
Contributor Author

l0f3n commented Nov 12, 2024

@sornas I thought one of the perks of being a webdev was not caring about performance

Anyway, maybe we just call it a failed experiment. Not super necessary, so maybe we just settle on not adding this

@l0f3n
Copy link
Contributor Author

l0f3n commented Nov 12, 2024

Hmm, what if we go the other route of not making the nodes draggable at all? Or are you guys fine with how it is now? I feel like it looks kinda bad when you drag the node without any feedback

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

Successfully merging this pull request may close these issues.

3 participants