Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion TablePro/Views/AIChat/AIChatPanelView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -234,7 +234,7 @@ struct AIChatPanelView: View {
}
.padding(.horizontal, 12)
.padding(.vertical, 6)
.background(Color.yellow.opacity(0.1))
.background(Color(nsColor: .systemYellow).opacity(0.1))
}

// MARK: - Input Area
Expand Down
8 changes: 5 additions & 3 deletions TablePro/Views/Components/PaginationControlsView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ struct PaginationControlsView: View {
// Settings button (gear icon) - opens popover
Button(action: { showSettings.toggle() }) {
Image(systemName: "slider.horizontal.3")
.font(.system(size: 12))
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.help("Pagination Settings")
Expand All @@ -59,7 +59,8 @@ struct PaginationControlsView: View {
// Previous page button
Button(action: onPrevious) {
Image(systemName: "chevron.left")
.font(.system(size: 11))
.imageScale(.small)
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.disabled(!pagination.hasPreviousPage || pagination.isLoading)
Expand All @@ -80,7 +81,8 @@ struct PaginationControlsView: View {
// Next page button
Button(action: onNext) {
Image(systemName: "chevron.right")
.font(.system(size: 11))
.imageScale(.small)
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.disabled(!pagination.hasNextPage || pagination.isLoading)
Expand Down
2 changes: 1 addition & 1 deletion TablePro/Views/Components/SyncStatusIndicator.swift
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ struct SyncStatusIndicator: View {
case .syncing:
return AnyShapeStyle(.secondary)
case .error:
return AnyShapeStyle(Color.orange)
return AnyShapeStyle(Color(nsColor: .systemOrange))
case .disabled:
return AnyShapeStyle(.tertiary)
}
Expand Down
2 changes: 2 additions & 0 deletions TablePro/Views/Connection/OnboardingContentView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,8 @@ struct OnboardingContentView: View {
.fill(i == currentPage ? Color.accentColor : Color(nsColor: .tertiaryLabelColor))
.frame(width: 8, height: 8)
.scaleEffect(i == currentPage ? 1.2 : 1.0)
.frame(width: 24, height: 24)
.contentShape(Circle())
.onTapGesture { goToPage(i) }
}
}
Expand Down
4 changes: 2 additions & 2 deletions TablePro/Views/DatabaseSwitcher/DatabaseSwitcherSheet.swift
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,7 @@ struct DatabaseSwitcherSheet: View {
Task { await viewModel.refreshDatabases() }
}) {
Image(systemName: "arrow.clockwise")
.font(.system(size: 14))
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.help("Refresh database list")
Expand All @@ -177,7 +177,7 @@ struct DatabaseSwitcherSheet: View {
if databaseType != .sqlite && !isSchemaMode {
Button(action: { showCreateDialog = true }) {
Image(systemName: "plus")
.font(.system(size: 14))
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.help("Create new database")
Expand Down
2 changes: 2 additions & 0 deletions TablePro/Views/Editor/ExplainResultView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,15 @@ struct ExplainResultView: View {
HStack(spacing: 4) {
Button(action: { fontSize = max(10, fontSize - 1) }) {
Image(systemName: "textformat.size.smaller")
.frame(width: 24, height: 24)
}
Text("\(Int(fontSize))")
.font(.caption)
.foregroundStyle(.secondary)
.frame(width: 24)
Button(action: { fontSize = min(24, fontSize + 1) }) {
Image(systemName: "textformat.size.larger")
.frame(width: 24, height: 24)
}
}
.buttonStyle(.borderless)
Expand Down
7 changes: 4 additions & 3 deletions TablePro/Views/Editor/HistoryPanelView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ private extension HistoryPanelView {
showClearAllAlert = true
} label: {
Image(systemName: "trash")
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.disabled(entries.isEmpty)
Expand Down Expand Up @@ -155,7 +156,7 @@ private extension HistoryPanelView {
VStack(spacing: 8) {
if !searchText.isEmpty || dateFilter != .all {
Image(systemName: "magnifyingglass")
.font(.system(size: ThemeEngine.shared.activeTheme.iconSizes.huge))
.font(.largeTitle)
.foregroundStyle(.tertiary)
Text("No Matching Queries")
.font(.system(size: ThemeEngine.shared.activeTheme.typography.body, weight: .medium))
Expand All @@ -166,7 +167,7 @@ private extension HistoryPanelView {
.multilineTextAlignment(.center)
} else {
Image(systemName: "clock.arrow.circlepath")
.font(.system(size: ThemeEngine.shared.activeTheme.iconSizes.huge))
.font(.largeTitle)
.foregroundStyle(.tertiary)
Text("No Query History Yet")
.font(.system(size: ThemeEngine.shared.activeTheme.typography.body, weight: .medium))
Expand Down Expand Up @@ -268,7 +269,7 @@ private extension HistoryPanelView {
var previewEmptyState: some View {
VStack(spacing: 8) {
Image(systemName: "doc.text")
.font(.system(size: ThemeEngine.shared.activeTheme.iconSizes.huge))
.font(.largeTitle)
.foregroundStyle(.tertiary)
Text("Select a Query")
.font(.system(size: ThemeEngine.shared.activeTheme.typography.title3, weight: .medium))
Expand Down
2 changes: 2 additions & 0 deletions TablePro/Views/Editor/QueryEditorView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -81,13 +81,15 @@ struct QueryEditorView: View {
// Clear button
Button(action: { queryText = "" }) {
Image(systemName: "trash")
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.help("Clear Query")

// Format button
Button(action: formatQuery) {
Image(systemName: "text.alignleft")
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.help("Format Query (⌥⌘F)")
Expand Down
1 change: 0 additions & 1 deletion TablePro/Views/Filter/FilterPanelView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,6 @@ struct FilterPanelView: View {
}
} label: {
Image(systemName: "ellipsis.circle")
.font(.system(size: ThemeEngine.shared.activeTheme.iconSizes.small))
}
.menuStyle(.borderlessButton)
.foregroundStyle(.secondary)
Expand Down
2 changes: 2 additions & 0 deletions TablePro/Views/Filter/FilterRowView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -140,6 +140,7 @@ struct FilterRowView: View {
HStack(spacing: 4) {
Button(action: onAdd) {
Image(systemName: "plus")
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.controlSize(.small)
Expand All @@ -148,6 +149,7 @@ struct FilterRowView: View {

Button(action: onRemove) {
Image(systemName: "minus")
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.controlSize(.small)
Expand Down
3 changes: 2 additions & 1 deletion TablePro/Views/Filter/SQLPreviewSheet.swift
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,8 @@ struct SQLPreviewSheet: View {
Spacer()
Button(action: { dismiss() }) {
Image(systemName: "xmark.circle.fill")
.font(.system(size: ThemeEngine.shared.activeTheme.iconSizes.default))
.imageScale(.large)
.frame(width: 24, height: 24)
.foregroundStyle(.tertiary)
}
.buttonStyle(.borderless)
Expand Down
2 changes: 1 addition & 1 deletion TablePro/Views/Main/Child/MainEditorContentView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -394,7 +394,7 @@ struct MainEditorContentView: View {
VStack(spacing: 12) {
Spacer()
Image(systemName: "tray")
.font(.system(size: 36))
.font(.largeTitle)
.foregroundStyle(.secondary)
Text("No rows returned")
.font(.system(size: ThemeEngine.shared.activeTheme.typography.body, weight: .medium))
Expand Down
4 changes: 3 additions & 1 deletion TablePro/Views/Results/InlineErrorBanner.swift
Original file line number Diff line number Diff line change
Expand Up @@ -26,21 +26,23 @@ struct InlineErrorBanner: View {
NSPasteboard.general.setString(message, forType: .string)
} label: {
Image(systemName: "doc.on.doc")
.frame(width: 24, height: 24)
.foregroundStyle(.secondary)
}
.buttonStyle(.plain)
.help(String(localized: "Copy error message"))
if let onDismiss {
Button { onDismiss() } label: {
Image(systemName: "xmark")
.frame(width: 24, height: 24)
.foregroundStyle(.secondary)
}
.buttonStyle(.plain)
}
}
.padding(.horizontal, 12)
.padding(.vertical, 8)
.background(Color.red.opacity(0.08))
.background(Color(nsColor: .systemRed).opacity(0.08))
}
}

Expand Down
2 changes: 1 addition & 1 deletion TablePro/Views/Results/ResultSuccessView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ struct ResultSuccessView: View {
VStack(spacing: 16) {
Spacer()
Image(systemName: "checkmark.circle.fill")
.font(.system(size: 36))
.font(.largeTitle)
.foregroundStyle(.green)
Text(String(format: String(localized: "%lld row(s) affected"), Int64(rowsAffected)))
.font(.system(size: ThemeEngine.shared.activeTheme.typography.body))
Expand Down
6 changes: 3 additions & 3 deletions TablePro/Views/Settings/AISettingsView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ struct AISettingsView: View {
removeSelectedProvider()
} label: {
Image(systemName: "minus")
.frame(width: 24, height: 20)
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.disabled(selectedProviderID == nil)
Expand All @@ -94,7 +94,7 @@ struct AISettingsView: View {
addProvider()
} label: {
Image(systemName: "plus")
.frame(width: 24, height: 20)
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)

Expand All @@ -107,7 +107,7 @@ struct AISettingsView: View {
}
} label: {
Image(systemName: "pencil")
.frame(width: 24, height: 20)
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.disabled(selectedProviderID == nil)
Expand Down
10 changes: 5 additions & 5 deletions TablePro/Views/Sidebar/FavoritesTabView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,7 @@ internal struct FavoritesTabView: View {
private var emptyState: some View {
VStack(spacing: 6) {
Image(systemName: "star")
.font(.system(size: 28, weight: .thin))
.font(.title.weight(.thin))
.foregroundStyle(Color(nsColor: .tertiaryLabelColor))

Text("No Favorites")
Expand All @@ -223,9 +223,9 @@ internal struct FavoritesTabView: View {
viewModel.createFavorite()
} label: {
Label(String(localized: "New Favorite"), systemImage: "plus")
.font(.system(size: 12))
}
.buttonStyle(.borderless)
.controlSize(.small)
.padding(.top, 4)
}
.frame(maxWidth: .infinity, maxHeight: .infinity)
Expand All @@ -234,7 +234,7 @@ internal struct FavoritesTabView: View {
private var noMatchState: some View {
VStack(spacing: 6) {
Image(systemName: "magnifyingglass")
.font(.system(size: 28, weight: .thin))
.font(.title.weight(.thin))
.foregroundStyle(Color(nsColor: .tertiaryLabelColor))

Text("No Matching Favorites")
Expand All @@ -252,9 +252,9 @@ internal struct FavoritesTabView: View {
viewModel.createFavorite()
} label: {
Label(String(localized: "New Favorite"), systemImage: "plus")
.font(.system(size: 11))
}
.buttonStyle(.borderless)
.controlSize(.small)
.foregroundStyle(Color(nsColor: .secondaryLabelColor))

Spacer()
Expand All @@ -263,7 +263,7 @@ internal struct FavoritesTabView: View {
viewModel.createFolder()
} label: {
Image(systemName: "folder.badge.plus")
.font(.system(size: 11))
.frame(width: 24, height: 24)
}
.buttonStyle(.borderless)
.foregroundStyle(Color(nsColor: .secondaryLabelColor))
Expand Down
4 changes: 2 additions & 2 deletions TablePro/Views/Structure/CreateTableView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -149,14 +149,14 @@ struct CreateTableView: View {
HStack(spacing: 8) {
Button(action: addNewRow) {
Image(systemName: "plus")
.frame(width: 16, height: 16)
.frame(width: 24, height: 24)
}
.help(String(localized: "Add Row"))
.disabled(!isGridTab)

Button(action: { handleDeleteRows(selectedRows) }) {
Image(systemName: "minus")
.frame(width: 16, height: 16)
.frame(width: 24, height: 24)
}
.help(String(localized: "Delete Selected"))
.disabled(!isGridTab || selectedRows.isEmpty)
Expand Down
2 changes: 2 additions & 0 deletions TablePro/Views/Structure/TableStructureView+Schema.swift
Original file line number Diff line number Diff line change
Expand Up @@ -119,13 +119,15 @@ extension TableStructureView {
HStack(spacing: 4) {
Button(action: { ddlFontSize = max(10, ddlFontSize - 1) }) {
Image(systemName: "textformat.size.smaller")
.frame(width: 24, height: 24)
}
Text("\(Int(ddlFontSize))")
.font(.caption)
.foregroundStyle(.secondary)
.frame(width: 24)
Button(action: { ddlFontSize = min(24, ddlFontSize + 1) }) {
Image(systemName: "textformat.size.larger")
.frame(width: 24, height: 24)
}
}
.buttonStyle(.borderless)
Expand Down
2 changes: 1 addition & 1 deletion TablePro/Views/Toolbar/ConnectionStatusView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ struct ConnectionStatusView: View {
private var databaseNameLabel: some View {
HStack(spacing: 4) {
Image(systemName: "cylinder")
.font(.system(size: 13))
.imageScale(.small)
.foregroundStyle(ThemeEngine.shared.colors.toolbar.secondaryTextSwiftUI)

Text(databaseName)
Expand Down
8 changes: 3 additions & 5 deletions TablePro/Views/Toolbar/SafeModeBadgeView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,9 @@ struct SafeModeBadgeView: View {
Button {
showPopover.toggle()
} label: {
HStack(spacing: 4) {
Image(systemName: safeModeLevel.iconName)
.font(.system(size: 12, weight: .medium))
.foregroundStyle(safeModeLevel.badgeColor)
}
Image(systemName: safeModeLevel.iconName)
.fontWeight(.medium)
.foregroundStyle(safeModeLevel.badgeColor)
}
.buttonStyle(.plain)
.help(String(format: String(localized: "Safe Mode: %@"), safeModeLevel.displayName))
Expand Down
Loading