Edgewall Software

Opened 3 years ago

Closed 3 years ago

#12381 closed defect (fixed)

Auto-preview in ticket view restores "Oldest first" of changelog

Reported by: Jun Omae Owned by: Jun Omae
Priority: normal Milestone: 1.0.11
Component: ticket system Version:
Severity: normal Keywords:
Release Notes:

Fix resetting Oldest first after auto-preview of ticket form.

API Changes:

Description (last modified by Jun Omae)

How to reproduce:

  1. Visit a ticket and select Newest first or Threaded in prefs of changelog.
  2. Type in textarea for ticket comment.
  3. Wait for the auto-preview
  4. Oldest first is selected and order of ticket comments is changed.
  5. XMLHttpRequest with ticket_comments_order:"oldest" is sent twice.

I think we should keep the order of ticket comments even if auto-preview is fired.

Attachments (0)

Change History (6)

comment:1 Changed 3 years ago by Jun Omae

Description: modified (diff)

comment:2 Changed 3 years ago by Jun Omae

Milestone: next-stable-1.0.x1.0.11
Owner: set to Jun Omae
Status: newassigned

Proposed changes in log:jomae.git@t12381, which added applyCommentsOrder() in threaded_comments.js.

comment:3 Changed 3 years ago by Ryan J Ollos

Changes look good. Test with Chrome 48 and Opera 35 on OSX 10.11.3.

comment:4 Changed 3 years ago by Jun Omae

Release Notes: modified (diff)

Thanks for the review! Committed in [14598]. However, merging the changes to trunk is conflicted. I'll rework for trunk.

comment:5 Changed 3 years ago by Jun Omae

Proposed changes for trunk in jomae.git@t12381.trunk ([255a864c1/jomae.git]).

comment:6 Changed 3 years ago by Jun Omae

Release Notes: modified (diff)
Resolution: fixed
Status: assignedclosed

Refactored threaded_comments.js for 1.0-stable in [14681] and committed in [14682].

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain Jun Omae.
The resolution will be deleted.
to The owner will be changed from Jun Omae to the specified user.

Add Comment

E-mail address and name can be saved in the Preferences .
Note: See TracTickets for help on using tickets.