Edgewall Software
Modify

Opened 11 years ago

Last modified 9 months ago

#6128 new enhancement

[Patch] Pagination for the Timeline

Reported by: Naoki Takezoe Owned by:
Priority: low Milestone: next-major-releases
Component: timeline Version: 0.10.4
Severity: normal Keywords: patch pagination performance
Cc: trac-ja@…
Release Notes:
API Changes:

Description

Trac displays a lot of events as one page at the timeline. So sometimes it causes performance problems like the ticket report.

Attached patch provides the pagination for the Timeline (for 0.10.4).

Attachments (3)

timeline.zip (1.7 KB ) - added by Naoki Takezoe 11 years ago.
trac-0.11dev_r6303-timeline_paginate.patch (8.7 KB ) - added by trac-ja@… 10 years ago.
patch againsts source:trunk@6303
trac-0.11b1-timeline_paginate.patch (9.6 KB ) - added by trac-ja@… 10 years ago.
Patch againsts Trac-0.11b1

Download all attachments as: .zip

Change History (16)

Changed 11 years ago by Naoki Takezoe

Attachment: timeline.zip added

comment:1 Changed 11 years ago by sid

This is possibly a duplicate of #215.

comment:2 Changed 11 years ago by Naoki Takezoe

Hmm… r4681 seems to paginate by the daysback.

But in my project, there are quite a lot of events in one day. So I think it's better to paginate by number of events than daysback.

comment:3 Changed 11 years ago by Noah Kantrowitz

Resolution: duplicate
Status: newclosed

Old release branches are frozen, we only fix bugs, no new features.

comment:4 Changed 11 years ago by Naoki Takezoe

Sorry, I'll try to port my patch to trunk.

Changed 10 years ago by trac-ja@…

patch againsts source:trunk@6303

comment:5 Changed 10 years ago by trac-ja@…

Cc: trac-ja@… added
Resolution: duplicate
Status: closedreopened

I ported Takezoe's patch to trunk.

comment:6 Changed 10 years ago by Christian Boos

Milestone: 0.12
Priority: normalhigh

Looks good. Note the default_maxrows_rss options is still named 'default_maxrows'.

Changed 10 years ago by trac-ja@…

Patch againsts Trac-0.11b1

comment:7 in reply to:  6 Changed 10 years ago by trac-ja@…

Replying to cboos:

Looks good. Note the default_maxrows_rss options is still named 'default_maxrows'.

This patch has four options (two pairs of daysback and maxrows):

  • default_daysback
  • default_maxrows
  • default_daysback_rss
  • default_maxrows_rss

One pair is for HTML, and another is for RSS.

There should be separated, I think.

comment:8 Changed 8 years ago by Christian Boos

Keywords: pagination added
Priority: highlow

The patch probably needs to be refreshed for trunk.

comment:9 Changed 3 years ago by figaro

Keywords: performance added

comment:10 Changed 3 years ago by Ryan J Ollos

Owner: Christian Boos deleted
Status: reopenednew

comment:11 Changed 3 years ago by figaro

Summary: Timeline Paginate Patch[Patch] Pagination for the Timeline

comment:12 Changed 2 years ago by figaro

Keywords: patch added

comment:13 Changed 9 months ago by Ryan J Ollos

#8701 is related, requesting that the timeline show at least N entries, even if those entries are older than daysback.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.
The ticket will be disowned.
as The resolution will be set.
The owner will be changed from (none) to anonymous.

Add Comment


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