Skip to content

feature: add per-application opacity controls - #97

Open
amanCodex148 wants to merge 7 commits into
pctrade:mainfrom
amanCodex148:feature/application-opacity
Open

feature: add per-application opacity controls#97
amanCodex148 wants to merge 7 commits into
pctrade:mainfrom
amanCodex148:feature/application-opacity

Conversation

@amanCodex148

Copy link
Copy Markdown

Summary

Adds per-application active opacity controls to the Hyprland settings page.

Features

  • Search installed applications
  • Enable or disable opacity override per application
  • Configure active opacity from 10% to 100%
  • Persist application opacity rules in the shell configuration
  • Generate Hyprland window_rule entries automatically
  • Synchronize changes through the existing HyprlandConfig service
  • Load generated application opacity rules through shellOverrides
  • Validate and clamp invalid opacity values
  • Generate shell override files atomically
  • Avoid unnecessary Hyprland reloads when configuration output is unchanged
  • Handle generator failures with a non-zero exit status

Screenshot

image

Validation

  • Tested on Hyprland with Firefox
  • 50% active opacity generated correctly
  • Configuration persisted to config.json
  • Changing opacity regenerated the Lua rule
  • Changing inactive opacity regenerated application rules
  • Disabling the override removed the generated rule
  • Invalid config handling returns a non-zero exit status
  • Re-running without changes does not reload Hyprland
  • git diff --check passed
  • Python syntax validation passed

Copilot AI lite review requested due to automatic review settings September 7, 2026 17:12

Copilot AI left a comment

Copy link
Copy Markdown

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.

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.

2 participants