Bug 4095 - Provide visual indication for filter rules with move/stop action
Summary: Provide visual indication for filter rules with move/stop action
Status: NEW
Alias: None
Product: Claws Mail (GTK 2)
Classification: Unclassified
Component: Filtering (show other bugs)
Version: 3.17.2
Hardware: PC Linux
: P3 enhancement
Assignee: users
URL:
Depends on:
Blocks:
 
Reported: 2018-09-22 10:51 UTC by George
Modified: 2018-09-22 21:03 UTC (History)
0 users

See Also:


Attachments

Description George 2018-09-22 10:51:40 UTC
Considering that Claws Mail stops processing next filter rules if there is a match for a rule with a move/stop action, it would be useful to have visual indication for those particular rules (and/or for those which don't move/stop). For example a different text color/background/icon for those rules. This would be helpful to assign correct order of rules easier.
Comment 1 Shai Berger 2018-09-22 18:52:28 UTC
The code's term is "final action". There are currently only three of them -- STOP & DELETE, for which "no more processing" seems obvious, and MOVE. Up until 3.16 MARK_AS_SPAM was also a final action, but this has changed in 3.17.

This is defined in a function named "filtering_is_final_action()" in the file filtering.c.

Given that the set of non-obviously-final actions includes only one action, I think that adding code to visually mark final actions differently is a bit of an overkill. That said, AFAIK the whole issue is not documented anywhere, and adding this documentation might be useful.
Comment 2 George 2018-09-22 19:33:50 UTC
> I think that adding code to visually mark final actions differently is a bit of an overkill.

I don't see why. To me it is an overkill to look at a long line of text (for more complex rules) in which even the action is not visually separated from the condition. A good UI combines function with visuals and currently nothing gives a visual hint about what is done and what is the effect of it (in the context of this whole suggestion)
Comment 3 Andrej Kacian 2018-09-22 20:49:40 UTC
I guess a small red "X" icon, or something similar, next to the rule name could work, especially with a tooltip that says what it means.
Comment 4 George 2018-09-22 21:03:39 UTC
Sounds good.

Another thing (in the context of my previous reply): It would be good also to have a visual separation between the action and the condition on the line of the rule. For example: the text of the action can be different color (or bold), e.g.

from matchcase "watever@example.com" [*style*]move "...somewhere"[/*style*]

Perhaps this should be in a separate bug. Please let me know if I should file one or it can be done at once for both things.

Note You need to log in before you can comment on or make changes to this bug.