Skip to content

Fix NSAlert window resolution, remove swipe actions, hide decorative icons#636

Merged
datlechin merged 1 commit intomainfrom
fix/nsalert-accessibility-polish
Apr 8, 2026
Merged

Fix NSAlert window resolution, remove swipe actions, hide decorative icons#636
datlechin merged 1 commit intomainfrom
fix/nsalert-accessibility-polish

Conversation

@datlechin
Copy link
Copy Markdown
Collaborator

Summary

  • Broaden NSAlert window resolution to minimize runModal() usage — try keyWindow, mainWindow, then first visible window before falling back to modal
  • Remove iOS-pattern .swipeActions from history panel (delete already available via keyboard and context menu)
  • Add .accessibilityHidden(true) to 9 decorative empty-state icons across 5 files

Test plan

  • Test SSH connection with unknown host key — verify prompt appears as sheet
  • Test SSH password prompt — verify it appears as sheet
  • History panel — verify delete works via Delete key and context menu
  • Enable VoiceOver — verify empty state icons are not announced

@datlechin datlechin merged commit 07813a9 into main Apr 8, 2026
2 checks passed
@datlechin datlechin deleted the fix/nsalert-accessibility-polish branch April 8, 2026 04:51
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