GNU bug report logs - #62234
Toolbar does not work properly (with Emacs 29 or 30)

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 62234 in the body.
You can then email your comments to 62234 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
To: bug-gnu-emacs <at> gnu.org
Subject: Toolbar does not work properly (with Emacs 29 or 30)
Date: Fri, 17 Mar 2023 08:55:54 +0100

To reproduce:

Build a fresh Emacs app.

Start the app. The save icon is grayed out only after a mouse movement (which can be as small as you like).
 

Konrad

Message #8 received at 62234 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
Cc: 62234 <at> debbugs.gnu.org
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Fri, 17 Mar 2023 10:27:20 +0200

> From: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
> Date: Fri, 17 Mar 2023 08:55:54 +0100
> 
> To reproduce:
> 
> Build a fresh Emacs app.
> 
> Start the app. The save icon is grayed out only after a mouse movement (which can be as small as you like).

Doesn't happen here.

Please post the system and Emacs configuration information collected
by "M-x report-emacs-bug", so we could see what kind of platform you
use and how was Emacs configured.

Message #11 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
To: bug-gnu-emacs <at> gnu.org,
 Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Fri, 17 Mar 2023 11:38:48 +0100

The platform is the NS port. Emacs was not configured at al.



> Am 17.03.2023 um 09:27 schrieb Eli Zaretskii <eliz <at> gnu.org>:
> 
>> From: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
>> Date: Fri, 17 Mar 2023 08:55:54 +0100
>> 
>> To reproduce:
>> 
>> Build a fresh Emacs app.
>> 
>> Start the app. The save icon is grayed out only after a mouse movement (which can be as small as you like).
> 
> Doesn't happen here.
> 
> Please post the system and Emacs configuration information collected
> by "M-x report-emacs-bug", so we could see what kind of platform you
> use and how was Emacs configured.

Message #14 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
Cc: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Fri, 17 Mar 2023 13:49:11 +0200

> From: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
> Date: Fri, 17 Mar 2023 11:38:48 +0100
> 
> The platform is the NS port.

OK, so maybe this is NS-specific.  Can someone who has access to the
NS port please try reproducing this?

> Emacs was not configured at al.

??? How can this be true?  Emacs needs to be configured before it is
built.  So I think there's a misunderstanding here regarding the
meaning of "configured".  What I meant were the values of
system-configuration-options and system-configuration-features.

Thanks.

Message #17 received at 62234 <at> debbugs.gnu.org (full text, mbox):

From: Daniel Martín <mardani29 <at> yahoo.es>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 62234 <at> debbugs.gnu.org, Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Fri, 17 Mar 2023 14:11:43 +0100

Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
>> Date: Fri, 17 Mar 2023 11:38:48 +0100
>> 
>> The platform is the NS port.
>
> OK, so maybe this is NS-specific.  Can someone who has access to the
> NS port please try reproducing this?
>

I can certainly reproduce an issue with the toolbar in the NS port.  I
think it's the same issue that the OP is describing.  Here's some steps
to reproduce it:

emacs -Q

The Save and the Cut(Kill) icons are enabled.

Resize the Emacs frame a little bit.

The Save and the Cut(Kill) icons are now disabled, as expected, because
the *scratch* buffer is not backed by a file, and there's no active
region.

Message #20 received at 62234 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Daniel Martín <mardani29 <at> yahoo.es>
Cc: 62234 <at> debbugs.gnu.org, konrad.podczeck <at> univie.ac.at
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Fri, 17 Mar 2023 15:32:16 +0200

> From: Daniel Martín <mardani29 <at> yahoo.es>
> Cc: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>,  62234 <at> debbugs.gnu.org
> Date: Fri, 17 Mar 2023 14:11:43 +0100
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> > OK, so maybe this is NS-specific.  Can someone who has access to the
> > NS port please try reproducing this?
> >
> 
> I can certainly reproduce an issue with the toolbar in the NS port.  I
> think it's the same issue that the OP is describing.  Here's some steps
> to reproduce it:
> 
> emacs -Q
> 
> The Save and the Cut(Kill) icons are enabled.
> 
> Resize the Emacs frame a little bit.
> 
> The Save and the Cut(Kill) icons are now disabled, as expected, because
> the *scratch* buffer is not backed by a file, and there's no active
> region.

