GNU bug report logs - #46257
28.0.50; use flat-button in modus-operandi theme

Previous Next

Package: emacs;

Reported by: Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com>

Date: Tue, 2 Feb 2021 11:35:01 UTC

Severity: normal

Tags: wontfix

Found in version 28.0.50

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

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

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

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#46257; Package emacs. (Tue, 02 Feb 2021 11:35:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Tue, 02 Feb 2021 11:35:01 GMT) Full text and rfc822 format available.

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

From: Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 28.0.50; use flat-button in modus-operandi theme
Date: Tue, 2 Feb 2021 12:34:06 +0100
[Message part 1 (text/plain, inline)]
This diff provides an example for the use of the :style flat-button in the
tab-line
of the modus-operandi theme. I've kept the original code commented.

Best, /Pedro A. Aranda


-- 
Fragen sind nicht da um beantwortet zu werden,
Fragen sind da um gestellt zu werden
Georg Kreisler
[Message part 2 (text/html, inline)]
[flat-button-example.diff (text/x-patch, attachment)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#46257; Package emacs. (Tue, 02 Feb 2021 13:25:02 GMT) Full text and rfc822 format available.

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

From: "Basil L. Contovounesios" <contovob <at> tcd.ie>
To: Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com>
Cc: Protesilaos Stavrou <info <at> protesilaos.com>, 46257 <at> debbugs.gnu.org
Subject: Re: bug#46257: 28.0.50; use flat-button in modus-operandi theme
Date: Tue, 02 Feb 2021 13:24:08 +0000
Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com> writes:

> This diff provides an example for the use of the :style flat-button in the tab-line
> of the modus-operandi theme. I've kept the original code commented.

Thanks, CCing Protesilao.

-- 
Basil




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#46257; Package emacs. (Tue, 02 Feb 2021 18:47:01 GMT) Full text and rfc822 format available.

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

From: Protesilaos Stavrou <info <at> protesilaos.com>
To: Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com>
Cc: "Basil L. Contovounesios" <contovob <at> tcd.ie>, 46257 <at> debbugs.gnu.org
Subject: Re: bug#46257: 28.0.50; use flat-button in modus-operandi theme
Date: Tue, 02 Feb 2021 20:46:42 +0200
On 2021-02-02, 13:24 +0000, "Basil L. Contovounesios" <contovob <at> tcd.ie> wrote:

> Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com> writes:
>
>> This diff provides an example for the use of the :style flat-button in the tab-line
>> of the modus-operandi theme. I've kept the original code commented.
>
> Thanks, CCing Protesilao.

Thanks Basil!

Hello Pedro!

That change looks good to me.  Please apply it wherever it would be
appropriate.  And please do the same for the modus-vivendi theme.

Now a question that is not pertinent to Emacs 28: does this flat-button
degrade gracefully on older versions, say, 27?  The reason I am asking
is because I will need to mirror any patch on my upstream repo; and I
know that there are users who run the themes on older versions of Emacs.

All the best,
Protesilaos or "Prot"

P.S. The version of the themes in Emacs is 0.13.0 while I currently have
them at 1.1.1.  This is due to change at some point, per bug#45068:
<https://debbugs.gnu.org/cgi/bugreport.cgi?bug=45068>.

-- 
Protesilaos Stavrou
protesilaos.com




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#46257; Package emacs. (Tue, 02 Feb 2021 19:22:01 GMT) Full text and rfc822 format available.

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

From: Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com>
To: Protesilaos Stavrou <info <at> protesilaos.com>
Cc: "Basil L. Contovounesios" <contovob <at> tcd.ie>, 46257 <at> debbugs.gnu.org
Subject: Re: bug#46257: 28.0.50; use flat-button in modus-operandi theme
Date: Tue, 2 Feb 2021 20:21:09 +0100
[Message part 1 (text/plain, inline)]
Hi Protesilaos,

the flat-button style is part of emacs28

Best, /PA

On Tue, 2 Feb 2021 at 19:46, Protesilaos Stavrou <info <at> protesilaos.com>
wrote:

> On 2021-02-02, 13:24 +0000, "Basil L. Contovounesios" <contovob <at> tcd.ie>
> wrote:
>
> > Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com> writes:
> >
> >> This diff provides an example for the use of the :style flat-button in
> the tab-line
> >> of the modus-operandi theme. I've kept the original code commented.
> >
> > Thanks, CCing Protesilao.
>
> Thanks Basil!
>
> Hello Pedro!
>
> That change looks good to me.  Please apply it wherever it would be
> appropriate.  And please do the same for the modus-vivendi theme.
>
> Now a question that is not pertinent to Emacs 28: does this flat-button
> degrade gracefully on older versions, say, 27?  The reason I am asking
> is because I will need to mirror any patch on my upstream repo; and I
> know that there are users who run the themes on older versions of Emacs.
>
> All the best,
> Protesilaos or "Prot"
>
> P.S. The version of the themes in Emacs is 0.13.0 while I currently have
> them at 1.1.1.  This is due to change at some point, per bug#45068:
> <https://debbugs.gnu.org/cgi/bugreport.cgi?bug=45068>.
>
> --
> Protesilaos Stavrou
> protesilaos.com
>


-- 
Fragen sind nicht da um beantwortet zu werden,
Fragen sind da um gestellt zu werden
Georg Kreisler
[Message part 2 (text/html, inline)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#46257; Package emacs. (Wed, 03 Feb 2021 17:47:02 GMT) Full text and rfc822 format available.

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

From: Juri Linkov <juri <at> linkov.net>
To: Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com>
Cc: 46257 <at> debbugs.gnu.org
Subject: Re: bug#46257: 28.0.50; use flat-button in modus-operandi theme
Date: Wed, 03 Feb 2021 19:22:24 +0200
> This diff provides an example for the use of the :style flat-button in the
> tab-line

BTW, there are two possible contrasted styles for tab-line:

1. the active tab is lighter than inactive tabs;
2. the active tab is darker than inactive tabs.

For example, some desktop environments use the second style,
but currently Emacs tabs by default use the first style.

I'm not sure which one is better.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#46257; Package emacs. (Thu, 04 Feb 2021 08:41:02 GMT) Full text and rfc822 format available.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Protesilaos Stavrou <info <at> protesilaos.com>
Cc: "Basil L. Contovounesios" <contovob <at> tcd.ie>, 46257 <at> debbugs.gnu.org,
 Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com>
Subject: Re: bug#46257: 28.0.50; use flat-button in modus-operandi theme
Date: Thu, 04 Feb 2021 09:40:10 +0100
Protesilaos Stavrou <info <at> protesilaos.com> writes:

> That change looks good to me.  Please apply it wherever it would be
> appropriate.  And please do the same for the modus-vivendi theme.
>
> Now a question that is not pertinent to Emacs 28: does this flat-button
> degrade gracefully on older versions, say, 27?  The reason I am asking
> is because I will need to mirror any patch on my upstream repo; and I
> know that there are users who run the themes on older versions of Emacs.

It won't degrade gracefully, so will applying the patch to Emacs 28
create problems for you?  If so, applying it doesn't seem all that
vital, since the visual result is the same (I think?), and it's just a
simplification (in Emacs 28).

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#46257; Package emacs. (Thu, 04 Feb 2021 12:11:02 GMT) Full text and rfc822 format available.

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

From: Protesilaos Stavrou <info <at> protesilaos.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: "Basil L. Contovounesios" <contovob <at> tcd.ie>,
 Protesilaos Stavrou <info <at> protesilaos.com>, 46257 <at> debbugs.gnu.org,
 Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com>
Subject: Re: bug#46257: 28.0.50; use flat-button in modus-operandi theme
Date: Thu, 04 Feb 2021 14:09:56 +0200
On 2021-02-04, 09:40 +0100, Lars Ingebrigtsen <larsi <at> gnus.org> wrote:

> Protesilaos Stavrou <info <at> protesilaos.com> writes:
>
>> That change looks good to me.  Please apply it wherever it would be
>> appropriate.  And please do the same for the modus-vivendi theme.
>>
>> Now a question that is not pertinent to Emacs 28: does this flat-button
>> degrade gracefully on older versions, say, 27?  The reason I am asking
>> is because I will need to mirror any patch on my upstream repo; and I
>> know that there are users who run the themes on older versions of Emacs.
>
> It won't degrade gracefully, so will applying the patch to Emacs 28
> create problems for you?  If so, applying it doesn't seem all that
> vital, since the visual result is the same (I think?), and it's just a
> simplification (in Emacs 28).

In that case I would prefer not to introduce the patch, even though I
understand it would otherwise align the themes with the latest state of
emacs.git.

That granted, the pattern of hiding the :box border by colourising it
the same as the background is uncommon.  So if you still wish to apply
the flat-button, I will just maintain separate patches for whenever I
submit a new version of the themes to emacs.git.

-- 
Protesilaos Stavrou
protesilaos.com




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#46257; Package emacs. (Thu, 04 Feb 2021 16:17:02 GMT) Full text and rfc822 format available.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Protesilaos Stavrou <info <at> protesilaos.com>
Cc: "Basil L. Contovounesios" <contovob <at> tcd.ie>, 46257 <at> debbugs.gnu.org,
 Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com>
Subject: Re: bug#46257: 28.0.50; use flat-button in modus-operandi theme
Date: Thu, 04 Feb 2021 17:16:40 +0100
Protesilaos Stavrou <info <at> protesilaos.com> writes:

> In that case I would prefer not to introduce the patch, even though I
> understand it would otherwise align the themes with the latest state of
> emacs.git.
>
> That granted, the pattern of hiding the :box border by colourising it
> the same as the background is uncommon.  So if you still wish to apply
> the flat-button, I will just maintain separate patches for whenever I
> submit a new version of the themes to emacs.git.

I think that sounds like too much work -- it's best for if these
packages/themes that are meant to work over a range of Emacs versions if
they avoid using cutting-edge features, I think.

So I'm closing this bug report, and not applying the patch.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no




Added tag(s) wontfix. Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Thu, 04 Feb 2021 16:17:02 GMT) Full text and rfc822 format available.

bug closed, send any further explanations to 46257 <at> debbugs.gnu.org and Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com> Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Thu, 04 Feb 2021 16:17:03 GMT) Full text and rfc822 format available.

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#46257; Package emacs. (Sat, 06 Feb 2021 10:34:02 GMT) Full text and rfc822 format available.

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

