Usability issues in SQL query tab

dgloeckner commented 1 year ago

No shortcut for run

CTRL+R or CMD+R on MacOS are common shortcuts for a run action. It would be awesome if I would not have to switch to my mouse to click the play button.

Statistics unclear

See the section marked in yellow.I assume 1178 rows refers to scanned rows in ClickHouse, not returned rows.

clickhouse-client has a much clearer way to show all of this info. Could we try to replicate this?

2 rows in set. Elapsed: 0.008 sec. Processed 8.19 thousand rows, 589.90 KB (1.04 million rows/s., 75.24 MB/s.)

Column / table searchbar is hard to use

It's not clear what the searchbar is supposed to do. Filter tables based on column names?

It's very slow when a few thousand tables are in the schema ** I understand that this is a client-side implementation and network latency might be an issue.
There should be a way to filter tables by name to work with large schemas.

List scroll bar is almost invisible due to color scheme

Dark grey on dark grey just doesn't do it ;)

dgloeckner commented 1 year ago

Don't get me wrong guys! The tool is awesome already!

dgloeckner commented 1 year ago

I made some changes in my fork. Regarding the issues mentioned above I addressed the following ones:

use filtering capabilities of el-tree-v2 to allow searching for non-empty tables, searching by database, table and column name
add a shortcut to run SQL queries
provide more clear table statistics

Please see https://github.com/dgloeckner/ClickCat/tree/improve-ux

wangqinghuan commented 1 year ago

Thank you @dgloeckner. Could you create a pr? We'll merge these changes.

dgloeckner commented 1 year ago

@wangqinghuan thanks and here's the PR: https://github.com/clickcat-project/ClickCat/pull/6

clickcat-project / ClickCat