all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Alan Third <alan@idiocy.org>
To: Aaron Jensen <aaronjensen@gmail.com>
Cc: 28872@debbugs.gnu.org
Subject: bug#28872: 26.0.90; non-native-fullscreen does not play well with ns-transparent-titlebar on macOS
Date: Thu, 19 Oct 2017 12:17:10 +0100	[thread overview]
Message-ID: <20171019111710.GA12599@breton.holly.idiocy.org> (raw)
In-Reply-To: <CAHyO48zUGQi0Y5aa1hHwfAHfmcO3wzwhROky+hUZjV=NyJFhcg@mail.gmail.com>

On Wed, Oct 18, 2017 at 08:08:25PM -0500, Aaron Jensen wrote:
> On October 17, 2017 at 7:26:01 AM, Alan Third
> (alan@idiocy.org(mailto:alan@idiocy.org)) wrote:
> 
> > The issue seems to be that when switching back to the non‐fullscreen
> > NSWindow the toolbar height calculation fails and produces a negative
> > value.
> >
> > We used to use a hard‐coded value for this, but the calculation works
> > better in every circumstance except, apparently, this one.
> 
> Would it be possible to use a hard-coded 0 for the case when there is
> no toolbar at least? Or do you have a proper fix in mind?

No, I was pretty much just thinking of returning a hard‐coded value
when it’s negative.

The toolbar size calculation takes the size of the actual contentView,
which should be the space below the toolbar, and subtracts that from
the size of a calculated contentView that doesn’t have a toolbar.

This is how apple recommend you calculate the size of the toolbar.

Non‐native fullscreen uses another NSWindow (like we were discussing
for undecorated frames), so when it switches back out of fullscreen it
changes back to the original NSWindow.

At this point, the calculated contentView is correctly sized, but the
actual contentView seems to take up the ENTIRE NSWindow, including the
titlebar.

I imagine we need to somehow force the NSWindow to recalculate its
sizes, but I’ve no idea how.
-- 
Alan Third





  reply	other threads:[~2017-10-19 11:17 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-17  3:00 bug#28872: 26.0.90; non-native-fullscreen does not play well with ns-transparent-titlebar on macOS Aaron Jensen
2017-10-17 14:25 ` Alan Third
2017-10-19  1:08   ` Aaron Jensen
2017-10-19 11:17     ` Alan Third [this message]
     [not found]       ` <CAHyO48y5jy7L7E+BZTtTAtd5eGcUb-W54FQXhfEh444WPx4j3A@mail.gmail.com>
2017-10-25 13:38         ` Alan Third
2017-10-25 15:15           ` Aaron Jensen
2017-10-28 13:19             ` Alan Third
2017-10-28 17:15               ` Aaron Jensen
2017-10-28 17:17                 ` Aaron Jensen
2017-11-05 18:59               ` Alan Third
2017-10-26  7:56           ` martin rudalics
2017-10-28 13:11             ` Alan Third
2017-10-31  8:40               ` martin rudalics

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=20171019111710.GA12599@breton.holly.idiocy.org \
    --to=alan@idiocy.org \
    --cc=28872@debbugs.gnu.org \
    --cc=aaronjensen@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.