Skip to content

feat(actions): add cross-workspace vertical focus and move variants - #11

Open
DarkJaguar91 wants to merge 1 commit into
noctalia-dev:mainfrom
DarkJaguar91:cross-workspace-vertical-focus
Open

feat(actions): add cross-workspace vertical focus and move variants#11
DarkJaguar91 wants to merge 1 commit into
noctalia-dev:mainfrom
DarkJaguar91:cross-workspace-vertical-focus

Conversation

@DarkJaguar91

@DarkJaguar91 DarkJaguar91 commented Aug 23, 2026

Copy link
Copy Markdown

Summary

Adds new IPC actions that extend the existing vertical focus/movement commands to cross workspace boundaries:

  • window-focus-or-workspace-up / window-focus-or-workspace-down
  • window-move-or-workspace-up / window-move-or-workspace-down

When the focused window is already at the top/bottom of its column, instead of silently doing nothing, these actions switch to the adjacent workspace and restore focus or move the window there. This matches the behavior of Niri and feels more natural for scrollable-tiling layouts where workspace boundaries are meant to be seamless.

The original window-focus-up/down and window-move-up/down actions remain unchanged for users who prefer workspace-local behavior.

Docs updated in docs/user/keybinds.md.

Motivation

Got used to the Niri feature, and was struggling without it.

Type of Change

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

Related Issue

None

Testing

just format
just test
just lint

In my NixOS using my fork as a flake

Manual Coverage

  • Tested in a nested Umbriel session
  • Tested in a native Umbriel session
  • Tested with multiple monitors
  • Tested with a scaled output
  • Tested with native Wayland applications
  • Tested with X11 applications through xwayland-satellite
  • Tested with the scrolling layout
  • Tested with the dwindle layout

Screenshots / Videos

https://drive.google.com/file/d/1rcqfkEse3F1VcwsvKoRQzVotbVzreKMA/view?usp=sharing

Checklist

  • This PR is ready for review, or it is marked as Draft.
  • I read and followed the relevant guidance in CONTRIBUTING.md.
  • I initialized and updated the SceneFX submodule where required.
  • I ran just format, or this PR has no C++ changes.
  • I ran the relevant build, test, lint, or verification commands, or explained why they were not run.
  • I functionally verified compositor behavior where automated checks are insufficient.
  • I self-reviewed the changes.
  • I checked for new warnings or errors.
  • I updated docs/ and examples/config.toml, or this PR does not change user-facing configuration or behavior.
  • I used canonical names for config keys, IPC actions, paths, and identifiers.

Add new IPC actions that extend the existing vertical focus/movement
commands to cross workspace boundaries:

- window-focus-or-workspace-up/down
- window-move-or-workspace-up/down

When the focused window is already at the top/bottom of its column,
instead of silently doing nothing, these actions switch to the adjacent
workspace and restore focus or move the window there. This matches
the behavior of Niri and feels more natural for scrollable-tiling
layouts where workspace boundaries are meant to be seamless.

The original window-focus-up/down and window-move-up/down actions
remain unchanged for users who prefer workspace-local behavior.

docs/user/keybinds.md updated to document the new actions.
@DarkJaguar91
DarkJaguar91 force-pushed the cross-workspace-vertical-focus branch from 46415c8 to e0a414d Compare August 23, 2026 05:41
@Yocraft-2000

Copy link
Copy Markdown
Contributor

How does this fix #7 ? #7 fixes the bug where dragging a pinned window over a fullscreen window make it disappear:

umbrielpinnedbug.mp4

@DarkJaguar91

Copy link
Copy Markdown
Author

How does this fix #7 ? #7 fixes the bug where dragging a pinned window over a fullscreen window make it

hmm, I was testing this at like 2am, and I guess I must have tested on maximized screen instead of full screen. Will remove that reference

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