Can you try figuring out why the button is enabled at startup?

Message #23 received at 62234 <at> debbugs.gnu.org (full text, mbox):

From: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
To: Eli Zaretskii <eliz <at> gnu.org>,
 62234 <at> debbugs.gnu.org
Cc: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Fri, 17 Mar 2023 16:21:18 +0100

[Message part 1 (text/plain, inline)]

>> Emacs was not configured at al.
> 
> ??? How can this be true?  Emacs needs to be configured before it is
> built.  So I think there's a misunderstanding here regarding the
> meaning of "configured".  What I meant were the values of
> system-configuration-options and system-configuration-features.
> 
After download, I did

./autogen.sh

./configure 

make

make install.

Resulted in:

In GNU Emacs 29.0.60 (build 1, aarch64-apple-darwin22.3.0, NS
 appkit-2299.40 Version 13.2.1 (Build 22D68)) of 2023-03-17 built on
 iMac-von-Konrad
Repository revision: a2222b9a9bfa039d66f836f06762ddea1544df11
Repository branch: emacs-29
Windowing system distributor 'Apple', version 10.3.2299
System Description:  macOS 13.2.1

Configured features:
ACL DBUS GNUTLS JSON LCMS2 LIBXML2 MODULES NOTIFY KQUEUE NS PDUMPER PNG
SQLITE3 THREADS TOOLKIT_SCROLL_BARS TREE_SITTER WEBP ZLIB

Important settings:
  locale-coding-system: utf-8-unix

Major mode: Fundamental

Minor modes in effect:
  tooltip-mode: t
  global-eldoc-mode: t
  show-paren-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  blink-cursor-mode: t
  buffer-read-only: t
  line-number-mode: t
  indent-tabs-mode: t
  transient-mark-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t

Load-path shadows:
None found.

Features:
(shadow sort mail-extr emacsbug message yank-media puny dired
dired-loaddefs rfc822 mml mml-sec epa derived epg rfc6068 epg-config
gnus-util text-property-search time-date mm-decode mm-bodies mm-encode
mail-parse rfc2231 mailabbrev gmm-utils mailheader sendmail rfc2047
rfc2045 ietf-drums mm-util mail-prsvr mail-utils company-autoloads
auctex-autoloads tex-site info dash-autoloads s-autoloads
parsebib-autoloads pdf-tools-autoloads package browse-url url url-proxy
url-privacy url-expand url-methods url-history url-cookie
generate-lisp-file url-domsuf url-util mailcap url-handlers url-parse
auth-source cl-seq eieio eieio-core cl-macs password-cache json subr-x
map byte-opt gv bytecomp byte-compile url-vars cl-loaddefs cl-lib rmc
iso-transl tooltip cconv eldoc paren electric uniquify ediff-hook
vc-hooks lisp-float-type elisp-mode mwheel term/ns-win ns-win
ucs-normalize mule-util term/common-win tool-bar dnd fontset image
regexp-opt fringe tabulated-list replace newcomment text-mode lisp-mode
prog-mode register page tab-bar menu-bar rfn-eshadow isearch easymenu
timer select scroll-bar mouse jit-lock font-lock syntax font-core
term/tty-colors frame minibuffer nadvice seq simple cl-generic
indonesian philippine cham georgian utf-8-lang misc-lang vietnamese
tibetan thai tai-viet lao korean japanese eucjp-ms cp51932 hebrew greek
romanian slovak czech european ethiopic indian cyrillic chinese
composite emoji-zwj charscript charprop case-table epa-hook
jka-cmpr-hook help abbrev obarray oclosure cl-preloaded button loaddefs
theme-loaddefs faces cus-face macroexp files window text-properties
overlay sha1 md5 base64 format env code-pages mule custom widget keymap
hashtable-print-readable backquote threads dbusbind kqueue cocoa ns
lcms2 multi-tty make-network-process emacs)

Memory information:
((conses 16 63746 4764)
 (symbols 48 7643 0)
 (strings 32 22662 1921)
 (string-bytes 1 694177)
 (vectors 16 15938)
 (vector-slots 8 221783 10774)
 (floats 8 27 32)
 (intervals 56 221 0)
 (buffers 984 11))

