Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

i3-sway: Allow providing order of other workspaces #6546

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KoviRobi
Copy link

I'm using i3, not sway.

For multi-monitor setups, I end up the second screen having workspace 10 (because it is bound to Mod4+0, so ordered before the other numbers).

Needs more testing, especially with sway/tweaking to get it to work with sway. But posting it here because it might be useful.

CC #6041

Description

Checklist

  • Change is backwards compatible -- I tried to make it so by making defaultWorkspace have a priority of 100. But strictly speaking this isn't backwards compatible, because it makes workspace number 1 the default, not number 10. (See tests).

  • Code formatted with ./format.

  • Code tested through nix-shell --pure tests -A run.all
    or nix build --reference-lock-file flake.lock ./tests#test-all using Flakes.

    This currently moves the workspace stuff next to eachother, because I thought it's neater, but it isn't necessary.

    Currently the sway tests are broken, because I haven't fixed sway up

  • Test cases updated/added. See example.

  • Commit messages are formatted like

    {component}: {description}
    
    {long description}
    

    See CONTRIBUTING for more information and recent commit messages for examples.

Maintainer CC

@sumnerevans

For multi-monitor setups, I end up the second screen having workspace 10
(because it is bound to Mod4+0, so ordered before the other numbers).
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.

1 participant