Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Some studio tables UI breaks when resizing browser window #23162

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

liambirkin
Copy link
Contributor

@liambirkin liambirkin commented Apr 22, 2024

I have read the CONTRIBUTING.md file.

YES

What kind of change does this PR introduce?

Bug fix
Updates to tailwind classes

What is the current behavior?

Some studio tables were breaking when resizing browser window to a small size.

Please link any relevant issues here.
#22979

What is the new behavior?

Table columns now do not hide on smaller screen sizes, and instead the table becomes horizontally scrollable.

Feel free to include screenshots if it includes visual changes.
Before:
CleanShot 2024-04-23 at 08 56 57@2x

After:
(Table is horizontally scrollable)
CleanShot 2024-04-23 at 08 59 51@2x

Additional context

Looks like this problem is affecting a lot more tables in studio than just the ones listed in the issue.
I'm happy to continue digging into the rest if this change is wanted.

Probably could also do with some wrapping on the create button above the tables when on a smaller screen size like the auth users table does.

@liambirkin liambirkin requested a review from a team as a code owner April 22, 2024 22:48
Copy link

vercel bot commented Apr 22, 2024

@liambirkin is attempting to deploy a commit to the Supabase Team on Vercel.

A member of the Team first needs to authorize it.

@Julius-Java
Copy link

Amazing work man, I hope this gets approved and merged soon.

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.

None yet

2 participants