Skip to content

Fix DaVinci Resolve focus lock#5926

Open
AksharP5 wants to merge 3 commits into
basecamp:devfrom
AksharP5:fix-davinci-resolve-focus-lock
Open

Fix DaVinci Resolve focus lock#5926
AksharP5 wants to merge 3 commits into
basecamp:devfrom
AksharP5:fix-davinci-resolve-focus-lock

Conversation

@AksharP5
Copy link
Copy Markdown

@AksharP5 AksharP5 commented May 20, 2026

Fixes #5887.

DaVinci Resolve should still float under XWayland, but applying stay_focused to every Resolve window prevents focusing other apps and monitors while Resolve is open.

This narrows the default rule to keep Resolve floating without globally locking focus.

Tested:

  • lua -e 'assert(loadfile("default/hypr/apps/davinci-resolve.lua"))'
  • bash test/omarchy-cli-test.sh

Copilot AI review requested due to automatic review settings May 20, 2026 21:49
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the Hyprland window rule for DaVinci Resolve to avoid focus-locking behavior by removing stay_focused and narrowing the rule to a simple floating configuration.

Changes:

  • Replaced the previous “float + stay_focused” Resolve rule with “float only”.
  • Expanded comments to document why stay_focused is not used.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread default/hypr/apps/davinci-resolve.lua Outdated
Copilot AI review requested due to automatic review settings May 20, 2026 21:55
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread default/hypr/apps/davinci-resolve.lua Outdated
Copilot AI review requested due to automatic review settings May 20, 2026 22:04
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@AksharP5 AksharP5 force-pushed the fix-davinci-resolve-focus-lock branch from 56cd366 to 1d38ac4 Compare May 20, 2026 22:06
@AksharP5 AksharP5 force-pushed the fix-davinci-resolve-focus-lock branch from 1d38ac4 to 930b28c Compare May 20, 2026 22:11
Copilot AI review requested due to automatic review settings May 20, 2026 22:11
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread default/hypr/apps/davinci-resolve.lua Outdated
Comment thread default/hypr/apps/davinci-resolve.lua Outdated
@AksharP5 AksharP5 force-pushed the fix-davinci-resolve-focus-lock branch from 930b28c to 69c0d81 Compare May 20, 2026 22:15
Copilot AI review requested due to automatic review settings May 20, 2026 22:29
@AksharP5 AksharP5 force-pushed the fix-davinci-resolve-focus-lock branch from 69c0d81 to d4fb140 Compare May 20, 2026 22:29
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@AksharP5 AksharP5 force-pushed the fix-davinci-resolve-focus-lock branch from d4fb140 to e2edced Compare May 20, 2026 22:30
@dhh
Copy link
Copy Markdown
Member

dhh commented May 21, 2026

CC @28allday

@28allday
Copy link
Copy Markdown
Contributor

will take a look

@28allday
Copy link
Copy Markdown
Contributor

here we go

Tested on omarchy 4.0.0.alpha (Hyprland-in-Lua) — the focus / cursor lock is gone ✅.

One thing worth catching before merge: float = true is applied to every Resolve window including the main editor, so it can no longer tile-split with another app. That breaks drag-and-drop of media from Nautilus → Resolve's timeline / Media Pool (common workflow, no visible drag source when Resolve fills the workspace).

Single-rule fix — drop-in replacement for default/hypr/apps/davinci-resolve.lua, supersedes this PR (keeps the stay_focused drop and the anchored class match, adds a title negative-lookahead so the main editor tiles, and center = true so off-screen dialogs from old monitor layouts come back on-screen):

-- DaVinci Resolve dialog focus handling.
-- Float Resolve's secondary windows (Project Manager, project/render
-- settings, save/open dialogs, etc.) so they don't tile and trap focus.
-- The main editor window is intentionally excluded so it can tile and
-- share the workspace with other apps (e.g. a file manager). The
-- negative-lookahead title pattern is the cleanest way to do this in a
-- single rule and avoids the float-vs-tile precedence issue you hit when
-- two separate class-only and class+title rules try to override each
-- other. center = true is required because Resolve restores its
-- last-known window coordinates per-display, which can place dialogs
-- off-screen when the monitor layout has since changed.
o.window(
{ class = "^[Rr]esolve([.]bin)?$", title = "^(?!DaVinci Resolve( Studio)? - )" },
{ float = true, center = true }
)

Verified on dual monitor (eDP-1 + HDMI-A-1) with both Resolve free and Resolve Studio — main editor tiles, Project Manager / Project Settings float

  • center, no cursor pinning, drag-and-drop from Nautilus to the timeline works.

Copilot AI review requested due to automatic review settings May 21, 2026 14:50
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread default/hypr/apps/davinci-resolve.lua Outdated
@28allday
Copy link
Copy Markdown
Contributor

yeah that will work as well nice

@AksharP5
Copy link
Copy Markdown
Author

AksharP5 commented May 21, 2026

@dhh @28allday changes have been made, kept it a single rule as you mentioned, switched to using Hyprland's supported negate regex syntax. the main window should now be able to tile normally while still keeping the secondary/dialog window(s) floating, and added the center property so dialogs don't reopen off screen.

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.

Issue with windows management - Davinci Resolve

4 participants