[Message part 2 (text/html, inline)]

Message #29 received at 62234 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Daniel Martín <mardani29 <at> yahoo.es>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 62234 <at> debbugs.gnu.org,
 Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Sat, 18 Mar 2023 08:29:36 +0800

Daniel Martín <mardani29 <at> yahoo.es> writes:

> Eli Zaretskii <eliz <at> gnu.org> writes:
>
>>> From: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
>>> Date: Fri, 17 Mar 2023 11:38:48 +0100
>>> 
>>> The platform is the NS port.
>>
>> OK, so maybe this is NS-specific.  Can someone who has access to the
>> NS port please try reproducing this?
>>
>
> I can certainly reproduce an issue with the toolbar in the NS port.  I
> think it's the same issue that the OP is describing.  Here's some steps
> to reproduce it:
>
> emacs -Q
>
> The Save and the Cut(Kill) icons are enabled.
>
> Resize the Emacs frame a little bit.
>
> The Save and the Cut(Kill) icons are now disabled, as expected, because
> the *scratch* buffer is not backed by a file, and there's no active
> region.

If it helps, I don't see this at all on GNUstep.  Both buttons are
initially disabled.

Message #32 received at 62234 <at> debbugs.gnu.org (full text, mbox):

From: Daniel Martín <mardani29 <at> yahoo.es>
To: Po Lu <luangruo <at> yahoo.com>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 62234 <at> debbugs.gnu.org,
 Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Sat, 18 Mar 2023 02:09:04 +0100

[Message part 1 (text/plain, inline)]

Po Lu <luangruo <at> yahoo.com> writes:

>> I can certainly reproduce an issue with the toolbar in the NS port.  I
>> think it's the same issue that the OP is describing.  Here's some steps
>> to reproduce it:
>>
>> emacs -Q
>>
>> The Save and the Cut(Kill) icons are enabled.
>>
>> Resize the Emacs frame a little bit.
>>
>> The Save and the Cut(Kill) icons are now disabled, as expected, because
>> the *scratch* buffer is not backed by a file, and there's no active
>> region.
>
> If it helps, I don't see this at all on GNUstep.  Both buttons are
> initially disabled.

I've browsed some old Apple documentation and I think the way the
toolbar validation is implemented in the NS port may not be totally
correct.
https://web.archive.org/web/20090405012930if_/http://developer.apple.com/documentation/Cocoa/Conceptual/Toolbars/Tasks/ValidatingTBItems.html#//apple_ref/doc/uid/20000753-1036447
states that "If an image item has a valid target/action pair, then the
toolbar will call NSToolbarItemValidation’s validateToolbarItem: on
target if the target implements it; otherwise the item is enabled by
default.".

So I've implemented the above idea in the attached patch and it fixes
the issue for me.  I've also removed the validateVisibleItems override,
which should not be needed (and it's probably incorrect, as Apple's
documentation says that if you override this method, you must call
super).

Could you check if the patch also works fine on GNUStep?  If not,
perhaps we could make the change conditional to Apple systems.  Thanks.

[0001-Fix-toolbar-item-validation-in-the-NS-port.patch (text/x-patch, attachment)]

Message #35 received at 62234 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Daniel Martín <mardani29 <at> yahoo.es>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 62234 <at> debbugs.gnu.org,
 Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Sat, 18 Mar 2023 11:11:26 +0800

Daniel Martín <mardani29 <at> yahoo.es> writes:

