Patch #776
closed
sort the journal of issues desc instead of asc
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
Updated by John Goerzen almost 18 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.
Updated by Eric Davis almost 18 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
Updated by Thomas Lecavelier almost 18 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.
Updated by Jean-Philippe Lang almost 18 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).