Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#7980 closed defect (bug) (fixed)

Threaded Comments Misc. Items

Reported by: filosofo's profile filosofo Owned by:
Milestone: 2.7 Priority: normal
Severity: normal Version: 2.7
Component: General Keywords: has-patch comments threaded-comments
Focuses: Cc:

Description

Misc. patches for the new threaded comments functionality, as I run across various issues.

Attachments (1)

comment_patches.diff (1.1 KB) - added by filosofo 15 years ago.

Download all attachments as: .zip

Change History (3)

#1 @filosofo
15 years ago

About comment_patches.diff:

  • comment_text() work being duplicated
  • Logical placement of </div> (super trivial, but may as well get it)
  • reverse_top_level: the problem is that if you pass the reverse_top_level argument in set to false, which means that you want it to be false, but option "comment_order" is set to anything other than 'asc', your reverse_top_level is set to true.

#2 @ryan
15 years ago

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

(In [9385]) Threaded comment fixes from filosofo. fixes #7980

Note: See TracTickets for help on using tickets.