Skip to content

Conversation

@camielvs
Copy link
Collaborator

@camielvs camielvs commented Jan 21, 2026

Description

A simple change to make the collapse buttons for the sidebar & context panel push up to the top of the viewport when not looking at a subgraph. They will remain below the subgraph bar when looking at a subgraph (current behaviour).

This behaviour is similar to the new Run Toolbar.

Related Issue and Pull requests

Type of Change

  • Improvement

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

Not looking at subgraph

image.png

Looking at subgraph (current)

image.png

Test Instructions

Confirm buttons still work as expected and are not in weird places.

Additional Comments

[Existing Behaviour] On very narrow screens (aka mobile) viewing a subgraph can cause the subgraph nav bar to spill over multiple lines which covers up these collapse buttons. This is something to look into and fix later.

Copy link
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@camielvs camielvs marked this pull request as ready for review January 21, 2026 01:36
@camielvs camielvs requested a review from a team as a code owner January 21, 2026 01:36
@camielvs camielvs changed the title chore: Side Panel Collapse Icons at top of view when not looking at a Subgraph chore: Side Panel Collapse Buttons at top of view when not looking at a Subgraph Jan 21, 2026
className="absolute top-[95px] z-0 transition-all duration-300 bg-white rounded-r-none shadow-md -translate-x-9"
className={cn(
"absolute z-0 transition-all duration-300 bg-white rounded-r-none shadow-md -translate-x-9",
isViewingSubgraph ? "top-23.75" : "top-14",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

tailwind linter insisted I use 23.75 instead of the full pixel value.

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.

2 participants