bananarama92 / MBS

Various additions and utility scripts for BC
Other
7 stars 7 forks source link

ENH: Crafted wheels with separate customizations #25

Closed anchillabc closed 8 months ago

anchillabc commented 1 year ago

I don't know if this is technically possible, but it would be very nice to be able to switch between different wheels with completely different content.

This might work by assigning a name to each wheel customization page which must be found in the name (or description) of the crafted wheel which is equipped. This would allow us to switch between wheels with longer or shorter timers or completely different content. This suggestion makes more sense in conjunction with my other suggestion of additional custom option pages for the wheel.

bananarama92 commented 1 year ago

Hm... so (with data storage in mind) I'm thinking of the following approach here:

having completely different wheel item sets for each of these proposed is probably off the table as the amount of data storage would balloon out of control to an extent that I'm not quite comfortable with.

On the other hand, what would be an option is to effectively have these presets only define which of the item set/option combination are currently active, each thus representing a subset of the total number of available item sets. This would have as a consequence though that it an item set were to be changed or removed, then it would affect all presets that it is a member of. Would this be an acceptable compromise?

anchillabc commented 1 year ago

That is not a compromise, this is exactly what I had in mind :) When writing the proposal, this is why I looked up the names of the pages specifically - no need to duplicate items when you use crafted wheel name / description to reference customization (checkboxes) page name field only.

bananarama92 commented 8 months ago

Implemented as of https://github.com/bananarama92/MBS/pull/79