Skip to content

update signals for shell 3.18#41

Open
dffischer wants to merge 4 commits intowilfm:masterfrom
dffischer:master
Open

update signals for shell 3.18#41
dffischer wants to merge 4 commits intowilfm:masterfrom
dffischer:master

Conversation

@dffischer
Copy link
Copy Markdown

Since gnome-shell version 3.18, the 'maximize' and 'unmaximize' signals were replaced by a single 'size-change' signal. The extension seems to work without fixing these errors, but makes gnome-tweak-tool and extensions.gnome.org refuse to activate it.

This is an extension of #40. As I do not know whether it is possible to push commits onto a pull request that I do not own, I created this new pull request containing both changes.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The size-change signal is not compatible with 3.16. It was added in 3.17.4. See: GNOME/gnome-shell@7305466

Gabriel Einsdorf and others added 2 commits October 27, 2015 13:13
Since gnome-shell version 3.18, the 'maximize' and 'unmaximize' signals
were replaced by a single 'size-change' signal.

This is a port of commit 4eadd6eba8c2104a3b203ff86a115a1723e9b7d7 of the
dash-to-dock extension.
@dffischer
Copy link
Copy Markdown
Author

It is now backwards compatible by adapting the signals to the current shell version.

Depending on the shell version, the adequate signals can be used.
dffischer added a commit to dffischer/gnome-shell-extensions that referenced this pull request Oct 28, 2015
This introduces shell version 3.18 compatibility. A pull request [1] is
already filed, but not merged a few weeks since, so compatibility is
shipped out via the fork until upstream merges.

[1]: wilfm/GnomeExtensionMaximusTwo#41
dffischer added a commit to dffischer/gnome-shell-extensions that referenced this pull request Apr 27, 2016
This introduces shell version 3.18 compatibility. A pull request [1] is
already filed, but not merged a few weeks since, so compatibility is
shipped out via the fork until upstream merges.

[1]: wilfm/GnomeExtensionMaximusTwo#41
@beatgammit
Copy link
Copy Markdown

@wilfm Any chance in merging this?

@jcuenod
Copy link
Copy Markdown

jcuenod commented May 20, 2016

It would be useful to support 3.20 as well since it's in the wild (I'm using it).

@dffischer
Copy link
Copy Markdown
Author

This repository seems quite inactive. I recommend using danielkza's fork instead, which currently seems most up to date.

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.

4 participants