Project

General

Profile

Actions

Patch #776

closed

sort the journal of issues desc instead of asc

Added by Charles-Marie DE GRAEVE over 16 years ago. Updated over 16 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Issues
Target version:
-
Start date:
2008-03-03
Due date:
% Done:

0%

Estimated time:

Description

When displaying the journal of an issue, it is more pretty to have the last modifications first, instead of scrolling down the whole page.
Here is a simple modification to do :

diff -Naur trunk_6.3/app/controllers/issues_controller.rb eads_ver/app/controllers/issues_controller.rb
--- trunk_6.3/app/controllers/issues_controller.rb    2007-12-18 19:17:40.000000000 +0100
+++ eads_ver/app/controllers/issues_controller.rb    2008-03-03 13:51:20.000000000 +0100
@@ -81,7 +81,7 @@

   def show
     @custom_values = @issue.custom_values.find(:all, :include => :custom_field, :order => "#{CustomField.table_name}.position")
-    @journals = @issue.journals.find(:all, :include => [:user, :details], :order => "#{Journal.table_name}.created_on ASC")
+    @journals = @issue.journals.find(:all, :include => [:user, :details], :order => "#{Journal.table_name}.created_on DESC")
     @status_options = @issue.status.find_new_statuses_allowed_to(User.current.role_for_project(@project), @issue.tracker)
     respond_to do |format|
       format.html { render :template => 'issues/show.rhtml' }


Related issues

Related to Redmine - Feature #1848: Too much scrolling on ticketsClosed2008-09-03

Actions
Actions #1

Updated by John Goerzen over 16 years ago

I'm not sure I agree with that. I like having it with everything in chronological order. debbugs (the Debian BTS), RT, Trac all do it that way. If it's changed, perhaps it should be a configurable option instead of a hardcoded setting.

Actions #2

Updated by Eric Davis over 16 years ago

I agree with John, the journals should stay in ASC order. Otherwise you would have to read upwards to follow the train of thought.

-1

Actions #3

Updated by Thomas Lecavelier over 16 years ago

My preference is for ASC order too. But it's a preference, a user preference :) This should be a feature request that enable user to configure journal order sorting.

Actions #4

Updated by Jean-Philippe Lang over 16 years ago

  • Status changed from New to Closed

User preference added in r1197 to choose between chronologic or reverse chronologic order.
Default is still chronological order (ASC).

Actions

Also available in: Atom PDF