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

Update design of query editor tab #383

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

jaclarke
Copy link
Member

@jaclarke jaclarke commented Nov 28, 2024

Closes #379

Todo:

  • Fixup mobile design
image image image image image

Mid width:
image
Mobile:
image
image

Also updated repl to use tree/table switcher:
image

@jaclarke jaclarke requested a review from 1st1 November 28, 2024 21:42
@jaclarke jaclarke force-pushed the query-editor-update branch from 50ded49 to b9860fe Compare November 29, 2024 20:00
@jaclarke jaclarke marked this pull request as ready for review November 29, 2024 20:06
@jaclarke jaclarke force-pushed the query-editor-update branch from b9860fe to 5db37d7 Compare November 29, 2024 21:06
@1st1
Copy link
Member

1st1 commented Nov 29, 2024

James, any way you can generate a preview link? I’ll be able to test this next week otherwise.

@jaclarke
Copy link
Member Author

jaclarke commented Nov 29, 2024

James, any way you can generate a preview link? I’ll be able to test this next week otherwise.

Yep: https://nebula-ui-git-sql-updates-edgedb.vercel.app/

(Ignore that the SQL bit doesn't work right, I need to hide that stuff in cloud for <6.0 server versions)

@1st1
Copy link
Member

1st1 commented Dec 5, 2024

Can't login to your preview link :/

CleanShot 2024-12-04 at 16 35 08@2x

@jaclarke
Copy link
Member Author

jaclarke commented Dec 5, 2024

Can't login to your preview link :/

It seems to work for me now. @1st1 Is it still broken for you?

@jaclarke jaclarke force-pushed the query-editor-update branch from 7c3a259 to 863322a Compare December 6, 2024 14:27
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.

Redesign the editor UI
2 participants