Sequel-Ace icon indicating copy to clipboard operation
Sequel-Ace copied to clipboard

View references invalid table(s) or column(s)

Open fmaltez opened this issue 1 year ago • 2 comments

MacOS 12.1 (Macbook M1) Sequel Ace Version 4.0.5 - Build 20046

When a view becomes invalid (ie: deleted column present in a view) Sequel Ace partly misidentifies a view as a table: screenshot

The options in that menu partly work (mysql in Console):

  • "Copy Create Table Syntax" (SHOW CREATE TABLE viewName) works
  • "Delete Table" (DROP TABLE viewName) doesn't work

Consequences for users:

  • if you want to fix the error, the user can copy "create view syntax" but it will have to drop the view manually.

My suggestion: For invalid views/tables use the same icon for a view/table but with an overlapped icon (example https://www.iconfinder.com/icons/2843656/empty_error_folder_no_wrong_icon) to identify the error or keeping the same style an icon with a red table for both (or an orange for an invalid view and a red for an invalid table)

Regarding the error message that currently appears:

  1. overlaps the view/table name
  2. is too big for the column size of the Table&Views

Since the error is available when you click in it, I would suggest totally removing the error message from the Tables&Views and to use solely the new icon(s) to identify these views/tables with errors. Or at least reduce the error message to the minimum (ie:error), since the text overlapping prevents reading the table/view name or the error message. Screenshot 2023-06-05 at 13 46 48

fmaltez avatar Jun 05 '23 12:06 fmaltez

the image with the context menu

fmaltez avatar Jun 05 '23 12:06 fmaltez

This issue has been marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar Aug 10 '23 04:08 stale[bot]