iridium icon indicating copy to clipboard operation
iridium copied to clipboard

Published 20 hours ago • verified publisher icon Idered

Improve UX

Open michaelsogos opened this issue 4 years ago • 6 comments

Dear @Idered

Great extension !!!

My suggestions are:

  1. add a full server browser side panel in order to see DATABASES and TABLES (VIEWS, FUNCTIONS, etc.)
  2. implement function for basic DDL (CREATE, DROP, TRUNCATE, etc.)
  3. query editor with full syntax highlighter and suggestor
  4. save query as workspace files
  5. execution plan (great if it can be more VISUAL than TEXTUAL)
  6. implement a way to dynamically download the right CLIENT (PG, MYSQL, MSSQL, etc.) and relative syntax suggestor engine

what do you think about ?

michaelsogos avatar Feb 18 '21 10:02 michaelsogos

Hey @michaelsogos thanks for suggestions.

  1. I'll have to redo the navigation but this is something I'll definitely do.
  2. Sure, basic Database/Table functions are a must have thing
  3. I was not planning any raw query related stuff but this project is open for contribution
  4. See 3.
  5. Can you please explain this a little more?
  6. And by client you mean the build in client used for queries? If yes then I'm using a knex which is an interface for db clients. I only need to install other clients and embed them within plugin. This is on my todo list. I'll add MySQL first, then SQLite. After that I'll leave the rest in community hands.

Right now I've switched to Figma extension and vscode backend foundation that will be used by all Iridium plugins. I'll get back to Database plugin after I release MVP version of Figma plugin.

Idered avatar Feb 18 '21 11:02 Idered

Neat extension! Some of the UI elements (especially the version select) feel inconsistent with "native" components though. May I suggest using https://github.com/microsoft/vscode-webview-ui-toolkit components 🙏🏼 Should I make a separate issue?

texastoland avatar Oct 08 '21 13:10 texastoland

@texastoland Thanks for showing me that link! I'm currently focused on node dependencies extension but I'll get back to this one soon

Idered avatar Oct 08 '21 13:10 Idered

Oops Node is the 1 I meant! Sorry I didn't read the issue thoroughly.

texastoland avatar Oct 08 '21 13:10 texastoland

Oops Node is the 1 I meant! Sorry I didn't read the issue thoroughly.

Gotcha, I'll definitely improve UI elements soon

Idered avatar Oct 08 '21 15:10 Idered

@texastoland Hey, I updated component styles. Let me know if you have any feedback :)

Idered avatar Oct 21 '21 13:10 Idered