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

Add legacy mode option to fix tooltip positioning issues (#398 and #760) #787

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

liuyd-dev
Copy link

We have observed longstanding issues with tooltip positioning since 2017. Unfortunately a direct fix was not immediately feasible due to upstream dependencies.

This PR humbly proposes a "Show Tooltip in Legacy Mode" option as a workaround to issues #398 and #760. The goal is to use a different method to help address the positioning problems temporarily.

Extensive testing suggests this workaround is reliable without new bugs. While not a permanent solution, it may help alleviate the problem without significant changes for now.

The option remains experimental by nature. We hope to identify a better approach once dependencies allow. Any feedback would be appreciated to progress this effort.

Approval of this proposed change would be greatly valued in resolving a long-troubling user experience issue. We thank you for your consideration and understanding.

@liuyd-dev
Copy link
Author

Hi @canton7 ,

I hope you're doing well. I submitted a PR (Number: #787) on SyncTrayzor and wanted to check in on its progress. It has been three months, and I was wondering if you have had a chance to review and address the PR?

If you need any additional information or further modifications, please let me know. I'm more than willing to collaborate to ensure that this PR is processed as soon as possible.

Thank you for your time and consideration. Looking forward to your response.

Best regards

@The-Fireplace
Copy link

@liuyd-dev Would you be willing to upload a build of this as a release on your fork?

@liuyd-dev
Copy link
Author

@liuyd-dev Would you be willing to upload a build of this as a release on your fork?

I'm not able to conveniently build and release it, but you can refer to this link for building it.

@Ryunam
Copy link

Ryunam commented Aug 26, 2024

I have compiled the modified x64 executable through Visual Studio 2019 with the changes implemented in this PR and I'm sharing the build output here for everyone's convenience.
Please note: this is just the SyncTrayzor.exe file itself. You should first install the official release of the software as normal. Afterwards, go to the install folder and replace the existing SyncTrayzor.exe file with this version.

You can download the build here: SyncTrayzor (exe with legacy tooltip option).zip

After replacing the executable, start SyncTrayzor as normal, then go into File -> Settings and check this box:

image

You should now have the tooltip fully functional, without any top-left shifting. I have been testing this on a Windows 22H2 install, with multiple modifications applied on top of the default taskbar, and the tooltip is showing up and performing as expected.

image

Major credits go to @liuyd-dev for fixing this annoyance. Many thanks!

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