[#844] Allow reversing the display order of comments in trackers

Description

Summary:

Detailed description

The following is a matter of taste.

Fusionforge displays tracker messages as: original message first, then comments in reverse chronological order.

This is particularly annoying in e-mail notifications, where the most recent stuff (which is potentially a short sentence) is hidden between of the initial post (which is possibly long), and the history (which is possibly long, too).

I understand that people get used to it, and breaking habits is not good.

But overall, personally, I hate it, and so do my coworkers. And I'm damn sure it's been so ever since I started using fusionforge.

Would there be a way to make that display order configurable ? In particular for the notification e-mails ?

In fact, for notification e-mails, having only the most recent text would be much better. (and proper In-Reply-To headers, if at all possible)

Best,

E.

General Information
Submitted by:
Emmanuel Thomé
Date Submitted: 2017-01-03 22:20
Last Modified by: Nobody
Last Modified: 2017-11-02 20:00
Permalink: https://fusionforge.org/tracker/a_follow.php/844
Actions
Monitor
Votes: 0/1 (0%)
Internal Fields
Data Type: Feature requests
Assigned to: Nobody (None)
State: Open
Priority: 1
Extra Fields
Resolution:
none
Difficulty:
none
Target release:
none
Follow-up tabs
Message  ↓
Date: 2017-01-05 13:26
Sender: Emmanuel Thomé

> In the "Comments" Tab, Close to "Message", there are 2 arrows. You can click on its to change the order.

Ah, ok. I had never noticed it. Thanks.

The thing is that the actual html page is clearly not the worst offender. To start with there's a different formatting for the initial post and the comments, and on the other hand as you mentioned, this arrow is there right in place (maybe that could be made more conspicuous, but I'm not even sure). And yes, as far as the html page goes, per-user is fine.

The problem is more with e-mail. Unless I'm mistaken, the trackermessagesorder will only act on the html rendering, not on the text rendering which goes in the e-mail. And that one is the one I'm most annoyed with.

Date: 2017-01-05 13:14
Sender: Franck Villaume

Ordering comments: In the "Comments" Tab, Close to "Message", there are 2 arrows. You can click on its to change the order.

About the code: Extract from ArtifactHTML.class.php: function showMessages() { global $HTML; $return = ''; if (sessionloggedin()) { $u = sessiongetuser(); $order = $u->getPreference('trackermessages_order'); } if (!isset($order) || !$order) $order = 'up';

             $result = $this->getMessages($order);

So, there is an user preference about comment ordering, ...., ...., and I just test it again and it works!!! The preference is per user, no preference at tracker level. But your comments answer partially to my question: you need a preference at tracker level then user could overwrite it? or would the per user approach be enough?

I will take a look at sample mbox you provide.

Date: 2017-01-05 12:59
Sender: Emmanuel Thomé

Here are two files in mbox format.

The first (mbox) is the one I got on my previous comment.

The second (mbox2) is edited by me so that it has only the latest comment, plus an In-Reply-To header.

Date: 2017-01-05 12:52
Sender: Emmanuel Thomé

> Today, FusionForge offers the possibility to change the order of comments

Really ? How ? I couldn't find that on the tracker admin tab.

> However the backend code does not allow to store the selected order as preference. (there is some code but looks buggy)

Hmm, that's barely useful then.

Date: 2017-01-05 11:03
Sender: Franck Villaume

Hi,

Could you provide some sample of email notification format you would like to get? How would you like to get the preference set-up?

Today, FusionForge offers the possibility to change the order of comments (chronological order or reverse) However the backend code does not allow to store the selected order as preference. (there is some code but looks buggy) What if the code was fixed to store the selected order as preference and then used to build the notification email, would it suit you?

Attachments:
Size Name Date By Download
14 KiBmbox2017-01-05 12:59
Emmanuel Thomé
mbox
11 KiBmbox22017-01-05 12:59
Emmanuel Thomé
mbox2

No related commits.

Field Old Value Date By
priority32018-04-22 08:59
Franck Villaume
File Added522: mbox2017-01-05 12:59
Emmanuel Thomé
File Added523: mbox22017-01-05 12:59
Emmanuel Thomé

No relations found.