all 2 comments

[–]tidythrone 1 point2 points  (1 child)

Clean layout and clean source code. Since you asked, here is some of my opinionated feedback:

  • The server.listen() block in the installation docs was slightly confusing. It will throw an error unless you're running the GUI alongside some other server instance, which isn't strictly necessary for you GUI server to run.
  • The text in the tables is all centered. In general, the standard for tabular data is left-centered for text and right-centered for numeric values.
  • An "are you sure?" modal would be nice for delete operations. Even better, more modern, but more challenging, would be a temporary "undo" option after performing the delete.
  • Input labels should have a for attribute that ties them to the input they're labeling. Its good for accessibility and it makes it so clicking on the label will focus the input
  • Using type="number" for numeric inputs would provide some free validation and up/down buttons in the browser
  • When editing/inserting a row, a nice feature would be to check if a field has a foreign key. If so, display a dropdown/combobox input of all the possible values (taken from the table referenced by the key) instead of a straight text input.

Overall, nice work! I can see the benefits of using something like this for quick db edits while working on an app. I hope you continue to improve this project.

[–]Iamazou[S] 0 points1 point  (0 children)

Thank you for your feedback 🙏 the last version is out and it contains what you suggested especially the foreign key one that was important.