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

Add support for pasting from primary clipboard buffer on linux #132

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lukasfink1
Copy link

On Linux, there exists another clipboard buffer (called primary). This buffer simply contains the text that was selected last. Usually the text from this buffer can be pasted by pressing the middle button on the mouse. I often use this feature, as it’s a bit faster than using Ctrl+C and Ctrl+V and it can be utilized completely without using the keyboard.

This pull requests adds support for pasting from this buffer to SmartCopyPaste.

@Eisa01
Copy link
Owner

Eisa01 commented Jan 19, 2024

Thanks, though I believe this is an optional change and perhaps users are already used to the current default config for pasting in Linux.
Perhaps I'll revisit this later after testing it to see if the behavior is good for default option.

Closed for now as this is only default config change.

@Eisa01 Eisa01 closed this Jan 19, 2024
@lukasfink1
Copy link
Author

If I understood your comment correctly, you consider this a change to the default behaviour, such that with this change it uses the primary buffer instead of the clipboard buffer.

I probably should have clarified this in my description better, but this is in fact not what this pull repuest does. This pull request adds an additional keybinding for pasting from the primary buffer (defaulting to MBTN_MID) alongside the existing ones. Ctrl+C, Ctrl+V and the other previously existing ones should work exactly the same as they did before.

So a user not knowing about this feature or prefering to exclusively use the clipboard buffer shouldn’t notice any difference, except if they want to bind MBTN_MID to something else, in which case they can simply remove it from the paste_primary_keybind option.

@Eisa01 Eisa01 reopened this Jan 19, 2024
@Eisa01
Copy link
Owner

Eisa01 commented Jan 19, 2024

Thanks for the context and PR. Makes sense, I'll review 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 this pull request may close these issues.

2 participants