Skip to content

[ACTV-454] add fallback when smart search button not found - #1376

Merged
cjasm merged 1 commit into
mainfrom
ACTV-454
Aug 21, 2026
Merged

[ACTV-454] add fallback when smart search button not found#1376
cjasm merged 1 commit into
mainfrom
ACTV-454

Conversation

@cjasm

@cjasm cjasm commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Related issues

Proposed changes

The Step Deck tutorial could construct an OverlayTarget with a missing widget when findChild("AnkiHubSmartSearchButton") returned None during Browser lifecycle timing windows (or stale Browser references). Overlay positioning then attempted to read geometry (topLeft/bottomRight) from a missing target.

How to reproduce

Manually run Step Deck tutorial in Browser:
◦ Verify Smart Search step highlights correctly when button is present.
◦ Verify no crash if button is temporarily unavailable.
◦ Verify fallback step is visible and usable (no half-hidden top-left dialog).

** you can set the return None on _get_smart_search_button_target or you can turn off the smart search feature flag

@cjasm
cjasm requested review from abdnh and r-lelis August 20, 2026 20:14
abdnh
abdnh previously approved these changes Aug 21, 2026
r-lelis
r-lelis previously approved these changes Aug 21, 2026
@cjasm
cjasm dismissed stale reviews from r-lelis and abdnh via 22d52d4 August 21, 2026 18:28
@cjasm
cjasm merged commit cb9931c into main Aug 21, 2026
17 of 19 checks passed
@cjasm
cjasm deleted the ACTV-454 branch August 21, 2026 19:02
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