Skip to content

chore(explore): Clean up page frame flag#114304

Merged
nsdeschenes merged 7 commits intomasterfrom
nd/chore-explore-clean-up-page-frame-flag
Apr 29, 2026
Merged

chore(explore): Clean up page frame flag#114304
nsdeschenes merged 7 commits intomasterfrom
nd/chore-explore-clean-up-page-frame-flag

Conversation

@nsdeschenes
Copy link
Copy Markdown
Contributor

Saw that there was some cleanup around the page frame flag, so I thought I'd pitch in for the explore pages 🎉

Remove useHasPageFrameFeature checks from ExploreContentSection and
StarSavedQueryButton, keeping only the page frame code path.
…gs, profiling

Remove useHasPageFrameFeature branching from these tab headers,
keeping only the TopBar.Slot code path and dropping the legacy
Layout.Header fallback.
Remove useHasPageFrameFeature branching from MetricsHeader and
MetricsTabFilterSection/MetricsTabBodySection. The add metric/equation
buttons and save-as are now always rendered in the filter section.
Remove useHasPageFrameFeature branching from SpansTabHeader and
MultiQueryMode, keeping only the TopBar.Slot code path.
Remove useHasPageFrameFeature branching from ReleaseHeader and
ReleasesHeader, keeping only the TopBar.Slot code path.
Remove useHasPageFrameFeature branching from replay list, detail,
header actions, item dropdown, and list controls. Drops the legacy
Layout.Header/FullViewport/ButtonActionsWrapper fallbacks.
Remove useHasPageFrameFeature branching from SavedQueriesView and
SavedQueriesLandingContent. The create query button logic that was
duplicated in index.tsx is removed since it now lives in the landing
content component unconditionally.
@nsdeschenes nsdeschenes requested a review from a team April 29, 2026 13:05
@nsdeschenes nsdeschenes marked this pull request as ready for review April 29, 2026 13:06
@nsdeschenes nsdeschenes requested review from a team as code owners April 29, 2026 13:06
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 29, 2026
Comment thread static/app/views/explore/releases/detail/header/releaseHeader.tsx
@JonasBa
Copy link
Copy Markdown
Member

JonasBa commented Apr 29, 2026

Thank you @nsdeschenes 🙏🏼

@nsdeschenes nsdeschenes merged commit 059492e into master Apr 29, 2026
68 checks passed
@nsdeschenes nsdeschenes deleted the nd/chore-explore-clean-up-page-frame-flag branch April 29, 2026 14:31
cleptric pushed a commit that referenced this pull request May 5, 2026
Saw that there was some cleanup around the page frame flag, so I thought
I'd pitch in for the explore pages 🎉
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