| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
I guess the check should be added to the protective checks at the method beginning. |
Sorry, something went wrong.
|
Yeah, that makes sense. Reads more cleanly too. Updated. |
Sorry, something went wrong.
|
Thanks for contributing! |
Sorry, something went wrong.
|
Thanks! What is the best way for us to get this into production quickly to fix the crashes for our users? |
Sorry, something went wrong.
|
@TeaDrivenDev If you are using still the v5 then we can add this fix to this version as a hotfix. |
Sorry, something went wrong.
|
@punker76 Yes, we are currently on 5.0.1. If a hotfix is possible, that would be much appreciated. |
Sorry, something went wrong.
Check for PresentationSource before attempting to move tooltip
|
Thanks! |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Describe the changes you have made to improve this project
This PR adds a check for a PresentationSource connected to the PlacementTarget before trying to move a tooltip, as suggested in #165.
Unit test
None.
Additional context
We are using the automove tooltip from ControlzEx in a few places. Some of our users don't use the application on their own machines, but rather connect to their environment via RDP and Citrix. When the RDP connection is closed after using the application for a while, and later reconnected, the application occasionally crashes with the following exception:
We haven't been able to reproduce the issue in a controlled environment, but we have a number of error reports regarding this from users.
Closed Issues
None.