Skip to content

fix: sort table list at app layer for all databases#562

Merged
datlechin merged 1 commit intomainfrom
fix/sort-table-name-app-layer
Apr 3, 2026
Merged

fix: sort table list at app layer for all databases#562
datlechin merged 1 commit intomainfrom
fix/sort-table-name-app-layer

Conversation

@datlechin
Copy link
Copy Markdown
Collaborator

Summary

  • Sort table list alphabetically in LiveTableFetcher using localizedCaseInsensitiveCompare
  • This covers all database types uniformly (MySQL, PostgreSQL, SQLite, MongoDB, BigQuery, DynamoDB, ClickHouse, Redis, Oracle, DuckDB, Cassandra, Etcd, CloudflareD1, MSSQL)
  • Supersedes fix: Sort table list by name in MongoDb, BigQuery, DynamoDB, MySql. #554 which added per-plugin sorting in only 4 plugins

Single-line change in SidebarViewModel.swift — sorts driver.fetchTables() results before caching.

Test plan

  • Connect to any database → verify sidebar table list is alphabetically sorted
  • Test with mixed case names (e.g., Users, accounts, Orders) → verify case-insensitive sorting

@datlechin datlechin merged commit 8d27b7b into main Apr 3, 2026
1 check passed
@datlechin datlechin deleted the fix/sort-table-name-app-layer branch April 3, 2026 03:08
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.

1 participant