> So I've implemented the above idea in the attached patch and it fixes
> the issue for me.  I've also removed the validateVisibleItems override,
> which should not be needed (and it's probably incorrect, as Apple's
> documentation says that if you override this method, you must call
> super).
>
> Could you check if the patch also works fine on GNUStep?  If not,
> perhaps we could make the change conditional to Apple systems.  Thanks.
>
> From eabab73b0dca6cd7aff67427b4443fd08f3b9bcc Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Daniel=20Mart=C3=ADn?= <mardani29 <at> yahoo.es>
> Date: Sat, 18 Mar 2023 01:47:32 +0100
> Subject: [PATCH] Fix toolbar item validation in the NS port
>
> * src/nsmenu.m ([EmacsToolbar validateVisibleItems]): Remove this
> override, which is not really needed.  Also, it doesn't do the right
> thing, because Apple's documentation states that this method, if
> overriden, must call super.
> * src/nsterm.m ([EmacsView validateToolbarItem:]): Implement this
> NSToolbarItemValidation method instead, which is called by AppKit when
> it needs to validate an image item.
> ---
>  src/nsmenu.m | 8 --------
>  src/nsterm.m | 5 +++++
>  2 files changed, 5 insertions(+), 8 deletions(-)
>
> diff --git a/src/nsmenu.m b/src/nsmenu.m
> index 2c1f575bdf2..f7a401dc407 100644
> --- a/src/nsmenu.m
> +++ b/src/nsmenu.m
> @@ -1346,14 +1346,6 @@ - (void) addDisplayItemWithImage: (EmacsImage *)img
>    enablement = (enablement << 1) | (enabled == YES);
>  }
>  
> -/* This overrides super's implementation, which automatically sets
> -   all items to enabled state (for some reason).  */
> -- (void)validateVisibleItems
> -{
> -  NSTRACE ("[EmacsToolbar validateVisibleItems]");
> -}
> -

I think this needs to be kept on GNUstep.

>  /* delegate methods */
>  
>  - (NSToolbarItem *)toolbar: (NSToolbar *)toolbar
> diff --git a/src/nsterm.m b/src/nsterm.m
> index 0e75cbf3f0f..8bb22c0f756 100644
> --- a/src/nsterm.m
> +++ b/src/nsterm.m
> @@ -8573,6 +8573,11 @@ - (instancetype)toolbarClicked: (id)item
>    return self;
>  }
>  
> +-(BOOL)validateToolbarItem:(NSToolbarItem *)toolbarItem
> +{
> +    return [toolbarItem isEnabled];
> +}
> +

This does no harm on GNUstep.  However, please indent with two spaces,
and place spaces after -, :, and ):

- (BOOL) validateToolbarItem: (NSToolBarItem *) toolbarItem
{
  return [toolbarItem isEnabled];
}

Otherwise, LGTM.  Thanks.

Message #38 received at 62234 <at> debbugs.gnu.org (full text, mbox):

From: Daniel Martín <mardani29 <at> yahoo.es>
To: Po Lu <luangruo <at> yahoo.com>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 62234 <at> debbugs.gnu.org,
 Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Sat, 18 Mar 2023 11:25:51 +0100

[Message part 1 (text/plain, inline)]

Po Lu <luangruo <at> yahoo.com> writes:

>> +-(BOOL)validateToolbarItem:(NSToolbarItem *)toolbarItem
>> +{
>> +    return [toolbarItem isEnabled];
>> +}
>> +
>
> This does no harm on GNUstep.  However, please indent with two spaces,
> and place spaces after -, :, and ):
>
> - (BOOL) validateToolbarItem: (NSToolBarItem *) toolbarItem
> {
>   return [toolbarItem isEnabled];
> }
>
> Otherwise, LGTM.  Thanks.

OK, I've attached a new version of the patch.  Konrad, could you check
if with this patch the Emacs toolbar works as you expect?  Thanks.

[0001-Fix-toolbar-item-validation-in-the-NS-port.patch (text/x-patch, attachment)]

Message #41 received at 62234 <at> debbugs.gnu.org (full text, mbox):

From: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
To: Daniel Martín <mardani29 <at> yahoo.es>
Cc: Po Lu <luangruo <at> yahoo.com>, Eli Zaretskii <eliz <at> gnu.org>,
 62234 <at> debbugs.gnu.org
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Sat, 18 Mar 2023 12:07:39 +0100

After playing around a little bit, I confirm that Emacs now works as expected.

Thanks for your effort,

konrad

