Skip to content

fix(control-center/monitor): render cards within a scrollable layout to accomodate more monitors - #3736

Merged
Ly-sec merged 1 commit into
noctalia-dev:mainfrom
TotalyEnglizLitrate:fix/scrollable-control-center-monitor-tab
Aug 1, 2026
Merged

fix(control-center/monitor): render cards within a scrollable layout to accomodate more monitors#3736
Ly-sec merged 1 commit into
noctalia-dev:mainfrom
TotalyEnglizLitrate:fix/scrollable-control-center-monitor-tab

Conversation

@TotalyEnglizLitrate

@TotalyEnglizLitrate TotalyEnglizLitrate commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Render Monitor cards in the control panel within a scrollable layout instead of directly to the root to allow for properly displaying more than a few monitors

Motivation

Previously, some monitor cards just got clipped off and were inaccessible,

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Build / packaging

Related Issue

Closes #3733

Testing

Open monitor tab in control centre, verify scrollable layout is present/usable.

Manual Coverage

  • Tested on Niri
  • Tested on Hyprland
  • Tested on Sway
  • Tested on another compositor:
  • Tested with different bar positions and density settings
  • Tested at different interface scaling values
  • Tested with multiple monitors

Screenshots / Videos

#3733 (comment)

Checklist

  • This PR is ready for review, or it is marked as Draft.
  • I read and followed the relevant guidance in CONTRIBUTING.md.
  • I ran just format with clang-format v22+ installed, or this PR has no code changes.
  • I ran the relevant build or test commands, or explained why they were not run.
  • I self-reviewed the changes.
  • I checked for new warnings or errors.
  • I will update end-user documentation after merge, or this PR does not change user-facing configuration or behavior.
  • I added or updated assets/translations/en.json, or this PR adds no new user-facing strings.
  • I did not edit non-English translation files unless this PR is explicitly for translation tooling, an import/export sync, or a maintainer-requested locale change.
  • I used the existing canonical names for config keys, IPC names, paths, and identifiers.

Additional Notes

N/A

@TotalyEnglizLitrate
TotalyEnglizLitrate force-pushed the fix/scrollable-control-center-monitor-tab branch from ba24488 to 3967d30 Compare August 1, 2026 07:08
@TotalyEnglizLitrate
TotalyEnglizLitrate force-pushed the fix/scrollable-control-center-monitor-tab branch from 3967d30 to 918b078 Compare August 1, 2026 07:09
@TotalyEnglizLitrate
TotalyEnglizLitrate marked this pull request as ready for review August 1, 2026 07:49
@Ly-sec

Ly-sec commented Aug 1, 2026

Copy link
Copy Markdown
Member

Looks good to me thank you very much @TotalyEnglizLitrate

@Ly-sec
Ly-sec merged commit 93ae25f into noctalia-dev:main Aug 1, 2026
2 checks passed
@TotalyEnglizLitrate
TotalyEnglizLitrate deleted the fix/scrollable-control-center-monitor-tab branch August 1, 2026 13:25
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.

[BUG] Control Center - Monitor: Only 3 Monitors Are Fully Visible and No Scroll For the Remaining Pills

2 participants