GNU bug report logs - #78305
[PATCH] Add a new user option to hide the header of old feeds in newsticker

Previous Next

To reply to this bug, email your comments to 78305 AT debbugs.gnu.org.

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

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

From: Baptiste Strazzulla <baptiste.strazzulla <at> protonmail.com>
To: "bug-gnu-emacs <at> gnu.org" <bug-gnu-emacs <at> gnu.org>
Subject: [PATCH] Add a new user option to hide the header of old feeds in
 newsticker
Date: Wed, 07 May 2025 22:48:19 +0000

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

Hello!

Here is a patch to hide the header of feeds without any new or immortal items in the plainview newsticker buffer. My use-case is that I'm subscribed to a lot of low-key feeds that are not updated often, so most of the time, the visible part of a fresh plainview buffer is just a list of empty feeds. This patch adds a new user option `newsticker-hide-old-feed-header-in-newsticker-buffer` which when set to `true` hides them, only leaving feeds with new items visible.

What do you think?

Regards,
Baptiste Strazzulla

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

[0001-Use-the-already-defined-age-variable.patch (application/octet-stream, attachment)]

[0002-Add-a-new-user-option-to-hide-the-header-of-old-feed.patch (application/octet-stream, attachment)]

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

From: Ulf Jasper <ulf.jasper <at> web.de>
To: Baptiste Strazzulla via "Bug reports for GNU Emacs, the Swiss army knife
 of text editors" <bug-gnu-emacs <at> gnu.org>
Cc: Baptiste Strazzulla <baptiste.strazzulla <at> protonmail.com>,
 78305 <at> debbugs.gnu.org
Subject: Re: bug#78305: [PATCH] Add a new user option to hide the header of
 old feeds in newsticker
Date: Mon, 02 Jun 2025 17:49:17 +0200

Am 07.05.2025 um 22:48 (+0000) schrieb Baptiste Strazzulla via "Bug
reports for GNU Emacs, the Swiss army knife of text editors":
> What do you think?

I think the patches are OK. We should apply them, I'd say.
I am not sure whether the changes require copyright assignment or they
are "trivial".  Besides I am out of practice regarding committing other
people's patches.  Can someone help out?

ulf

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

From: Robert Pluim <rpluim <at> gmail.com>
To: 78305 <at> debbugs.gnu.org
Cc: Ulf Jasper <ulf.jasper <at> web.de>, baptiste.strazzulla <at> protonmail.com
Subject: Re: bug#78305: [PATCH] Add a new user option to hide the header of
 old feeds in newsticker
Date: Mon, 02 Jun 2025 17:57:47 +0200

>>>>> On Mon, 02 Jun 2025 17:49:17 +0200, Ulf Jasper via "Bug reports for GNU Emacs, the Swiss army knife of text editors" <bug-gnu-emacs <at> gnu.org> said:

    Ulf> Am 07.05.2025 um 22:48 (+0000) schrieb Baptiste Strazzulla via "Bug
    Ulf> reports for GNU Emacs, the Swiss army knife of text editors":
    >> What do you think?

    Ulf> I think the patches are OK. We should apply them, I'd say.
    Ulf> I am not sure whether the changes require copyright assignment or they
    Ulf> are "trivial".  Besides I am out of practice regarding committing other
    Ulf> people's patches.  Can someone help out?

I donʼt know about copyright assignment, but could the user option be
something shorter than
`newsticker-hide-old-feed-header-in-newsticker-buffer'? Would
`newsticker-hide-old-feed-header' not be enough?

Robert
-- 

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ulf Jasper <ulf.jasper <at> web.de>, baptiste.strazzulla <at> protonmail.com
Cc: 78305 <at> debbugs.gnu.org
Subject: Re: bug#78305: [PATCH] Add a new user option to hide the header of old
 feeds in newsticker
Date: Mon, 02 Jun 2025 19:30:41 +0300

> Cc: baptiste.strazzulla <at> protonmail.com
> Date: Mon, 02 Jun 2025 17:49:17 +0200
> From:  Ulf Jasper via "Bug reports for GNU Emacs,
>  the Swiss army knife of text editors" <bug-gnu-emacs <at> gnu.org>
> 
> Am 07.05.2025 um 22:48 (+0000) schrieb Baptiste Strazzulla via "Bug
> reports for GNU Emacs, the Swiss army knife of text editors":
> > What do you think?
> 
> I think the patches are OK. We should apply them, I'd say.
> I am not sure whether the changes require copyright assignment or they
> are "trivial".  Besides I am out of practice regarding committing other
> people's patches.  Can someone help out?

Accepting this does require copyright assignment.

Baptiste, would you like to start your copyright assignment paperwork
at this time?  If yes, i will send you the form to fill and the
instructions to go with it.

Thanks.

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

