-
Notifications
You must be signed in to change notification settings - Fork 66
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
taskbar-autohide-better 1.0 #328
Conversation
c85cbc4
to
fe54e1e
Compare
mods/taskbar-autohide-better.wh.cpp
Outdated
// ==WindhawkModReadme== | ||
/* | ||
# Better Taskbar Autohide | ||
Allow taskbar autohide when inactive window notified |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Up to you, but I think the description can be improved. What does "inactive window notified" mean? What happens without the mod and what changes when the mod is used?
I've just noticed that the image link is incorrect, it should have been https://i.imgur.io/BNyrMQc.gif, not https://imgur.io/BNyrMQc.gif. Feel free to submit an update. Also, while at it, it can be helpful to specify the supported Windows versions. I assume this mod is only for Windows 10? |
it can compatible with Windows 11. But before that, consider provide a api to check Windows version like: windhawk-mods/mods/taskbar-thumbnail-reorder.wh.cpp Lines 648 to 669 in 8494ed1
|
This function name is slightly misleading, a better name would be You can copy that function, or, if you just want to truly check the Windows version, you already have Windows APIs for that, for example: windhawk-mods/mods/no-taskbar-item-glow.wh.cpp Lines 51 to 55 in 8494ed1
|
preview: