Idleon-Companion/IdleonCompanion

[FEAT] Multiple recipe selection for recipe calculator w/ tracking

Opened this issue · 7 comments

Which categories does your suggestion apply to?

  • UI changes
  • Tracking
  • Calculators
  • Other tools/misc

Is your feature request related to a problem? Please describe.
Let's say I want to create a whole armor set for my character, it's nice to see what I will need in my inventory to do it all at once.

Describe the solution you'd like
Allow the user to add multiple recipes to a list of recipes to make. Also allowing the user to check off what they have in their inventory (like a grocery list for crafting) I made a little drawing showing what I mean.
image

What data is needed for the suggestion?
Nothing that isn't already part of the current build.

Additional context
If my explanation is bad, feel free to DM me on discord ItsDan#6969

That's a great idea. I don't know if it would be possible with the way the data is currently formatted, but to keep in mind for the v2 of the calculator

I was curious if the data was saved as a total for each item entered or if it actually calls on the subcomponents to figure out the total.
Just with Lava saying about adjusting crafting recipes if updating the calculator will be a quick just update the affected recipies or if you will need to go thru every recipe to make sure you have all the adjustments right.
If its the later a V2 of the calc might be needed just for Future Proofing against balance changes.

I'm working on this one, as it was, the recipe data was using the totals for each item, so i changed it and opened the #161
Ween the PR gets accepted i gonna start working on this new feature.

Still refactoring to fix some quantity issues, but i need a opinion about this.
Take a look at my version of recipe calculator here

  • Multiple recipe only make sense in list view, since it only list all materials, without sub-components structure.
  • Pretty intuitive: click on + to add, click on line to remove
  • if you want to change a recipe quantity, remove it and add with the new quantity

Still fixing some issues with the materials quantities, refactoring this, but what you guys think about those functional choices?

So this is where this discussion was taking place. Sorry I'm late to the party ( Github Noob ).

I am 100% on board with being able to select multiple recipes.

I'd look into disabling the "View" dropdown selector when multiple recipes are selected. I just tested it, and although the multiple recipes weren't reset or anything... it's more of an aesthetic thing for me.

I'm on board with removing the old recipe/quantity and having to re-add it. I expect people will want to be able to edit their recipe list, but that seems fairly low priority.

I'd recommend a simple/short explanation that multiple recipes are supported, as I'm of the opinion it's better to spell it out rather than assuming people will just "get it".

Don't worry, I still getting used with collaborate on a open source project, sometimes forget to check for responses and things like that 😢

I'd look into disabling the "View" dropdown selector when multiple recipes are selected. I just tested it, and although the multiple recipes weren't reset or anything... it's more of an aesthetic thing for me.

Seem a good idea, always let the user add more recipes, but if he does, look the view dropdown, with a tooltip "Multiple recipes only suport list view" os something like that.

I'm on board with removing the old recipe/quantity and having to re-add it. I expect people will want to be able to edit their recipe list, but that seems fairly low priority.

Agreed, let's see how it goes...

I'd recommend a simple/short explanation that multiple recipes are supported, as I'm of the opinion it's better to spell it out rather than assuming people will just "get it".

Nice idea, i kinda suck on those things like describe the feature, where to add this explanation... things like that, so ween we have this feature 100% done, I can take some help with it 😅

@dougllima, I just took a look at it and tested it out.

Couple of things I thought of:

For the view being List / Tree, you can always make it "List" and not have the option to select "tree" (allows you to remove view altogether). The tree feature will still be used; however, it will show the tree for the recipe once the user clicks on the recipe they are crafting. This allows for multiple recipes to be added and not worry about switching views and having the recipes disappear.

Additionally, you could allow users to click the individual resource items required to craft the items in the tree; clicking the items can change the item's color to green or something to show it has been acquired in the user's inventory.

Once the user has all the items in the tree, the tree can minimize automatically; The user can then right-click the recipe to remove it from the list of items to craft once they have created it.

Edit: Updated for grammar and clarity