Make WordPress Core

Opened 15 years ago

Closed 14 years ago

#12797 closed enhancement (fixed)

Add comment permalink to plain-text email notification

Reported by: lancewillett's profile lancewillett Owned by:
Milestone: 3.1 Priority: normal
Severity: normal Version: 3.0
Component: Comments Keywords: has-patch
Focuses: Cc:

Description

In comment notification emails there isn't a full link to the actual comment. If a post has a large number of comments, having a permalink with anchor ID to jump to the exact comment would be a nice improvement.

Currently, the email provides a link to the main comment anchor only:
http://example.com/2010/02/03/hot-air/#comments

The improved link would be to the exact comment:
http://example.com/2010/02/03/hot-air/#comment-6

Attachments (3)

12797-patch.diff (637 bytes) - added by lancewillett 15 years ago.
Add comment permalink to author notification email
t12797-string-space.diff (812 bytes) - added by demetris 14 years ago.
Removes space before colon
12797.diff (807 bytes) - added by johnpbloch 14 years ago.
Refreshing demetris' patch

Download all attachments as: .zip

Change History (13)

@lancewillett
15 years ago

Add comment permalink to author notification email

#1 @mikeschinkel
14 years ago

  • Cc mikeschinkel@… added

+1 to the idea.

#2 @scribu
14 years ago

  • Keywords needs-refresh added
  • Milestone changed from Awaiting Triage to 3.1

#3 @lancewillett
14 years ago

  • Cc lance@… added

#4 @nacin
14 years ago

Especially when you have threaded comments, trying to scroll to find the newest comment is difficult.

+1.

#5 @Alphawolf
14 years ago

  • Cc scripts@… added

+1 to the idea.

#6 @shidouhikari
14 years ago

  • Cc shidouhikari added

me wants!

#7 @automattor
14 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [15553]) Link directly to comment in email notification. Props lancewillett. fixes #12797

@demetris
14 years ago

Removes space before colon

#8 @demetris
14 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Initial patch had an unneeded space before a colon. Attached patch removes it.

@johnpbloch
14 years ago

Refreshing demetris' patch

#9 @johnpbloch
14 years ago

  • Keywords needs-refresh removed

#10 @nacin
14 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [16760]) Remove unnecessary space. props demetris, JohnPBloch. fixes #12797.

Note: See TracTickets for help on using tickets.