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

fix show_tooltip for AHK v2 #363

Merged
merged 2 commits into from
Jan 15, 2025
Merged

fix show_tooltip for AHK v2 #363

merged 2 commits into from
Jan 15, 2025

Conversation

spyoungtech
Copy link
Owner

@spyoungtech spyoungtech commented Jan 15, 2025

ensures that arguments are correctly parsed as numbers when provided and fixes a problem in AHKv2 where the tooltip window creation may be blocked by returning to the blocking daemon input loop until the next command invocation

Previously, an error may be thrown when x or y parameters were not provided when using AHK v2 and tooltips for successful calls may be delayed until a subsequent command is received.

Completes the fix from #362 and fully resolves #361

This also reverts a change that pinned mypy to 1.13.0 to avoid new typing issues that arose with abstract methods when upgrading mypy to 1.14.1 -- the typing issues were resolved and mypy has been unpinned.

ensures that arguments are correctly parsed as numbers when provided and fixes a problem in AHKv2 where the tooltip window creation may be blocked by returning to the blocking daemon input loop until the next command invocation
@coveralls
Copy link

coveralls commented Jan 15, 2025

Coverage Status

coverage: 76.876% (+0.04%) from 76.833%
when pulling caadd3a on fix-tooltip-v2
into a6e7a0f on main.

@spyoungtech spyoungtech merged commit b2e9f5d into main Jan 15, 2025
13 checks passed
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.

Error when calling ahk.show_tooltip()
2 participants