After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 698439 - Merge "Completed On" filter option with "Follow Up"
Merge "Completed On" filter option with "Follow Up"
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
3.6.x (obsolete)
Other Linux
: Normal minor
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2013-04-20 09:35 UTC by André Klapper
Modified: 2015-03-23 11:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
evo patch (6.31 KB, patch)
2015-03-13 11:27 UTC, Milan Crha
accepted-commit_after_freeze Details | Review

Description André Klapper 2013-04-20 09:35:07 UTC
"Edit > Message Filters > Add > Add Condition" offers "Completed On".
As parameter I expected a date selector, but it offers only "is set" and "is not set".

I *assume* it refers to the "Follow Up" item right before, but the "Message > Mark Up > Follow Up..." dialog only offers a "Completed" checkbox.
If the filter criterion refers to this flag, then it should maybe get renamed from "Completed On" to "Follow Up Completed", or even better somehow merged into "Follow Up" which currently only offers "is Flagged" and "is not Flagged" (yes, with that capitalization). If not merging it, maybe rename "Follow Up" to "Follow Up Flag" to better differentiate?

After clarifying help/C/mail-filters-conditions.page needs an update.
Comment 1 André Klapper 2013-04-20 09:59:03 UTC
I moved the message in the docs to xinclude-filter-vfolder-conditions.xml
Comment 2 Matthew Barnes 2013-04-20 13:01:54 UTC
I think I'd actually merge both into the "Status" condition.  So you'd have:

  [Status]  [is    ]  [Replied to          ]
            [is not]  [Draft               ]
                      [Imporant            ]
                      [Read                ]
                      [Junk                ]
                      [Flagged for Followup]  (new)
                      [Followup Completed  ]  (new)

I think that would cover all the variations, right?

Need to research this a bit more to make sure I'm not missing the rationale for the "Follow Up" / "Completed On" conditions being split out.
Comment 3 Matthew Barnes 2013-04-20 13:05:20 UTC
Of course, migrating existing filters could be fairly painful if we decide to rearrange conditions like this.
Comment 4 Milan Crha 2015-03-13 10:31:11 UTC
The "Completed On" is a date, when the follow-up flag was completed, but the filter is there to search for completed and uncompleted follow-ups only. I would not merge the two under the Status, I'm afraid people might not find it there. Let's rename the two followup options instead.
Comment 5 Milan Crha 2015-03-13 11:27:24 UTC
Created attachment 299302 [details] [review]
evo patch

for evolution;

This merges "Completed On" with "Follow Up" criterion making it options:
   is Flagged
   is not Flagged
   is Completed
   is not Completed
(really this capitalization).

Feel free to complain before I'll commit for 3.17.1.
Comment 6 Milan Crha 2015-03-23 11:12:13 UTC
Created commit 9774499 in evo master (3.17.1+)