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

[BUG] - No option to remove an image from a recipe #4595

Open
6 tasks done
jo-gross opened this issue Nov 22, 2024 · 2 comments · May be fixed by #4708
Open
6 tasks done

[BUG] - No option to remove an image from a recipe #4595

jo-gross opened this issue Nov 22, 2024 · 2 comments · May be fixed by #4708
Labels
bug Something isn't working triage

Comments

@jo-gross
Copy link

First Check

  • This is not a feature request.
  • I added a very descriptive title to this issue (title field is above this).
  • I used the GitHub search to find a similar issue and didn't find it.
  • I searched the Mealie documentation, with the integrated search.
  • I already read the docs and didn't find an answer.
  • This issue can be replicated on the demo site (https://demo.mealie.io/).

What is the issue you are experiencing?

I want to remove the image from a recipe

Steps to Reproduce

  1. create/edit a recipe
  2. add an image
  3. save
  4. edit the recipe
    -> no option to remove the image, expectation is to find a 'remove image' button at the image modal

Please provide relevant logs

Mealie Version

v2.2.0

Deployment

Docker (Linux)

Additional Deployment Details

No response

@jo-gross jo-gross added bug Something isn't working triage labels Nov 22, 2024
@Tomnowell Tomnowell linked a pull request Dec 11, 2024 that will close this issue
@Tomnowell
Copy link

I've made a draft pull request to add this. feat: Add delete button to recipe image modal #4708

@jo-gross
Copy link
Author

Thanks a lot, looks really nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants