Repository navigation
Show thread model from Core config in footer; float sidebar below tooltip layer - #622
Merged
Merged
Conversation
…ltip layer DmConversation.agent_info names the thread's configured model through Core resolve_thread_model, the same answer the sidebar rows and tooltips read, while context usage still comes from the runtime observation. The floating sidebar painted as an overlay subtree, which the compositor orders above every named layer, so each thread-row tooltip lost its cells to the sidebar background: placed on-screen with the right content, never painted. The sidebars now float by absolute position on a comms-sidebar layer below _tooltips instead of overlay=screen.
The layers property only feeds the screen's direct children; nested arrangement reads the authored layers rule, so the sidebar kept index 0 and the session content covered it. The matching rule keeps the sidebar above content and below the tooltip screen-wide.
trissim
marked this pull request as ready for review
October 14, 2026 07:46
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The DM footer now reads the model only from Core's configured-thread answer, and floating sidebars no longer cover tooltips.
Cause of the missing tooltip:
SideBarusedoverlay=screen, whose compositor order discarded the authored layer and sorted the screen tooltip behind the sidebar. Floating sidebars now use absolute positioning oncomms-sidebar, below Textual's tooltip layer.DmConversationconsumescomms.configuration.answer(target)directly, andPromptrenders its existing status owner for simple DM inputs so the model is visible rather than dormant.Installed-path verification on isolated display
:77used Toad code1bd5ca63126f4a4dbfb49ebb33a5565d3c737cc2and Coref99657c05838b3a9014732fab4378396f46c7d02;pip checkpassed. The footer paintedopenai-codex/gpt-5.6-sol, matching Core's configured answer, while the tooltip painted in front of the floating sidebar.