-
Notifications
You must be signed in to change notification settings - Fork 96
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 notifcation/splash screen to show whats happenig after screenshot selection #27
Comments
Hi @dynobo When no text is present to be copied I think it really looks nice informing that content is copied to their clipboard.
Let me know your thoughts on this. If the text seems fine to you I can send you a merge request. |
Thanks for your proposal @ashishpatel1992. I like your idea to use the systems notification system for displaying this information! I'll do some research, if I can find a cross-platform python package that can provide this functionality. :-) |
Thanks, @ashishpatel1992 ! One question: I'm hesitating to publish 0.1.6 as release, because I discovered an issue on Windows: Shortly before the notification appears, a Powershell-Window pops up for a very short moment. This happens only for the compiled .exe, not when running as Python package. I think it's an upstream issue with notify-py which leverages a powershell command under the hood. I'm already investigating... |
Hi @dynobo , So, I have manually placed python38.dll to rectify the issue. I am not sure how I can fix this. Do you have any idea about this? |
Hi @ashishpatel1992, In the meanwhile I filed an PR for the notification Package: ms7m/notify-py#18 |
Solve with release v0.1.6. |
Glad to know that issue is fixed. 🥇 |
E.g.:
The text was updated successfully, but these errors were encountered: