Skip to content

Run render loop regardless of canvas visibility#30

Merged
glopesdev merged 1 commit into
bonsai-rx:mainfrom
glopesdev:render-loop-fix
May 4, 2026
Merged

Run render loop regardless of canvas visibility#30
glopesdev merged 1 commit into
bonsai-rx:mainfrom
glopesdev:render-loop-fix

Conversation

@glopesdev
Copy link
Copy Markdown
Member

Emit the frame notification each frame in the visualizer render handler, regardless of whether the canvas window is the active tab.

The previous approach of conditioning emission on Begin caused tab focus to flicker when operators maintained their own dockable windows. In case both windows were docked and the main window lost tab selection, the other window Begin/End pair was never called and so ImGui reselected the main window tab, which again redrew and selected the other window, causing a loop.

Emit the frame notification each frame in the visualizer render handler,
regardless of whether the canvas window is the active tab. The
previous approach of conditioning emission on Begin caused tab focus to
flicker when operators maintained their own dockable windows.
In case both windows were docked and the main window lost tab selection,
the other window Begin/End pair was never called and so ImGui reselected
the main window tab, which again redrew and selected the other window,
causing a loop.
@glopesdev glopesdev added the fix Pull request that fixes an issue label May 4, 2026
@glopesdev glopesdev merged commit c1afd56 into bonsai-rx:main May 4, 2026
10 checks passed
@glopesdev glopesdev deleted the render-loop-fix branch May 4, 2026 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Pull request that fixes an issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant