Skip to content

Some fixes for long-standing issues - #1821

Open
siddhpant wants to merge 15 commits into
pop-os:master_noblefrom
siddhpant:master_resolute
Open

Some fixes for long-standing issues#1821
siddhpant wants to merge 15 commits into
pop-os:master_noblefrom
siddhpant:master_resolute

Conversation

@siddhpant

Copy link
Copy Markdown
Contributor

No description provided.

@siddhpant

Copy link
Copy Markdown
Contributor Author

Closes: #1758 #1794 #1819 #1820 #1801

@leviport
leviport requested review from a team July 6, 2026 16:46
Comment thread README.md
This reverts commit 7898b65.

GNOME 49+ support is a breaking change for older versions.

Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
This reverts commit 3cb093b.

GNOME 49+ support is a breaking change for older versions.

Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
…nd added gnome 49 shell version in metadata"

This reverts commit d22d2ff.

GNOME 49+ support is a breaking change for older versions.

Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
The first window is fully maximized, and then later when a second
window is opened, we don't get border for the first window when it
is in focus because after tiling the first window has stale value
for smart_gapped, and thus is_single_max_screen() returns true,
causing permitted in show_border() to be false.

Let's set smart_gapped for each window in a tree each time we tile.

Closes: pop-os#1758
Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
It caused GNOME crash due to bug in mutter, which was fixed in v49.

But regardless, we should not spam the close signal. Also people might
be using versions before 49.

Closes: pop-os#1794
Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
wl-clipboard invisible window (literally just a dot shows up) can cause
the tiler to make it take an entire window worth of space, and it is
supposed to be immediately closed so we see a tiling flicker. And also
sometimes the window dangles (there is an open bug for it), so it takes
up unnecessary space.

This window is not supposed to be tiled, in fact it has skip-taskbar
flag set. Let's add exceptions for it.

Closes: pop-os#1819
Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
Nautilus file previews, like pressing space on a PDF file, is a
temporary window and it can't zoom in, it is supposed to be overlaid.

Let's add exception for it.

Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
Entering window-management mode removed the normal bindings and
re-added the management bindings, then did the inverse on exit.
This leads to journalctl spam since we overwrite a keybinding,
and mutter logs overwrite warnings.

Register the window-management mode bindings once with it being
inactive initially (Shell.ActionMode.NONE), and later enable it
(Shell.ActionMode.NORMAL) when entering the management mode, and
then disabling on exit.

Closes: pop-os#1820
Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
This branch supports GNOME 46+ only according to README.

Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
…and added gnome 49 shell version in metadata"

This reverts commit 506d799.

Add back the fix for GNOME 49+ branch.

Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
This reverts commit a57d7e2.

Add back the fix for GNOME 49+ branch.

Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
This reverts commit c1f89c3.

Add back the fix for GNOME 49+ branch.

Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
The API had changed here:
https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/4415

Thus we had warnings being logged when in stacked mode. The commit
which added support for GNOME 49 forgot to change that.

Also, to be in line with the fixes commit, we remove the
	unmaxize(HORIZONTAL);
	unmaximize(VERTICAL);
	unmaximize(BOTH);
code to checking for whether the window is maximized and then using
BOTH (i.e. its replacement) directly.

Closes: pop-os#1801
Fixes: d22d2ff ("fix: update window maximization logic to use new properties and added gnome 49 shell version in metadata")
Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
This branch is GNOME 49+ only, and the fix is for GNOME 49+.

We have reverted the commit orignally to restore old compatibility and
re-applied it in this new GNOME-49+-only branch.

So we no longer need the fallback.

Signed-off-by: Siddh Raman Pant <25429745+siddhpant@users.noreply.github.com>
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