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

Paste Action fails to paste but copies to clipboard #626

Closed
timvisher opened this issue Dec 20, 2011 · 20 comments
Closed

Paste Action fails to paste but copies to clipboard #626

timvisher opened this issue Dec 20, 2011 · 20 comments

Comments

@timvisher
Copy link

@timvisher timvisher commented Dec 20, 2011

I'm running ß63 (3912) on 10.6.8.

When I try to use the Paste action on anything, I here a system beep and then I have to use ⌘-v or it's equivalent to actually paste the clipboard in.

To be clear, it does copy whatever I was trying to paste in the way I was trying to paste it into the clipboard, it just fails to actually paste it in.

Nothing gets printed to Console.app as far as I can tell.

@skurfer
Copy link
Member

@skurfer skurfer commented Dec 20, 2011

Hmmm. Do you have a copy of the Core Support plug-in in ~/Library/Application Support/Quicksilver/PlugIns?

@timvisher
Copy link
Author

@timvisher timvisher commented Dec 20, 2011

Not that I know of. Here's a listing of that directory

@skurfer
Copy link
Member

@skurfer skurfer commented Dec 20, 2011

Don’t see it there. I do see two versions of BezelHUD. It must not have gotten cleaned up on the art update, so you might want to trash the old one.

I don’t know what else could cause that. Does this happen with any application and with anything in the first pane?

@timvisher
Copy link
Author

@timvisher timvisher commented Dec 20, 2011

Yep. Any application and anything in the first pane, AFAICT.

I removed the old BezelHUD plugin and it doesn't seem to affect anything.

I have not tried deleting all plug-ins. Think that's a logical next step?

@timvisher
Copy link
Author

@timvisher timvisher commented Dec 21, 2011

Just did a complete clean (renamed ~/Library/Application Support…, ~/Library/Caches/…, ~/Library/Preferences as outlined in the FAQ and the Paste and Paste as Plain Text actions still do not work and get a system beep. I also still don't see any Core Support plug-in in my Plugins directory.

@skurfer
Copy link
Member

@skurfer skurfer commented Dec 21, 2011

Still no idea. I’ve looked at the code and there is a scenario where it will beep instead of pasting, but only if the “add to clipboard” step returns NO, and you say that’s succeeding. Besides, the method that adds to the clipboard unconditionally returns YES, so I don’t see how the beep could ever be triggered.

How good are you with Xcode’s debugger? ;-)

@pjrobertson worked on this recently. Maybe he’ll have some ideas when he gets back in Jan.

@timvisher
Copy link
Author

@timvisher timvisher commented Dec 22, 2011

On Wed, Dec 21, 2011 at 10:55 AM, Rob McBroom
reply@reply.github.com
wrote:

Still no idea. I’ve looked at the code and there is a scenario where it will beep instead of pasting, but only if the “add to clipboard” step returns NO, and you say that’s succeeding. Besides, the method that adds to the clipboard unconditionally returns YES, so I don’t see how the beep could ever be triggered.

Ooo, an unconditional success path? I always like those.

How good are you with Xcode’s debugger? ;-)

0 experience, unfortunately.

@pjrobertson worked on this recently. Maybe he’ll have some ideas when he gets back in Jan.

Perhaps we can get it sorted out then.

In Christ,

Timmy V.

http://blog.twonegatives.com/
http://five.sentenc.es/ -- Spend less time on mail

@pjrobertson
Copy link
Member

@pjrobertson pjrobertson commented Dec 31, 2011

Hi Tim,

Following what Rob has said about the only time the method returns an system 'beep' is when the 'add to clipboard step fails' - which succeeds in your case - the only idea I have is that the ⌘V step is failing.

Do you use a different shortcut for pasting on your system? Unfortunately Quicksilver is hard coded to simulate the key presses of ⌘V (there is no other way), and so if you have the shortcut then it will not work for you.

If it's not the case, then I'll look into it further, but when you said:

I have to use ⌘-v or it's equivalent

It sounds like you're using an equivalent…

P.S. As Rob has said, I'm not really available until mid Jan, this is a lucky internet connection I have found!
On 22 Dec 2011, at 15:46, Tim Visher wrote:

On Wed, Dec 21, 2011 at 10:55 AM, Rob McBroom
reply@reply.github.com
wrote:

Still no idea. I’ve looked at the code and there is a scenario where it will beep instead of pasting, but only if the “add to clipboard” step returns NO, and you say that’s succeeding. Besides, the method that adds to the clipboard unconditionally returns YES, so I don’t see how the beep could ever be triggered.

Ooo, an unconditional success path? I always like those.

How good are you with Xcode’s debugger? ;-)

0 experience, unfortunately.

@pjrobertson worked on this recently. Maybe he’ll have some ideas when he gets back in Jan.

Perhaps we can get it sorted out then.

In Christ,

Timmy V.

http://blog.twonegatives.com/
http://five.sentenc.es/ -- Spend less time on mail


Reply to this email directly or view it on GitHub:
#626 (comment)

@timvisher
Copy link
Author

@timvisher timvisher commented Jan 1, 2012

Thanks for the response, Patrick!

On Sat, Dec 31, 2011 at 11:27 AM, Patrick Robertson
reply@reply.github.com
wrote:

