Comment permalinks in notification emails should include pagination
|Reported by:||eatingrules||Owned by:||nacin|
[This is my first time submitting a trac ticket - I apologize if I'm not doing it correctly. I searched the support forms (and posted a week ago but got no response), and have searched the trac for existing tickets, but didn't turn up anything that matched. Thanks!]
In Settings->Discussion, we have "Break Comments into pages with 25 top level comments per page and the last page displayed by default. We also have comments should be displayed with the older comments at the top of each page.
When a comment is first left by the reader, there may already be several pages of comments. The comment initially will appear on the default page (because we have the last page displayed by default), so the pagination is not necessary in the permalink:
However, by the time our moderator clicks the link, there have usually been several more pages of comments - so the anchor #comment-12345 doesn't do any good (because the original comment is no longer on the most recent page). She then has to page through the comments manually to find the comment, which is time-consuming and frustrating.
It would be far preferable for the permalink in the email notification to include the fully paginated link:
Change History (8)
- Keywords has-patch added
- Milestone changed from Awaiting Review to 3.9