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

Bug Report/Improvement request: Have the program only say "Copied" after the data is actually in the clipboard #178

Open
CorvusUrro opened this issue Sep 15, 2024 · 3 comments

Comments

@CorvusUrro
Copy link

I don't know if this is some issue with an unanticipated delay with moving data to the actual clipboard, or a design decision I personally don't vibe with,

But when copying data from the program there is a small delay, which in itself is fine, but the feedback feels slightly misleading to me.

When initiating a copy the clip highlights green and says "Copied" however the data doesn't appear in the actual clipboard until after the feedback has already disappeared.

I feel that the program should not say "Copied" until the data is actually ready to be used and the instantaneous feedback should perhaps be just the green highlight initially with "Copied" appearing once the process is complete. (Perhaps having loading spinner/"throbber" in place of the word if desired)

@kurdin
Copy link
Contributor

kurdin commented Sep 15, 2024

@CorvusUrro Not sure where you can see a problem. When you copy something into clipboard you see Copied, the app detects it in the clipboard and add it to the clipboard history. I don't see the feedback or the workflow issues. When the app detects copied data, it will process, it could take sometimes, depend on your system and other things, to display the new copied item. The delay between Copied feedback and displaying the new item in the history item is pretty normal.

@CorvusUrro
Copy link
Author

This is the best I can do for demonstrating the problem.
I double click to copy then paste it twice, and you'll notice that the first time I paste it's the previous data I had in my clipboard.
notepad++_FMN5nhvV8i
I fully anticipate this could be some kind of quirk with my particular system.

@kurdin
Copy link
Contributor

kurdin commented Sep 17, 2024

Thank you for the video demonstrating the issue. Your proficiency with the keyboard and mouse is evident in how quickly you navigate the screen.
I've confirmed and reproduced the minimal delay you're experiencing with copied items. However, I don't consider this a significant problem. Let me explain why:

Copying process: When you copy a saved clip to the clipboard, our app must:

  1. Locate the clip in the database
  2. Retrieve it
  3. Check various parameters (item type, settings, etc.)
  4. Copy the clip's content to the system clipboard

This process naturally takes a small amount of time.
UI feedback: I prioritize immediate user feedback. The UI updates instantly when you initiate an action, without waiting for the clipboard operation to complete.
Technical limitations: Given the necessary steps involved, I don't believe I can significantly speed up the process of copying a clip's value to the clipboard.

While I always strive for the best performance, this slight delay is a result of ensuring accurate clipboard operations. I believe the current implementation strikes a good balance between speed and functionality.

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