Skip to content

Conversation

kemaldotearth
Copy link
Contributor

I have read the CONTRIBUTING.md file.

YES

What kind of change does this PR introduce?

This moves the click through on Query Performance to a sheet as opposed to a resizable area. This gives us more space to play with and sets us up for the Query Details panel revamp.

What this changes:

  • Removed ResizeablePanelGroup.
  • Introduced Sheet.
  • Some tidy up on the tab behaviour.
Before After
Screenshot 2025-09-18 at 13 12 20 Screenshot 2025-09-18 at 13 12 32

This is a small prototype @jordienr - if there's a better way we can approach the implementation, let me know!

This moves the click through on Query Performance to a sheet as opposed to a resizable area. This gives us more space to play with and sets us up for the Query details revamp.
Copy link

vercel bot commented Sep 18, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
design-system Ready Ready Preview Comment Sep 19, 2025 0:37am
docs Ready Ready Preview Comment Sep 19, 2025 0:37am
studio-self-hosted Ready Ready Preview Comment Sep 19, 2025 0:37am
studio-staging Ready Ready Preview Comment Sep 19, 2025 0:37am
ui-library Ready Ready Preview Comment Sep 19, 2025 0:37am
zone-www-dot-com Ready Ready Preview Comment Sep 19, 2025 0:37am
2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
cms Ignored Ignored Sep 19, 2025 0:37am
studio Ignored Ignored Preview Sep 19, 2025 0:37am

Copy link

supabase bot commented Sep 18, 2025

This pull request has been ignored for the connected project xguihxuzqibwxjnimxev because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coveralls
Copy link

coveralls commented Sep 18, 2025

Coverage Status

coverage: 70.533%. remained the same
when pulling ae3218e on feat/move-query-details-to-sheet
into 756e5d7 on master.

@kemaldotearth kemaldotearth marked this pull request as draft September 19, 2025 07:56
@kemaldotearth kemaldotearth marked this pull request as ready for review September 19, 2025 09:59
@kemaldotearth kemaldotearth requested a review from a team as a code owner September 19, 2025 09:59
Copy link
Contributor

@fsansalvadore fsansalvadore left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@kemaldotearth kemaldotearth merged commit 19cfa66 into master Sep 19, 2025
21 of 34 checks passed
@kemaldotearth kemaldotearth deleted the feat/move-query-details-to-sheet branch September 19, 2025 14:01
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.

4 participants