wilfm / GnomeExtensionMaximusTwo

Removes the title bar on maximised windows. See: https://extensions.gnome.org/extension/844/maximus-two/
63 stars 27 forks source link

Shell 3.16 support #24

Open ptpt opened 9 years ago

ptpt commented 9 years ago

Gnome 3.16 support #21

Fixed some bugs due to API changes

yutzhead commented 9 years ago

Yeah, hi, it's Bill Lumburgh. Lumburgh What's happenin'?

This build still has a problem on Gnome 3.16. On some windows, the header bar hides but the space stays black instead until the window is unmaximized and maximized again.

So, if someone could just go ahead and fix it as soon as possible, that would be terrific.

Mkay?

ptpt commented 9 years ago

Hi @yutzhead , I reproduced the issue you mentioned. If you hide title bar with xprop directly, the black area will appear there.

xprop -id XWINDOW_ID -f _GTK_HIDE_TITLEBAR_WHEN_MAXIMIZED 32c -set _GTK_HIDE_TITLEBAR_WHEN_MAXIMIZED 0x1

If you lock your screen and then come back it appears again. This is likely to be a bug in Gnome Shell that doesn't work with the GTK property very well. I will try to report the problem to Gnome Shell.

MichaelTunnell commented 9 years ago

ptpt yea this is not a problem with this update this a problem with the extension and the shell entirely, it has always happened for me actually.

desyncr commented 9 years ago

@yutzhead As mentioned before the black bar on top is not caused by the update or related with 3.16. And it's addressed here. Pull those changes locally and report any problem there.

@ptpt Can report the extension loads and work properly on 3.16. The only issue I saw is:

(gnome-shell:7036): Gjs-WARNING **: JS ERROR: TypeError: Shell.StageInputMode is undefined onFocusChange@/home/user/.local/share/gnome-shell/extensions/maximus-two@wilfinitlike.gmail.com/app_menu.js:79

Which is also addressed with https://github.com/wilfm/GnomeExtensionMaximusTwo/pull/26 at here.

ptpt commented 9 years ago

@desyncr Thanks for the PR. It works very well.

jamerlan commented 8 years ago

so.. will someone merge it???

desyncr commented 8 years ago

Maybe @wilfm doesn't have time or interest to review and merge stuff. Hope a contributor to handle merges and stuff is added eventually.

jamerlan commented 8 years ago

I tested ptpt version with fedora 22 and it works good! Please merge pull request