Skip to content

Enter key doesn't copy content to clipboard #362

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

Closed
Streppel opened this issue Oct 4, 2018 · 5 comments · Fixed by #363
Closed

Enter key doesn't copy content to clipboard #362

Streppel opened this issue Oct 4, 2018 · 5 comments · Fixed by #363

Comments

@Streppel
Copy link
Contributor

Streppel commented Oct 4, 2018

Flameshot version
0.6.0

Describe the bug
The return key is commonly called as enter key in many times during our day to day usage, as opposed to return key, at least from my experience. So, when we use a big keyboard with a dedicated numeric keyboard on the side, we usually expect the return key to have the same behavior as the enter key.

On flameshot, return key saves the selection on GUI mode to the clipboard, but the enter key does not replicate this behavior. I stumbled upon this a couple of times when I reached the numeric enter key, as it is closer to the mouse.

To Reproduce

  • Start GUI mode
  • Make a selection
  • Press "Enter" on numeric keyboard
  • Nothing Happens

Expected behavior

  • Start GUI mode
  • Make a selection
  • Press "Enter" on numeric keyboard
  • Content should be copied to clipboard

System Information
Ubuntu 16.04

This is nothing critical, so please feel free to reply in case you don't agree

btw much love from brazil <3 flameshot is great

@Streppel Streppel mentioned this issue Oct 4, 2018
lupoDharkael pushed a commit that referenced this issue Oct 4, 2018
This fixes #362, making the enter key copy the content of the GUI selection copy the content to the clipboard
@alexamy
Copy link

alexamy commented Feb 26, 2019

Hi! I encounter similar misleading behaviour too. I'm using Ubuntu 18.04.2 and just install flameshot.

I type flameshot gui -p ~/Pictures in terminal, select some region on the screen, and I expect that if I press 'enter' (or 'return') key on my keyboard, the region will be saved in ~/Pictures. But I have to manually click the save button to do so. It's very inconvenient.

@alexamy
Copy link

alexamy commented Feb 26, 2019

UPD: I've updated flameshot version to 0.6.0. Now 'enter' key copies selection to clipboard. I think it is convinient when I'm using flameshot -c, but why do I need that when I'm using just flameshot -p?

@Streppel
Copy link
Contributor Author

Streppel commented Feb 26, 2019

@alexamy both enter and return keys behave alike in this situation? This issue was intended to make the enter key behave exactly as the return key does

@alexamy
Copy link

alexamy commented Feb 27, 2019

@Streppel Both keys are copying selection to clipboard. But I dont fully understand your intention here. If I'm using flameshot gui -p ~/Pictures command, first of all I expect flameshot to save screenshot to ~/Pictures. I can achieve this by clicking 'Save' button in gui selection mode.
But I also expect a shorter way to do this by just pressing 'Enter' (or 'Return') key. At this moment If I press one of these keys, selection is just copied to clipboard, but isn't saved at ~/Pictures. I find this as a misleading behavior.

@Streppel
Copy link
Contributor Author

@alexamy I agree with you, this is misleading behavior. However, it's not related to this issue (362), so I believe the best thing to move on with this is to open a different issue and keep track on it.

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