Skip to content

feat(config): Add window count and workspace pattern support for dyna…#1269

Open
xand3r40r93 wants to merge 2 commits intonikitabobko:mainfrom
xand3r40r93:dynamic-gaps
Open

feat(config): Add window count and workspace pattern support for dyna…#1269
xand3r40r93 wants to merge 2 commits intonikitabobko:mainfrom
xand3r40r93:dynamic-gaps

Conversation

@xand3r40r93
Copy link
Copy Markdown

…mic gaps

Enhance DynamicConfigValue to support window count and workspace pattern matching in per-monitor configurations. This allows for more granular control over gaps based on the number of windows and workspace names.

Key changes:

  • Add windows and workspace parameters for PerMonitorValue
  • Implement window count aware gap configurations
  • Add workspace pattern matching support
  • Update documentation with new configuration examples
  • Add comprehensive test coverage

…mic gaps

Enhance DynamicConfigValue to support window count and workspace pattern matching in per-monitor configurations. This allows for more granular control over gaps based on the number of windows and workspace names.

Key changes:
- Add windows and workspace parameters for PerMonitorValue
- Implement window count aware gap configurations
- Add workspace pattern matching support
- Update documentation with new configuration examples
- Add comprehensive test coverage

@MainActor
func testIgnoreFloatingWindows() {
// Этот тест проверяет, что плавающие окна игнорируются при подсчете
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Comments in Russian, you may want to translate so others can read them :)

@nikitabobko nikitabobko force-pushed the main branch 2 times, most recently from f324d2e to 7aff813 Compare November 24, 2025 01:31
Davincible added a commit to Davincible/AeroSpace that referenced this pull request Feb 11, 2026
# Conflicts:
#	Sources/AppBundle/config/parseGaps.swift
#	Sources/AppBundleTests/config/ConfigTest.swift
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