Skip to content

Apply paste delay after key press and increase slider range#1465

Merged
cjpais merged 3 commits into
cjpais:mainfrom
sboys3:main
Jul 8, 2026
Merged

Apply paste delay after key press and increase slider range#1465
cjpais merged 3 commits into
cjpais:mainfrom
sboys3:main

Conversation

@sboys3

@sboys3 sboys3 commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Before Submitting This PR

Please confirm you have done the following:

Human Written Description

The main purpose of this is for people to test if increasing the delay after improves clipboard pasting reliability. This fix applies the existing paste delay config option to after the paste key presses. Both the delay before and after are now controlled by the one setting. The clipboard seems to be reverted before the key presses go through to some applications, resulting in previous clipboard content being pasted instead of the new content. This slider on the debug page has also been extended out to 500ms, because on my system when there is heavy load, it is required to be that high. The only change this should have for normal users is it changes the second delay from 50 to 60 milliseconds by default.

Related Issues/Discussions

Discussion: #502

Community Feedback

Testing

I have used this fix for several days now, during which some of the time my system was under heavy load, and I have not had a paste failure since. Although 1000 milliseconds in total(500ms for the setting) does block the main thread for a while. Previously, it was a common occurrence to get incorrect content pasted.

AI Assistance

  • No AI was used in this PR
  • AI was used (please describe below)

@cjpais

cjpais commented Jun 3, 2026

Copy link
Copy Markdown
Owner

Is it possible to make these two separate sliders @sboys3

Where: std::thread::sleep(std::time::Duration::from_millis(50)); was not previously using any variable

@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

🧪 Test Build Ready

Build artifacts for PR #1465 are available for testing.

Download artifacts from workflow run

Artifacts expire after 30 days.

@hbenthow

Copy link
Copy Markdown

Is there an ETA of when this pull request might be implemented? I keep checking for updates to Handy and there haven't been any in a long time.

@cjpais

cjpais commented Jun 25, 2026

Copy link
Copy Markdown
Owner

I am in the process of releasing transcribe.cpp and integrating into handy in parallel. After this is done I will be reviewing issues and merging appropriate features.

@cjpais cjpais merged commit f0f7e7f into cjpais:main Jul 8, 2026
4 checks passed
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.

3 participants