> Am 18.03.2023 um 11:25 schrieb Daniel Martín <mardani29 <at> yahoo.es>:
> 
> Po Lu <luangruo <at> yahoo.com> writes:
> 
>>> +-(BOOL)validateToolbarItem:(NSToolbarItem *)toolbarItem
>>> +{
>>> +    return [toolbarItem isEnabled];
>>> +}
>>> +
>> 
>> This does no harm on GNUstep.  However, please indent with two spaces,
>> and place spaces after -, :, and ):
>> 
>> - (BOOL) validateToolbarItem: (NSToolBarItem *) toolbarItem
>> {
>>  return [toolbarItem isEnabled];
>> }
>> 
>> Otherwise, LGTM.  Thanks.
> 
> OK, I've attached a new version of the patch.  Konrad, could you check
> if with this patch the Emacs toolbar works as you expect?  Thanks.
> 
> <0001-Fix-toolbar-item-validation-in-the-NS-port.patch>

Message #46 received at 62234-done <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Daniel Martín <mardani29 <at> yahoo.es>
Cc: luangruo <at> yahoo.com, 62234-done <at> debbugs.gnu.org,
 konrad.podczeck <at> univie.ac.at
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Thu, 23 Mar 2023 10:10:10 +0200

> From: Daniel Martín <mardani29 <at> yahoo.es>
> Cc: Eli Zaretskii <eliz <at> gnu.org>,  62234 <at> debbugs.gnu.org,  Konrad Podczeck
>  <konrad.podczeck <at> univie.ac.at>
> Date: Sat, 18 Mar 2023 11:25:51 +0100
> 
> Po Lu <luangruo <at> yahoo.com> writes:
> 
> >> +-(BOOL)validateToolbarItem:(NSToolbarItem *)toolbarItem
> >> +{
> >> +    return [toolbarItem isEnabled];
> >> +}
> >> +
> >
> > This does no harm on GNUstep.  However, please indent with two spaces,
> > and place spaces after -, :, and ):
> >
> > - (BOOL) validateToolbarItem: (NSToolBarItem *) toolbarItem
> > {
> >   return [toolbarItem isEnabled];
> > }
> >
> > Otherwise, LGTM.  Thanks.
> 
> OK, I've attached a new version of the patch.  Konrad, could you check
> if with this patch the Emacs toolbar works as you expect?  Thanks.

Thanks, installed on the emacs-29 branch, and closing the bug.

Message #49 received at 62234-done <at> debbugs.gnu.org (full text, mbox):

From: Konrad Podczeck <konrad.podczeck <at> univie.ac.at>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: luangruo <at> yahoo.com, 62234-done <at> debbugs.gnu.org,
 Daniel Martín <mardani29 <at> yahoo.es>
Subject: Re: bug#62234: Toolbar does not work properly (with Emacs 29 or 30)
Date: Thu, 23 Mar 2023 15:57:35 +0100

[Message part 1 (text/plain, inline)]

Thanks!

Konrad

> Am 23.03.2023 um 09:10 schrieb Eli Zaretskii <eliz <at> gnu.org>:
> 
>> From: Daniel Martín <mardani29 <at> yahoo.es <mailto:mardani29 <at> yahoo.es>>
>> Cc: Eli Zaretskii <eliz <at> gnu.org <mailto:eliz <at> gnu.org>>,  62234 <at> debbugs.gnu.org <mailto:62234 <at> debbugs.gnu.org>,  Konrad Podczeck
>> <konrad.podczeck <at> univie.ac.at <mailto:konrad.podczeck <at> univie.ac.at>>
>> Date: Sat, 18 Mar 2023 11:25:51 +0100
>> 
>> Po Lu <luangruo <at> yahoo.com> writes:
>> 
>>>> +-(BOOL)validateToolbarItem:(NSToolbarItem *)toolbarItem
>>>> +{
>>>> +    return [toolbarItem isEnabled];
>>>> +}
>>>> +
>>> 
>>> This does no harm on GNUstep.  However, please indent with two spaces,
>>> and place spaces after -, :, and ):
>>> 
>>> - (BOOL) validateToolbarItem: (NSToolBarItem *) toolbarItem
>>> {
>>>  return [toolbarItem isEnabled];
>>> }
>>> 
>>> Otherwise, LGTM.  Thanks.
>> 
>> OK, I've attached a new version of the patch.  Konrad, could you check
>> if with this patch the Emacs toolbar works as you expect?  Thanks.
> 
> Thanks, installed on the emacs-29 branch, and closing the bug.

[Message part 2 (text/html, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.