stochaztic / lttp-tracker

Item tracker for link to the past randomizer
MIT License
10 stars 8 forks source link

Added combined items for switchables items. #20

Closed emerald000 closed 6 years ago

emerald000 commented 6 years ago

Bow/silvers, shovel/flute and mushroom/powder can be tracked in a single item (or two as before).

Changed the layout of the edit mode items to make more sense.

Might want to put those combined items into the default layout?

stochaztic commented 6 years ago

Rather than doing it this way, I was thinking about having combined items done by having slightly smarter logic in the item itself, so the data layer wouldn't change at all, and, for instance, the combined shovel/flute item would toggle the both data items and be set correctly accordingly. (So you could have all of shovel, flute, and combined shovel/flute on screen, and changing one would change the other. Don't know why you would do this, but you could.)

Was planning on saving this for when we were more Vue-enabled, but I suppose it could be implemented now.

Let me know what you think, and if you'd want to re-work this into that idea or if you just want to close this.

emerald000 commented 6 years ago

While I agree it would make things nicer, I'm not sure how to go to do this though. I'll close this and maybe revisit later when the backend has been updated.