Skip to content

2dosi omi mac#3528

Merged
beastoin merged 3 commits intomainfrom
2dosi_omi_mac
Nov 27, 2025
Merged

2dosi omi mac#3528
beastoin merged 3 commits intomainfrom
2dosi_omi_mac

Conversation

@beastoin
Copy link
Copy Markdown
Collaborator

No description provided.

@beastoin beastoin merged commit f21261e into main Nov 27, 2025
1 check passed
@beastoin beastoin deleted the 2dosi_omi_mac branch November 27, 2025 15:30
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces several enhancements to the floating control bar and the 'Ask omi' feature on macOS. The changes include adding a guard to prevent UI updates when the AI conversation is not visible, and fixing the resizability of the control bar. It also improves the reliability of showing the AI conversation from a global shortcut. However, I've found a critical race condition when opening the chat window from the menu bar, which is not handled with the same care as the global shortcut. I've also suggested a refactoring to reduce code duplication and prevent similar bugs in the future.

Comment thread app/macos/Runner/MainFlutterWindow.swift
Comment thread app/macos/Runner/MainFlutterWindow.swift
Glucksberg pushed a commit to Glucksberg/omi-local that referenced this pull request Apr 28, 2026
* Prevent window resize when AI conversation is closed

* Activate app before showing floating control bar and AI chat

* fix: disable window resizing for non-response views

Co-authored-by: aider (anthropic/claude-sonnet-4-5-20250929) <aider@aider.chat>

---------

Co-authored-by: aider (anthropic/claude-sonnet-4-5-20250929) <aider@aider.chat>
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