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

Improved Status Message GUIs #239

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

Conversation

zspz
Copy link

@zspz zspz commented Mar 8, 2025

When using other windows on the same monitor as the running instances, the status messages sometimes display above your active window, which can be a distraction.

Screen.Recording.2025-03-07.190830.mp4

To fix this, I've improved the way status messages are shown so that they no longer go above other windows. I've also optimised the GUI using SetTextAndResize so that the status messages can be updated without creating a new GUI (reducing flicker and speeding up this frequently called function). I've also added the instance window as owner of the status messages, to fix the common issue of them sometimes hiding behind the instance window.


PTCGPB.ahk:

  • Set average time and total packs GUI position to behind the active window before it is shown, so that it doesn't show above.

1.ahk:

  • Set buttons GUI's position to the bottom of the z-stack before it is shown. Since the instance window is its owner, the GUI will still show above it.
  • Optimised CreateStatusMessage function to apply the above modification and also added the ability to resize without creating a new GUI (see comment in code).

Main.ahk:

  • Applied the same changes that I made to 1.ahk. This needs testing though, because I don't run Main on my bot.

PTCGPB.ahk:
- Set average time and total packs GUI position to behind the active window before it is shown, so that it does not show above.

1.ahk:
- Set buttons GUI's position to the bottom of the z-stack before it is shown. Since the instance window is its owner, the GUI will still show above it.
- Optimised `CreateStatusMessage` function to apply the above modification and also added the ability to resize without creating a new GUI (see comment in code).

Main.ahk:
- Applied the same changes that I made to 1.ahk. This needs testing though, because I don't run Main on my bot.
@Arturo-1212
Copy link
Owner

This is great. There was recently a large update pushed to the main. Would you be able to edit your pull request and make it compatible with the latest main version?

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.

2 participants