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

disable the paste action for multiple items. Fixes #593 #1167

Merged
merged 1 commit into from Oct 12, 2012

Conversation

pjrobertson
Copy link
Member

@pjrobertson pjrobertson commented Oct 11, 2012

The best solution would be to implement the new NSPasteboard methods introduced in 10.6, as you can pass an array of items to the pasteboard, and it will paste them all. This is a lot harder than it looks

This is an interim fix to avoid the crash.

The best solution would be to implement the new NSPasteboard methods introduced in 10.6, as you can pass an array of items to the pasteboard, and it will paste them all. This is an interim fix to avoid the crash.
skurfer added a commit that referenced this issue Oct 12, 2012
disable the paste action for multiple items. Fixes #593
@skurfer skurfer merged commit 0ed19f0 into quicksilver:master Oct 12, 2012
@skurfer
Copy link
Member

@skurfer skurfer commented Oct 12, 2012

Merged, as long as you promise to implement “the new NSPasteboard methods introduced in 10.6” one day. ;-)

@pjrobertson
Copy link
Member Author

@pjrobertson pjrobertson commented Oct 13, 2012

Merged, as long as you promise to implement “the new NSPasteboard
methods introduced in 10.6” one day. ;-)

Oh god... please no! :P
It's not deprecated, so we should be OK for a while longer... :)

On 12 October 2012 16:26, Rob McBroom notifications@github.com wrote:

Merged, as long as you promise to implement “the new NSPasteboard
methods introduced in 10.6” one day. ;-)


Reply to this email directly or view it on GitHubhttps://github.com//pull/1167#issuecomment-9380039.

@skurfer
Copy link
Member

@skurfer skurfer commented Oct 13, 2012

Heh. Not because we have to. Just because it sounds cool to be able to paste multiple things.

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.

None yet

2 participants