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

Second monitor doesn't have tiling/Crash #816

Closed
An-Eagle opened this issue Apr 7, 2024 · 4 comments · Fixed by #825
Closed

Second monitor doesn't have tiling/Crash #816

An-Eagle opened this issue Apr 7, 2024 · 4 comments · Fixed by #825
Labels
bug Undesirable behavior

Comments

@An-Eagle
Copy link

An-Eagle commented Apr 7, 2024

Describe the bug
When trying to switch a window to my second monitor, using shift+super+ctrl+arrows, it causes a hard crash. The top bar still functions, I can still open quick settings, but everything else just goes black and breaks.
If i do this just after starting gnome starts (I have around 30 seconds), it doesn't crash, but the windows on the second display are floating.

I don't know if this is a missing feature, but I also can't drag a window to another display/workspace, it could be related
To Reproduce
Steps to reproduce the behavior:

  1. attempt to move a window using the paperwm keybinds
  2. crash, or get a floating window on the second monitor

Expected behavior
The window moves to the second monitor, and joins a stack on the second monitor

System information:
Please execute ./gather-system-info.sh in you PaperWM clone and paste the output below.
Distribution: Gentoo
GNOME Shell 45.5
Display server: Wayland
Enabled extensions:

Additional context
I only have 1 workspace enabled in gnome settings, I have made sure that I disabled all the settings that weren't recommended in the README.

@An-Eagle An-Eagle added the bug Undesirable behavior label Apr 7, 2024
@Lythenas
Copy link
Collaborator

Lythenas commented Apr 7, 2024

I only have 1 workspace enabled in gnome settings, I have made sure that I disabled all the settings that weren't recommended in the README.

I think that might be the issue. Although I thoght we automatically set the settings to a usable value when PaperWM is enabled. In PaperWM (unlike what Gnome normally does) each monitor uses it's own workspace.

So can you try to set the number of workspaces to at least two?

@An-Eagle
Copy link
Author

An-Eagle commented Apr 7, 2024

I only have 1 workspace enabled in gnome settings, I have made sure that I disabled all the settings that weren't recommended in the README.

I think that might be the issue. Although I thoght we automatically set the settings to a usable value when PaperWM is enabled. In PaperWM (unlike what Gnome normally does) each monitor uses it's own workspace.

So can you try to set the number of workspaces to at least two?

Yes, that allows tiling. Is it normal for it to treat both monitors as 2 completely different workspaces ?

@jtaala
Copy link
Collaborator

jtaala commented Apr 7, 2024

Yes, that allows tiling. Is it normal for it to treat both monitors as 2 completely different workspaces ?

Yes, in actual fact it's required in PaperWM, it's the per monitor workspaces part of the PaperWM description:

PaperWM is a Gnome Shell extension which provides scrollable tiling of windows and per monitor workspaces. It's inspired by paper notebooks and tiling window managers.

It allows separated tiling that PaperWM relies on for multiple monitors. It's not without it's issues though, (see #389 (comment)).

I think that might be the issue. Although I thoght we automatically set the settings to a usable value when PaperWM is enabled. In PaperWM (unlike what Gnome normally does) each monitor uses it's own workspace.

@Lythenas, well, if users use dynamic workspaces we ensure that there's at least one workspaces per detected monitor. However, there's currently no check for fixed number of workspaces option so we better prob add one (e.g. change the setting if it's under the required num workspaces). Note: the default number of workspaces in Gnome (fixed) is 4.

@An-Eagle
Copy link
Author

An-Eagle commented Apr 8, 2024

Yes, that allows tiling. Is it normal for it to treat both monitors as 2 completely different workspaces ?

Yes, in actual fact it's required in PaperWM, it's the per monitor workspaces part of the PaperWM description:

PaperWM is a Gnome Shell extension which provides scrollable tiling of windows and per monitor workspaces. It's inspired by paper notebooks and tiling window managers.

It allows separated tiling that PaperWM relies on for multiple monitors. It's not without it's issues though, (see #389 (comment)).

I think that might be the issue. Although I thoght we automatically set the settings to a usable value when PaperWM is enabled. In PaperWM (unlike what Gnome normally does) each monitor uses it's own workspace.

@Lythenas, well, if users use dynamic workspaces we ensure that there's at least one workspaces per detected monitor. However, there's currently no check for fixed number of workspaces option so we better prob add one (e.g. change the setting if it's under the required num workspaces). Note: the default number of workspaces in Gnome (fixed) is 4.

Makes sense, Thank you

Closing as this is solved

@An-Eagle An-Eagle closed this as completed Apr 8, 2024
jtaala added a commit that referenced this issue Apr 15, 2024
This PR resolves #824.

It adds the following:
- keybinds (`Ctrl`+`Alt`+`left`/`right`/`up`/`down`) which moves the
current workspace to a different monitor in that direction;
- if a workspace is the last on a monitor, we fallback to swapping that
space with the target monitor space;
- informs user with notification if can't move / swap a space;
- FIXES #816 by creating spaces to meet a minimum (i.e. PaperWM breaks
if have 2 monitors and only one space...);
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Undesirable behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants