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

Wrong paste behaviour after using "Paste ingredients" option #19

Closed
cydanil opened this issue Aug 20, 2021 · 4 comments · Fixed by #35
Closed

Wrong paste behaviour after using "Paste ingredients" option #19

cydanil opened this issue Aug 20, 2021 · 4 comments · Fixed by #35

Comments

@cydanil
Copy link
Contributor

cydanil commented Aug 20, 2021

This is a copy of this issue.

When creating a new recipe, after having used "Paste ingredients" option from Ingredients page, if you use "Ctrl-v" in the Instructions page, the paste is done in the ingredients page.

Steps to Reproduce

  1. Create a new recipe
  2. Copy ingredients list from a recipe somewhere on the web for example.
  3. In the Ingredients page, use the "Paste ingredients" option to paste your list of ingredients.
  4. Now copy instructions from the recipe somewhere on the web.
  5. Move to the Ingredients page in Gourmet, and use Ctrl-v to paste your instructions : the content is pasted into the ingredients page.

NOTE 1 : using right click, Paste option is ok, the content is correctly pasted in the instructions page.
NOTE 2 : if you don't use first the "Paste ingredients" option, then Ctrl-v wortks as expected in the instructions pages.

Expected Behavior

Ctrl-v should paste at the current page.

Environment

  • Operating System: Ubuntu 20.04
  • Version or commit id: 1.0.0 or 1.0.1 (same)
  • Installed as Flatpak or Python package?: Flatpak
@cydanil
Copy link
Contributor Author

cydanil commented Aug 20, 2021

This is partially solved: all tabs can independently be pasted to, but text is no longer automatically parsed on a paste in the ingredient tab.

@pled
Copy link

pled commented Sep 3, 2021

Hum... that feature (parsing ingredients list) is very useful when copying a recipe from the net.
I prefer to use the "Right-clik-Paste" for instructions option workaround than loosing the parsing feature.
Just my opinion though.

@cydanil
Copy link
Contributor Author

cydanil commented Sep 3, 2021

Absolutely! I will address this issue for the next release.

@cydanil
Copy link
Contributor Author

cydanil commented Sep 8, 2021

I've fixed it as shown below:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants