all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Carlos Pita <carlosjosepita@gmail.com>
To: Robert Pluim <rpluim@gmail.com>
Cc: 31968@debbugs.gnu.org,
	Jonathan Kyle Mitchell <kyle@jonathanmitchell.org>
Subject: bug#31968: 26.1.50; Allow to hide title bar on maximize (gtk/gnome/csd)
Date: Mon, 2 Jul 2018 11:22:45 -0300	[thread overview]
Message-ID: <CAELgYhf2cONdmw_M7QL_rJAuyvSoztXdtzuEKSbd+aO-smxduQ@mail.gmail.com> (raw)
In-Reply-To: <87muv9epcz.fsf@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1507 bytes --]

I would oppose to any change that exlicitly undecorates and redecorates the
window because (I'm summing up below what I already mentioned in a more
haphazardly way above):

1. Both under Gnome and under Cinnamon there are scenarios in which
changing the client decorated state wreaks havoc with WM operation. In
Gnome when you remove decorations the client lose focus. In Cinnamon
undecorating and redecorating a maximized window makes it completely
inaccessible, as if the WM couldn't show it anymore. I managed to make the
maximize-undecorate / unmaximize-decorate combo work under Cinnamon, but
that's all. Expect problems under other DEs and WMs.

2. Using Jonathan hook shares the problems described in point 1, since it
explicitly deals with decorations instead of just hinting the WM, but also
fails to redecorate the window in many circumstances, since the hook seems
to be called before the maximized property was removed from the frame
properties alist. Even if there were a more convenient event to hook from
(I was unable to find one in frame.el) I would disregard the solution
because of 1.

So that let's us able to support WMs that offers
_GTK_HIDE_TITLEBAR_WHEN_MAXIMIZED or a similar mechanism, or that
explicitly advertise they allow decoration change events triggered by the
client.

I've requested support for _GTK_HIDE_TITLEBAR_WHEN_MAXIMIZED in the
Cinnamon tracker [1], maybe you can argue a bit more for the feature there.

---

[1] https://github.com/linuxmint/Cinnamon/issues/7681

[-- Attachment #2: Type: text/html, Size: 2007 bytes --]

  reply	other threads:[~2018-07-02 14:22 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-25 20:43 bug#31968: 26.1.50; Allow to hide title bar on maximize (gtk/gnome/csd) Carlos Pita
2018-06-27 10:07 ` Robert Pluim
2018-06-27 13:02   ` Carlos Pita
2018-06-27 13:25     ` Robert Pluim
2018-06-27 15:20       ` Carlos Pita
2018-06-28  4:22         ` Jonathan Kyle Mitchell
2018-06-28  8:02           ` martin rudalics
2018-06-28 12:23             ` Robert Pluim
2018-06-28  8:02     ` martin rudalics
2018-06-28 15:25       ` Robert Pluim
2018-06-29  8:42         ` martin rudalics
2018-06-29  8:48           ` Robert Pluim
2018-06-30  8:33             ` martin rudalics
2018-06-30 22:32               ` Jonathan Kyle Mitchell
2018-06-30 22:54                 ` Carlos Pita
2018-06-30 23:25                   ` Carlos Pita
2018-07-01  0:13                     ` Carlos Pita
2018-07-01  9:04                       ` martin rudalics
2018-07-01 17:23                         ` Carlos Pita
2018-07-01 17:35                           ` Carlos Pita
2018-07-01 17:46                             ` Carlos Pita
2018-07-01 18:08                               ` Carlos Pita
2018-07-02 13:24                                 ` Robert Pluim
2018-07-02 14:28                                   ` Carlos Pita
2018-07-02 17:44                                     ` Carlos Pita
2018-07-02 20:03                                       ` Robert Pluim
2018-07-02 22:29                                         ` Carlos Pita
2018-07-03  2:06                                           ` Carlos Pita
2018-07-03  2:09                                             ` Carlos Pita
2018-07-03 23:35                                               ` Carlos Pita
2018-07-02  9:13                           ` martin rudalics
2018-07-02 13:21                     ` Robert Pluim
2018-07-01  9:02                   ` martin rudalics
2018-07-02 13:31                   ` Robert Pluim
2018-07-02 14:22                     ` Carlos Pita [this message]
2018-07-01  9:02                 ` martin rudalics
2018-07-02  5:06                   ` Jonathan Kyle Mitchell
2020-09-04  4:15         ` Lars Ingebrigtsen
2022-04-28 12:03           ` Lars Ingebrigtsen
2022-04-29  4:28           ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-04-29 10:03             ` Lars Ingebrigtsen
2022-04-29 10:25               ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-04-29 10:31                 ` Lars Ingebrigtsen
2022-04-29 10:42                   ` Lars Ingebrigtsen
2022-04-29 17:11                   ` Juri Linkov
2022-04-29 18:57                     ` Eli Zaretskii

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAELgYhf2cONdmw_M7QL_rJAuyvSoztXdtzuEKSbd+aO-smxduQ@mail.gmail.com \
    --to=carlosjosepita@gmail.com \
    --cc=31968@debbugs.gnu.org \
    --cc=kyle@jonathanmitchell.org \
    --cc=rpluim@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.