Skip to content

Handle inactive monitors better#447

Draft
jeremypw wants to merge 10 commits into
introduce-layout-managerfrom
testing/inactive-widget
Draft

Handle inactive monitors better#447
jeremypw wants to merge 10 commits into
introduce-layout-managerfrom
testing/inactive-widget

Conversation

@jeremypw
Copy link
Copy Markdown
Collaborator

@jeremypw jeremypw commented Jun 3, 2026

Fixes #446

This PR addresses a number of issues around toggling the enabled property of monitors.

  • Inactive monitors no longer are drawn as part of the layout, but as a smaller widget on the side
  • Layouts that include a disabled monitor have a different key
  • After changing the active state of a monitor the menu pops down and the "Apply" button is made sensitive
  • After changing the active state of a monitor the remaining displays are rescaled and redrawn as if the change has been applied

This heads towards #74 although the disabled displaywidgets are currently still drawn on the overlay. A future improvement would be to put them in a separate sidebar widget.

Copy link
Copy Markdown
Collaborator

@leonardo-lemos leonardo-lemos left a comment

Choose a reason for hiding this comment

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

Code looks good. Thanks.

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