emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#62234: closed (Toolbar does not work properly (with Emacs 29 or 30))


From: GNU bug Tracking System
Subject: bug#62234: closed (Toolbar does not work properly (with Emacs 29 or 30))
Date: Thu, 23 Mar 2023 08:11:02 +0000

Your message dated Thu, 23 Mar 2023 10:10:10 +0200
with message-id <838rfn6g7x.fsf@gnu.org>
and subject line Re: bug#62234: Toolbar does not work properly (with Emacs 29 
or 30)
has caused the debbugs.gnu.org bug report #62234,
regarding Toolbar does not work properly (with Emacs 29 or 30)
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
62234: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=62234
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
--- Begin Message --- 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



--- End Message ---
--- Begin Message --- 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@yahoo.es>
> Cc: Eli Zaretskii <eliz@gnu.org>,  62234@debbugs.gnu.org,  Konrad Podczeck
>  <konrad.podczeck@univie.ac.at>
> Date: Sat, 18 Mar 2023 11:25:51 +0100
> 
> Po Lu <luangruo@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.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]