all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Jan Djärv" <jan.h.d@swipnet.se>
To: David De La Harpe Golden <david@harpegolden.net>
Cc: Luka Novsak <lnovsak@gmail.com>, 9181@debbugs.gnu.org
Subject: bug#9181: 24.0.50; Alpha transparency no longer works
Date: Fri, 29 Jul 2011 11:34:15 +0200	[thread overview]
Message-ID: <4E327E97.4070200@swipnet.se> (raw)
In-Reply-To: <4E31EBC3.8050303@harpegolden.net>



David De La Harpe Golden skrev 2011-07-29 01:07:

> *** So why _was_ it working? emacs used to special-case in
> xterm.c/x_set_frame_alpha() that set the property on the parent window of the
> emacs window rather than the emacs window itself [1], it got removed in trunk
> rev 104095 to fix bug #8608 (Jan cc'd)
>

First of all, not all window managers reparent the application top level 
windows.  So putting a property blindly in the parent window does not work for 
those window managers.

Secondly, it is the job of the window manager to replicate the property from 
the application window to any window it may put as parent to that window.
http://lists.freedesktop.org/archives/xdg/2003-December/001413.html:
"Window managers MUST forward the value of this property to any enclosing
frame window."

Note however that _NET_WM_WINDOW_OPACITY isn't in EWMH, so this is not a 
formal specification.

But from my point of view, this is a window manager bug, not a bug in Emacs.

> I for one am not at all sure restoring it is the right thing to do, it seems
> problematic. Emacs doesn't notionally "own" the window manager frame around
> it, and some reparenting window managers could actually use more than one
> level of window between the root and the app window too anyway. (I don't think
> reparenting window managers actually do typically copy the property up in
> practice as suggested under #8608, either, I'm afraid. That would be rather
> unique handling of the property AFAIK)

Reverting the change breaks other window managers as you say.  If a window 
manager does not copy the property it must have a very good rationale for not 
doing so, since the only text about _NET_WM_WINDOW_OPACITY specifically 
specifies that a window manager must do so.

>
> So maybe setting the property in both places (either optionally or always) is
> the best we can do, at least if we want to support xcompmgr.
>

It is not xcompmgr that is buggy, it it the window manager.  But the bug 
report does not say what window manager is used.

	Jan D.







  reply	other threads:[~2011-07-29  9:34 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-25 20:41 bug#9181: 24.0.50; Alpha transparency no longer works Luka Novsak
2011-07-28 12:10 ` David De La Harpe Golden
     [not found]   ` <CAE6FkMJNzKxmfpx+tCE4PaACMJQL3ih4RiOrWXGGHMjcrJuaqA@mail.gmail.com>
2011-07-28 18:16     ` Luka Novsak
2011-07-28 23:07       ` David De La Harpe Golden
2011-07-29  9:34         ` Jan Djärv [this message]
     [not found]           ` <CAE6FkM+TJPFJ9ETwYM+1fg9f6LE54YeC=2t8yN8xLx_KqwBRCw@mail.gmail.com>
2011-07-29 17:46             ` Luka Novsak
2011-07-30  7:04             ` Jan Djärv
2011-08-02 20:23               ` Luka Novsak
2011-08-02 20:28                 ` Lars Magne Ingebrigtsen
2011-08-02 20:49                   ` Luka Novsak
2011-08-03  8:13                 ` Jan Djärv
2011-08-03  8:22                   ` David De La Harpe Golden
2011-08-03 11:50                     ` Jan Djärv
2011-08-03 15:12                       ` David De La Harpe Golden
2011-08-03 17:29                         ` Luka Novsak
2011-08-03 19:01                           ` David De La Harpe Golden
2011-08-04 11:14                 ` Jan Djärv

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=4E327E97.4070200@swipnet.se \
    --to=jan.h.d@swipnet.se \
    --cc=9181@debbugs.gnu.org \
    --cc=david@harpegolden.net \
    --cc=lnovsak@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.