From: "Basil L. Contovounesios" <contovob <at> tcd.ie>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: Protesilaos Stavrou <info <at> protesilaos.com>, 46257 <at> debbugs.gnu.org,
 Pedro Andres Aranda Gutierrez <paaguti <at> gmail.com>
Subject: Re: bug#46257: 28.0.50; use flat-button in modus-operandi theme
Date: Sat, 06 Feb 2021 10:33:05 +0000
Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> Protesilaos Stavrou <info <at> protesilaos.com> writes:
>
>> In that case I would prefer not to introduce the patch, even though I
>> understand it would otherwise align the themes with the latest state of
>> emacs.git.
>>
>> That granted, the pattern of hiding the :box border by colourising it
>> the same as the background is uncommon.  So if you still wish to apply
>> the flat-button, I will just maintain separate patches for whenever I
>> submit a new version of the themes to emacs.git.
>
> I think that sounds like too much work -- it's best for if these
> packages/themes that are meant to work over a range of Emacs versions if
> they avoid using cutting-edge features, I think.

Alternatively, when Elisp wants to work over a range of Emacs versions,
it can test for present features or the current Emacs version, and
expand to the appropriate form, as is done for the :extend keyword, for
example.

-- 
Basil




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Sat, 06 Mar 2021 12:24:06 GMT) Full text and rfc822 format available.

This bug report was last modified 3 years and 49 days ago.

Previous Next


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