Make WordPress Core

Opened 11 years ago

Closed 10 years ago

#12797 closed enhancement (fixed)

Add comment permalink to plain-text email notification

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


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:

The improved link would be to the exact comment:

Attachments (3)

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

Download all attachments as: .zip

Change History (13)

11 years ago

Add comment permalink to author notification email

#1 @mikeschinkel
11 years ago

  • Cc mikeschinkel@… added

+1 to the idea.

#2 @scribu
11 years ago

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

#3 @lancewillett
11 years ago

  • Cc lance@… added

#4 @nacin
11 years ago

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


#5 @Alphawolf
11 years ago

  • Cc scripts@… added

+1 to the idea.

#6 @shidouhikari
11 years ago

  • Cc shidouhikari added

me wants!

#7 @automattor
11 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

10 years ago

Removes space before colon

#8 @demetris
10 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

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

10 years ago

Refreshing demetris' patch

#9 @johnpbloch
10 years ago

  • Keywords needs-refresh removed

#10 @nacin
10 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.