From: Ulf Jasper <ulf.jasper <at> web.de>
To: Robert Pluim <rpluim <at> gmail.com>
Cc: baptiste.strazzulla <at> protonmail.com, 78305 <at> debbugs.gnu.org
Subject: Re: bug#78305: [PATCH] Add a new user option to hide the header of
 old feeds in newsticker
Date: Mon, 02 Jun 2025 22:17:14 +0200

Am 02.06.2025 um 17:57 (+0200) schrieb Robert Pluim:
> I donʼt know about copyright assignment, but could the user option be
> something shorter than
> `newsticker-hide-old-feed-header-in-newsticker-buffer'? Would
> `newsticker-hide-old-feed-header' not be enough?

Agree.  I think that would be enough.

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

From: Baptiste Strazzulla <baptiste.strazzulla <at> protonmail.com>
To: Ulf Jasper <ulf.jasper <at> web.de>
Cc: Robert Pluim <rpluim <at> gmail.com>, 78305 <at> debbugs.gnu.org
Subject: Re: bug#78305: [PATCH] Add a new user option to hide the header of
 old feeds in newsticker
Date: Tue, 03 Jun 2025 09:50:47 +0000

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

> Agree. I think that would be enough.

I updated the second patch.

> Baptiste, would you like to start your copyright assignment paperwork at this time? If yes, i will send you the form to fill and the instructions to go with it.

I started the process the day I sent the patches. There's already been a back and forth of emails the other week. I can ping back here when it's completed.

Cool!

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

[0001-Use-the-already-defined-age-variable.patch (application/octet-stream, attachment)]

[0002-Add-a-new-user-option-to-hide-the-header-of-old-feed.patch (application/octet-stream, attachment)]

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

From: Robert Pluim <rpluim <at> gmail.com>
To: Baptiste Strazzulla <baptiste.strazzulla <at> protonmail.com>
Cc: Ulf Jasper <ulf.jasper <at> web.de>, 78305 <at> debbugs.gnu.org
Subject: Re: bug#78305: [PATCH] Add a new user option to hide the header of
 old feeds in newsticker
Date: Tue, 03 Jun 2025 12:17:10 +0200

>>>>> On Tue, 03 Jun 2025 09:50:47 +0000, Baptiste Strazzulla <baptiste.strazzulla <at> protonmail.com> said:

    >> Agree. I think that would be enough.
    Baptiste> I updated the second patch.

    >> Baptiste, would you like to start your copyright assignment
    >> paperwork at this time? If yes, i will send you the form to fill and
    >> the instructions to go with it.

    Baptiste> I started the process the day I sent the patches. There's already been
    Baptiste> a back and forth of emails the other week. I can ping back here when
    Baptiste> it's completed.

Just two comments:

- can you wrap your ChangeLog entries? See CONTRIBUTE for some Emacs
  functionality that can help you with that
- could you put (Bug#78305) somewhere in the commit message, it helps
  us with tracking

Thanks

Robert
-- 

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

From: Baptiste Strazzulla <baptiste.strazzulla <at> protonmail.com>
To: Robert Pluim <rpluim <at> gmail.com>
Cc: Ulf Jasper <ulf.jasper <at> web.de>, 78305 <at> debbugs.gnu.org
Subject: Re: bug#78305: [PATCH] Add a new user option to hide the header of
 old feeds in newsticker
Date: Tue, 03 Jun 2025 13:40:10 +0000

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

> Just two comments:
>
> - can you wrap your ChangeLog entries? See CONTRIBUTE for some Emacs
> functionality that can help you with that
> - could you put (Bug#78305) somewhere in the commit message, it helps
> us with tracking

I filled the ChangeLog entries and added the bug number.

On Tuesday, June 3rd, 2025 at 12:17 PM, Robert Pluim rpluim <at> gmail.com wrote:

>>>>>> On Tue, 03 Jun 2025 09:50:47 +0000, Baptiste Strazzulla baptiste.strazzulla <at> protonmail.com said:
>
>>> Agree. I think that would be enough.
>
> Baptiste> I updated the second patch.
>
>>> Baptiste, would you like to start your copyright assignment
>
>>> paperwork at this time? If yes, i will send you the form to fill and
>
>>> the instructions to go with it.
>
> Baptiste> I started the process the day I sent the patches. There's already been
>
> Baptiste> a back and forth of emails the other week. I can ping back here when
>
> Baptiste> it's completed.
>
> Just two comments:
>
> - can you wrap your ChangeLog entries? See CONTRIBUTE for some Emacs
> functionality that can help you with that
> - could you put (Bug#78305) somewhere in the commit message, it helps
> us with tracking
>
> Thanks
>
> Robert
> --

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

[0002-Add-a-new-user-option-to-hide-the-header-of-old-feed.patch (application/octet-stream, attachment)]

[0001-Use-the-already-defined-age-variable.patch (application/octet-stream, attachment)]

Previous Next

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