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

Feature request: removing ingredients from recipe confirmation (or undo) #469

Open
opicron opened this issue Jul 4, 2024 · 3 comments
Open
Labels
enhancement New feature or request

Comments

@opicron
Copy link

opicron commented Jul 4, 2024

What's the feature 🧐

I noticed this has been spoken about a lot. I'd like to open the discussion if I may.

When adding of editing a recipe it seems very easy to remove a ingredient by accident.

To avoid saving I notice myself tapping the < button and not saving the recipe. And then edit the recipe again to avoid re-adding the removed ingredient. Which is removed so easily that I mostly miss which ingredient (and sub quantities) I removed in the first place.

In my experience it would be very helpfull to either confirm removal of an ingredient in an recipe.

An recipe is usally fixed, there will not be a lot of need to remove ingredients.

The current user experience benefits easy removal. While I believe removal shoud be somewhat 'protected'. Considering the nature of a recipe.

Extra information and references

I have tried understading the current UX. Im having issues seeing the benefit. Sorry ;).

@opicron opicron added the enhancement New feature or request label Jul 4, 2024
@opicron
Copy link
Author

opicron commented Jul 5, 2024

To add to this point after a night sleep. In the settings we can select how ingredients are removed. Either by click or by confirmation. It would be nice to see this propagated into the recipes edit functionality.

Also I thought, maybe it is a good idea when removing (accidently) ingredients from an recipe to make them unselected as it where. So we can 'undo' the removal by reselecting them.

This last idea is not fleshed out, is it a good functionality to save unselected ingredients? For future additions to the recipe for example. It gives a lot more functionality. The only question will then be, how to actually delete the ingredient.

@TomBursch
Copy link
Owner

You're right that it's probably a bit too easy to remove ingredients. And I think the same could apply to meals in the recipe planner. From a UX perspective, it felt weird to have the same button react different in recipes and shopping lists (Although you're right, it currently differs anyway when using the confirmation behavior from the settings).

To add to this, would an undo/redo button be enough to solve this for you?

So now we have the following possible solutions:

  • Honor the shopping list interaction settings while editing recipes and use the same removal system from the shopping list
  • Add Undo/Redo
  • Ask for confirmation when deleting an item
  • Gray the ingredients out and only remove them once the recipe has been saved (I feel like communicating to a user what is happening will be difficult for this)

I'm not sure what I prefer.

@opicron
Copy link
Author

opicron commented Jul 7, 2024

Im right there with you. Not sure which flow would work best.

The consistent choice would be using the configuration setting. However, Im afraid some users might not dig deep for the setting and get demoralized by the default flow in editting ingredients.

Ive read that you would like to avoid multiple actions on the three dots (or holding the icon on phone). Actually I do see a use for a delete there, along with a change icon for example.

If no multiple actions are possible on the ingredients, I would prefer a confirm action on click of the ingredient. Regardless of settings.

Though call to make.

PS: transaction ref 9F334927PM0487602

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Planned
Development

No branches or pull requests

2 participants