Skip to content

Sticky nodes prototype - #285943

Draft
Benjamin Christopher Simmonds (benibenj) wants to merge 1 commit into
mainfrom
benibenj/chilly-rattlesnake
Draft

Sticky nodes prototype#285943
Benjamin Christopher Simmonds (benibenj) wants to merge 1 commit into
mainfrom
benibenj/chilly-rattlesnake

Conversation

@benibenj

@benibenj Benjamin Christopher Simmonds (benibenj) commented Jan 5, 2026

Copy link
Copy Markdown
Contributor

This is a prototype for defining nodes to be sticky which are not parent nodes. The code is not fully ready. I implemented two examples for this feature:

  • SCM input and action button
  • chat request messages.

SCM Issues: There are difficulties with the SCM input as it updates it's height after initially rendering it. There is no reference to the SCM action button which is why I was not able to easily support it.

Chat Issues: All chat messages are siblings in the tree which causes in a chat session with 5 requests to render all 5 requests above each other instead of only showing the most recent one. Will require an API to specify the max number of nodes that can be sticky which are siblings.

// cc Ladislau Szomoru (@lszomoru)

@benibenj

Copy link
Copy Markdown
Contributor Author
Code_-_OSS_4UG0zXIQmt.mp4

This comment was marked as resolved.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@lszomoru

Copy link
Copy Markdown
Member

Nice! Looking forward to trying this one out and adopting it in the SCM Changes view.

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