Skip to content

feat(explore): Move to a sticky header layout#113780

Merged
nsdeschenes merged 7 commits intomasterfrom
nd/feat-explore-move-to-a-sticky-header-layout
Apr 27, 2026
Merged

feat(explore): Move to a sticky header layout#113780
nsdeschenes merged 7 commits intomasterfrom
nd/feat-explore-move-to-a-sticky-header-layout

Conversation

@nsdeschenes
Copy link
Copy Markdown
Contributor

@nsdeschenes nsdeschenes commented Apr 23, 2026

This PR updates the explore page filter level to be sticky, so when users scroll their page filters travel with them.

Examples:

Traces
Screenshot 2026-04-23 at 07 15 48


Logs
Screenshot 2026-04-23 at 07 15 38


Application Metrics
Screenshot 2026-04-23 at 07 15 57

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 23, 2026
ViewportConstrainedPage applies overflow: hidden and contain: size,
which prevented the ExploreBodySearch sticky positioning on the logs
page from working — the constrained container has no scroll for the
sticky element to participate in.

Move ViewportConstrainedPage inward to wrap only the body region
(sidebar + content) instead of the entire page shell, leaving the
search/filter row in the page's normal scroll context. This matches
the metrics page layout and preserves the table-expand viewport
behavior.

Co-Authored-By: Claude <noreply@anthropic.com>
The interface is no longer used outside useTableExpando.tsx after the
sticky header refactor. Drop the export to satisfy knip.

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 933246d. Configure here.

Comment thread static/app/views/explore/logs/logsTab.tsx
Move the footer-hiding selector to the logs page wrapper so it can still
reach the global footer after the constrained table layout moved under an
extra DOM wrapper. Keep the sticky header structure intact while preserving
expanded-table and short-viewport footer hiding.

Co-Authored-By: GPT-5.5 <noreply@openai.com>
Made-with: Cursor
…e-to-a-sticky-header-layout

# Conflicts:
#	static/app/views/explore/logs/logsTab.tsx
@nsdeschenes nsdeschenes marked this pull request as ready for review April 27, 2026 14:34
@nsdeschenes nsdeschenes requested a review from a team as a code owner April 27, 2026 14:34
Comment thread static/app/views/explore/logs/content.tsx
Copy link
Copy Markdown
Member

@nikkikapadia nikkikapadia left a comment

Choose a reason for hiding this comment

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

lgtm! and thanks for the reminder i'll put this in the errors ui as well 😌

Copy link
Copy Markdown
Member

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

This is really snazzy.

...I wonder if we can actually do away with the logs table expando work by standardizing on this scroll behavior? If we're ok with losing the chart visualization on scroll up (I certainly am!) the differences become a lot lesser...

@nsdeschenes nsdeschenes merged commit b5b7788 into master Apr 27, 2026
65 checks passed
@nsdeschenes nsdeschenes deleted the nd/feat-explore-move-to-a-sticky-header-layout branch April 27, 2026 16:15
cleptric pushed a commit that referenced this pull request May 5, 2026
This PR updates the explore page filter level to be sticky, so when
users scroll their page filters travel with them.

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: GPT-5.5 <noreply@openai.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants