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

Option to automatically copy to clipboard? #48

Closed
orschiro opened this issue Oct 9, 2017 · 11 comments
Closed

Option to automatically copy to clipboard? #48

orschiro opened this issue Oct 9, 2017 · 11 comments

Comments

@orschiro
Copy link
Contributor

orschiro commented Oct 9, 2017

Dear @OttoAllmendinger,

Would you consider an option in the settings to always automatically copy the image to clipboard?

@OttoAllmendinger
Copy link
Owner

It already exists, the setting is called "Default Clipboard Content"

@orschiro
Copy link
Contributor Author

Got you!

Thanks. :-)

Wasn't obvious to me. Maybe call it "Automatically copy to clipboard"?

@OttoAllmendinger
Copy link
Owner

The problem there is that the auto-copy url function for imgur will override this (as it currently does). The dropdown should contain the options "default", "image data" and "imgur URL". It is actually a bug that it is not like that at the moment.

But I agree the naming is suboptimal ;)

@orschiro
Copy link
Contributor Author

Haha, I see.

Maybe say:

Default Clipboard Content: Image data

And then below a sentence that says:

"Select Image Data to automatically copy screenshots to clipboard"

@orschiro
Copy link
Contributor Author

Hence no changes to the text for now?

@OttoAllmendinger
Copy link
Owner

Not for now, and the confusing text is a separate issue.

Your suggestion goes in the right direction but is a bit to verbose.

Maybe it should say not "Default Clipboard Content" but instead

Auto-copy to clipboard: "Nothing"/"Image Data"/"Imgur Link"

@orschiro
Copy link
Contributor Author

Auto-copy to clipboard: "Nothing"/"Image Data"/"Imgur Link"

Nice, like this one!

Can we implement that?

@OttoAllmendinger
Copy link
Owner

Yes, #50

Downside is that translations need to updated

@orschiro
Copy link
Contributor Author

@OttoAllmendinger

Downside is that translations need to updated

Can help with that!

It's this folder, correct?

https://github.com/OttoAllmendinger/gnome-shell-screenshot/tree/master/src/locale

@OttoAllmendinger
Copy link
Owner

Cool!

Yes, it is that folder but I will have to update somefiles before you can change them. I will do it later today :)

@orschiro
Copy link
Contributor Author

Nice! Let me know when I can start. :-)

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

No branches or pull requests

2 participants