716551 - Add List-Id to the default list of field the filter UI provides

Status: NEW

(MailNews Core :: Filters, enhancement)

Description

[Ludovic Hirlimann [:Usul]]

I would like to add the List-Id header to the default list of headers in the filter UI.

I believe List-Id is set by default on mailman run mailing lists , which means a good proportion of the mailing lists available.

That would help all new installs that would not have to add the List-Id via the customize submenu.

Blake thoughts ?

Comment 1

[:aceman]

I could try to implement this if UI people confirm this would be acceptable.

But this needs specification what values this header can contain and which matching operators should be allowed on it (like 'is', 'isnt', 'contains', etc.)

Comment 2

[Blake Winton (:bwinton) (:☕️)]

Yeah, I'ld be up for that, assuming you're talking about the Message Filters dialog, and not the Quick Filter Bar.  ;)

Aceman, Ludo, I'll let the two of you decide who'll spec it out.

Thanks,
Blake.

Comment 3

[Ludovic Hirlimann [:Usul]]

(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #2)
> Yeah, I'ld be up for that, assuming you're talking about the Message Filters
> dialog, and not the Quick Filter Bar.  ;)

We are.
 
> Aceman, Ludo, I'll let the two of you decide who'll spec it out.

I'll spec it here then I think {contains, doesn't contain} is enough. Once we got that we could implement more based on the feedback from our users.

Comment 4

[Kent James (:rkent)]

(In reply to Ludovic Hirlimann [:Usul] from comment #3)

> I'll spec it here then I think {contains, doesn't contain} is enough.

The amount of effort to add the other standard items (Is, Isnt, Begins With, Ends With) is trivial compared to the effort required to do another bug. I think you would need to have a good motivation why NOT to include these.

I don't know what the spec is for a List-ID, but the ones that I see look like email addresses. You'll need to spec whether this should be parsed like an email address or not. IIRC, for email addresses "List-ID" "Is" "tb-enterprise.mozilla.org" would work, but it would not if this is not parsed like an email address.

Comment 5

[Magnus Melin [:mkmelin]]

Agreed, no point in not adding all the standard items.

Comment 6

[Karsten Düsterloh]

List-Id is basically a subset of email addresses if you look at RfC 2919, and thus should be treated as such IMO. No need to make arbitrary restrictions now that need to be fixed later.

Comment 7

[Ludovic Hirlimann [:Usul]]

:aceman you have you specs.

Comment 8

[:aceman]

Yes, thanks. I'll look how hard this is, if it is possible to reuse code parsing email addresses in filters, etc.

Comment 9

[Kent James (:rkent)]

Another issue to consider: presumably many people are already using list id as a custom header. When a new field is added, how will that interact with those existing custom headers? That is, will it appear twice in the field list, or once and overwrite the old header? If it appears twice, then that will be confusing to existing users. Will it mess up existing filters that use that header?

Comment 10

[:aceman]

Thanks for the hint. I can test that if when it is implemented.

Comment 13

[Wayne Mery (:wsmwk)]

success of this may depend on bug 622307.

Comment 14

[Ludovic Hirlimann [:Usul]]

ping ?

Comment 15

[Ludovic Hirlimann [:Usul]]

Wayne, Magnus, Ace, this is a quite easy addition to defaults that would make managing mailing list way easier in TB, can we get the ball rolling?

Comment 16

[sgougeon]

ping
Why Severity=S3 while bug 622307 prevents using required filters on selected messages?
What would be the work-around? I was told about trying the external module QuickFilters ; but it looks extremely intrusive (sure far less than an IA agent...), so finally not installed.
Severity S2 would be more fair.