Skip to content

Exempt DaVinci Resolve from default window opacity - #6382

Merged
dhh merged 2 commits into
basecamp:quattrofrom
28allday:resolve-full-opacity
Jul 26, 2026
Merged

Exempt DaVinci Resolve from default window opacity#6382
dhh merged 2 commits into
basecamp:quattrofrom
28allday:resolve-full-opacity

Conversation

@28allday

@28allday 28allday commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

The default-opacity tag (0.985 focused / 0.96 unfocused) currently applies to DaVinci Resolve, so the viewer and scopes render slightly translucent over the wallpaper. For a colour-grading app that visibly shifts what you're looking at, in the same way it would for video playback — which is why the browser video/PiP, qemu, and RetroArch rules already opt out.

This adds the same opt-out to the existing Resolve rule (tag = "-default-opacity", opacity = "1 1"), mirroring retroarch.lua.

Running 4.0.0.alpha (r1373) with Resolve 21 under XWayland here; verified locally with an equivalent user-level opacity override (loads clean, no interaction with the float/stay_focused handling).

Copilot AI review requested due to automatic review settings July 26, 2026 15:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Hyprland window-rule config so DaVinci Resolve windows aren’t affected by the global default-opacity tag, preventing unintended translucency that can distort color-critical grading.

Tip

If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.

Changes:

  • Extend the existing DaVinci Resolve window rule to remove the default-opacity tag.
  • Force Resolve windows to remain fully opaque via opacity = "1 1".
  • Expand the inline comment to explain why Resolve is exempted.

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

Comment thread default/hypr/apps/davinci-resolve.lua Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 26, 2026 22:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@dhh
dhh merged commit d8397ee into basecamp:quattro Jul 26, 2026
28allday added a commit to 28allday/DaVinci-Resolve-Omarchy that referenced this pull request Jul 29, 2026
Rebased onto the Omarchy 4 update (4857667), which the branch these commits
came from predated. Kept both sides:

- the O4 fixes stay (per-package pacman installs, gtk2 dropped, wrapper
  unsets QT_STYLE_OVERRIDE/QT_QPA_PLATFORMTHEME)
- the full-opacity block stays alongside the new rules, noted as redundant
  on Omarchy >= basecamp/omarchy#6382 but still needed on older 4.x builds

README renumbered accordingly and the "002 not fully verified" note dropped,
since both fixes are now verified end-to-end.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.

3 participants