Skip to content

Add model reload button#19

Open
rmsc wants to merge 5 commits intoconnorslade:mainfrom
rmsc:add_model_reload_button
Open

Add model reload button#19
rmsc wants to merge 5 commits intoconnorslade:mainfrom
rmsc:add_model_reload_button

Conversation

@rmsc
Copy link
Copy Markdown
Contributor

@rmsc rmsc commented Mar 5, 2026

This is very useful when you're iterating on a model and don't want to keep going through the delete->load->reposition loop.

It was not as straightforward as I expected, so I've split it into several atomic commits to make it easier to review.

I'm not particularly happy with how supports track their parent model, but a cleaner implementation would likely need some deeper changes (maybe to the Project struct, where models are currently held in a Vec<Models>). I'm definitely open to suggestions.

@rmsc rmsc force-pushed the add_model_reload_button branch from 4e89591 to bc2f5ec Compare March 13, 2026 14:42
@rmsc rmsc force-pushed the add_model_reload_button branch from bc2f5ec to 02e644e Compare March 13, 2026 14:59
@connorslade
Copy link
Copy Markdown
Owner

This is a really good idea. Right now support generation is still in development and not usable, so I don't think its worth worrying about tracking the supports.

Also, I'm not sure how I feel about serializing the file paths. They should probably be stored relative to the project file, but you may not always want the path embedded at all. Maybe there should be a checkbox to save it, or maybe it should only be saved if the model is in the same directory as the save.

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.

2 participants