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

Design/new data source+query editor #2185

Merged
merged 84 commits into from
Jan 18, 2018

Conversation

kocsmy
Copy link
Collaborator

@kocsmy kocsmy commented Dec 27, 2017

TODO

Query Editor

  • Flexible layout: resizable sections
  • Less vertical and horizontal space consumed
  • Move ... to page-header
  • Move Execute and other 2nd action buttons below the query editor
  • Fix spacing on smaller screens
  • Replace refresh icon with play icon on the visualisation section
  • 1 rows @arikfr
  • Show data scanner property @arikfr
  • Content height must match nav height in order to cover overlapping elements
  • Move updated at time to the right
  • The results area now takes the whole remaining screen, so it pushes down the other parts for no reason. Need to apply media queries there so it uses the height of the screen only on bigger screens.
  • Mobile phone sizes: when the Publish button is visible it pushes Execute off screen.
  • In tablet and smaller sizes we need to change the metadata part to be a single row again (Created X ago by Y | Updated Z ago by YY | Refresh Schedule) that will show up below the query editor.
  • Mobile phone sizes: hide Edit Visualization button and Schema Browser.
  • Cross browser check @kocsmy
  • Can't test in IE due to 429 too many requests 50 per 1 hour error
  • Redash themed editor which is lighter
  • Page doesn't load in Safari due to some JS error
  • Table should be called Data (or Results) on Query @arikfr
  • Double/Pointer click to expand and collapse section

New Data Source Flow

  • More visual flow with database logos
  • 2 sections: Databases and Other Data Sources
  • Search input added
  • Add missing "Save" button for the 2nd step
  • Listing page should use the new box design
  • Implement JS functionality

Screenshots

Query Editor

query

screenshot 2017-12-27 14 49 20

New Data Source

Data Source Listing:
screenshot 2017-12-27 11 24 10

New Data Source Step 1 with placeholder logos:
screenshot 2017-12-27 11 24 31

New Data Source Step 2 (+ edit also looks the same):
screenshot 2017-12-27 11 25 08

…xpandable query editor, better smaller screen layout
@kocsmy kocsmy mentioned this pull request Jan 11, 2018
11 tasks
@arikfr
Copy link
Member

arikfr commented Jan 11, 2018

Table should be called Data (or Results) on Query

It's a bit tricky so I decided to skip for now.

Hide name at query metadata on smaller screens

You probably meant on thinner left pane sizes? Because on smaller screens the metadata section has the whole width of the screen and plenty of room for the name. If this is the case, then we can skip this for now (unless <15 minutes of work).

Some notes on responsive views:

  • the results area now takes the whole remaining screen, so it pushes down the other parts for no reason. Need to apply media queries there so it uses the height of the screen only on bigger screens.
  • Mobile phone sizes: when the Publish button is visible it pushes Execute off screen.
  • in tablet and smaller sizes we need to change the metadata part to be a single row again (Created X ago by Y | Updated Z ago by YY | Refresh Schedule) that will show up below the query editor.
  • Mobile phone sizes: hide Edit Visualization button and Schema Browser.

@kocsmy
Copy link
Collaborator Author

kocsmy commented Jan 13, 2018

Thanks for the comments @arikfr — I agree and I'll fix those tomorrow 👌

@kocsmy
Copy link
Collaborator Author

kocsmy commented Jan 14, 2018

You probably meant on thinner left pane sizes? Because on smaller screens the metadata section has the whole width of the screen and plenty of room for the name. If this is the case, then we can skip this for now (unless <15 minutes of work).

This is a legacy task that can be removed now, I'll clear it.

Regarding the other notes: I'm also handling them, thanks for the highlight!

@arikfr
Copy link
Member

arikfr commented Jan 17, 2018

Page doesn't load in Safari due to some JS error

I couldn't reproduce this. What was the error you saw? Something with angular being undefined and you got a blank page?

@kocsmy
Copy link
Collaborator Author

kocsmy commented Jan 18, 2018

screenshot 2018-01-18 02 36 21

and the xmlhttprequest runs forever...

@arikfr
Copy link
Member

arikfr commented Jan 18, 2018 via email

@arikfr
Copy link
Member

arikfr commented Jan 18, 2018

Nevermind -- ran a system update and indeed after the upgrade I get the same error. I don't think it's specific to this branch though. I will make sure and if it isn't, will merge.

@arikfr arikfr merged commit c2e28b2 into getredash:master Jan 18, 2018
@arikfr
Copy link
Member

arikfr commented Jan 18, 2018

Merged. 👏 👏

@denisov-vlad
Copy link
Member

Nice update! Thanks.

But I see that there is no image for python datasource.

@RichardLitt
Copy link

Thanks for noting that, @denisov-vlad. Want to open a new issue to track this?

@denisov-vlad
Copy link
Member

@RichardLitt done: #2240

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.

5 participants