Following what Rob has said about the only time the method returns an system 'beep' is when the 'add to clipboard step fails' - which succeeds in your case - the only idea I have is that the ⌘V step is failing.

Seems logical.

Do you use a different shortcut for pasting on your system? Unfortunately Quicksilver is hard coded to simulate the key presses of ⌘V (there is no other way), and so if you have the shortcut then it will not work for you.

If it's not the case, then I'll look into it further

In almost all applications I don't touch the hallowed ⌘ v sequence.
:) The reason I mentioned, "or it's equivalent" was because I also use
Emacs and ordinarily pasting means pressing ^ y in that app,
although of course you can rebind it.

As further proof, TextExpander.app works fine and it follows the same
strategy for text entry of hard-coding the ⌘ v sequence.

Could it be possible that something about my set up simply doesn't let
focus get back to the intended window in time to let the paste happen?
I don't know what would have changed recently since pasting was
definitely working till a recent updated, but it's a possibility.
Perhaps some other app that doesn't support pasting grabs focus for
that split second and then focus goes back to the intended app an
instant later.

Look forward to figuring this out!

In Christ,

Timmy V.

http://blog.twonegatives.com/
http://five.sentenc.es/ -- Spend less time on mail

@pjrobertson
Copy link
Member

@pjrobertson pjrobertson commented Mar 26, 2012

Hi @timvisher

Could you please test this issue with the latest version of Quicksilver (ß66)

If paste still does not work, and you're sure that "Paste" has the ⌘V shortcut next to it in the Edit menu in the menu bar, then can you please check through previous versions of Quicksilver (ß60 and ß54) to see if the issue persists.

You can download these older versions from https://github.com/quicksilver/Quicksilver/downloads

NOTE be sure to back up your Quicksilver settings before downgrading. See http://qsapp.com/wiki/FAQ#How_to_Quickly_Reset_Quicksilver_Settings for which files you should back up (duplicate them to your desktop)

@timvisher
Copy link
Author

@timvisher timvisher commented May 29, 2012

Very sorry to have taken so long to get back around to this. I can't even get previous versions of quicksilver to work, let alone to paste. They'll start but there are no actions, etc.

I cleared out the application support directories and caches before trying to use them.

Problem persists even with a totally green install of B68.

@skurfer
Copy link
Member

@skurfer skurfer commented May 31, 2012

Sounds to me like a bad or missing Core Support plug-in. Maybe you’ve got some stuff in /Library instead of ~/Library? I would also try creating a new user to see you get the same results.

@timvisher
Copy link
Author

@timvisher timvisher commented May 31, 2012

A vanilla install of of B54 does indeed work with Paste. B60 exhibits the exact same bahaviour (i.e. paste fails with a beep, but clipboard contents do get updated). I have at this point AppZapper'd each install of Quicksilver and tried them all with no plugins installed but the most basic.

Even stranger is that up through B63, as far as I know, things worked fine. But now that I try back versioning, they don't appear to work.

What can I provide next?

@skurfer
Copy link
Member

@skurfer skurfer commented Jun 1, 2012

What does this show?

mdfind "kMDItemKind == 'Quicksilver Plug-in'"

@timvisher
Copy link
Author

@timvisher timvisher commented Jun 3, 2012

On Fri, Jun 1, 2012 at 2:10 PM, Rob McBroom
reply@reply.github.com
wrote:

What does this show?

   mdfind "kMDItemKind == 'Quicksilver Plug-in'"

https://gist.github.com/2863344

@skurfer
Copy link
Member

@skurfer skurfer commented Jun 4, 2012

I don’t see any stray plug-ins, but I notice you have the Clipboard plug-in, and not the Shelf. It’s an infamous problem that these don’t work without each other. I forget the exact problems it causes (and I don’t expect it to fix your issue), but installing it is worth a try.

@timvisher
Copy link
Author

@timvisher timvisher commented Jun 8, 2012

Got it!

To reproduce this issue:

  1. Switch your keyboard layout to Dvorak!
  2. Open text edit.
  3. Make the document plain text
  4. Open Quicksilver and try to paste something in.

Somehow, the Paste action is sending the keycode to the system corresponding with the position of the V key in Qwerty, and the system is interpreting it as the K key, because the system layout is set to Dvorak.

It has to be possible to get around this, because other systems that hardwire ⌘-v as their paste command work within Dvorak, and Quicksilver used to work as well. Although I suppose an OS update could have made however Quicksilver is doing it invalid at this point. But like I said, TextExpander still works and it follows the same strategy.

The reason I was getting a system beep is because in a plain text document in TextExpander, ⌘-k is not a valid shortcut. I figured it out by running it in a rich text document and ⌘-k there runs the insert link command.

Boom! So what can we do?

@pjrobertson
Copy link
Member

@pjrobertson pjrobertson commented Jun 8, 2012

What we can do is fix it :)

Great work on hunting this down. I have just worked on this and implemented a fix - all thanks to you and your investigative skills!

Wait for the next version :D

@pjrobertson
Copy link
Member

@pjrobertson pjrobertson commented Jun 8, 2012

Fixed by 096b4a1

@timvisher
Copy link
Author

@timvisher timvisher commented Jun 10, 2012

Fantastic! Thanks so much pjrobertson!

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

3 participants