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

[Editor] More granular updating of Extra Settings fields #510

Open
Emral opened this issue Aug 17, 2022 · 2 comments
Open

[Editor] More granular updating of Extra Settings fields #510

Emral opened this issue Aug 17, 2022 · 2 comments

Comments

@Emral
Copy link
Contributor

Emral commented Aug 17, 2022

Is your feature request related to a problem? Please describe.
When working with a lot of items that have extra settings, you can quickly have many NPCs with slightly different extra settings configurations in a level. If you want to bulk-edit one extra settings option, it will overwrite ALL extra settings options on all these NPCs with the one that happened to appear in the properties box.

Describe the solution you'd like
Only the value that is changed should be updated in the extra settings box for all selected objects.

Describe alternatives you've considered
Alternatives like removing multi-edit on extra settings and adding a context menu item to copy individual extra settings are simply worse UX and should not be considered further than the passing thought I gave them.

It might make sense to have a toggle between the current behaviour and the new one, like when multi-editing objects with extra settings, a button that says "apply to all" or something.
In general it might be nice to be able to select the object in a multi-edit that you are viewing the values of, but that leans a bit hard into redesigning the entire window so I won't go down that rabbit hole here.

Additional context
A demonstration of the current behaviour. Changing the "Always fire" field here should set it for the selected cannons, but should not set the Delay field on the left cannon.
tmp

@Wohlstand
Copy link
Member

In general, when we select objects with different values, these fields should be semi-checked/semi-greyed to mean many objects contains this field different value between each other, and then, user is able to set the same value for all objects of the same type. I think to make the more advanced and flexible properties system during the reconstruction, and that also should allow to see properties of objects even if they different types (blocks, BGOs, NPCs, etc.)

@Wohlstand
Copy link
Member

Anyway, for the planned major reconstruction the new system of settings system is planned, so, it should simplify a lot of cases. Current settings system implementation sucks for many reasons. So, will just keep this issue as a note for the future major update.

@Wohlstand Wohlstand added this to the Moondust Project Neo milestone Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants