Skip to:
Content

bbPress.org

Opened 12 years ago

Last modified 11 years ago

#2244 new task (blessed)

In loop-single-reply.php to replace divs with ul and li

Reported by: alexvorn2's profile alexvorn2 Owned by:
Milestone: Future Release Priority: low
Severity: minor Version: 2.2.3
Component: Appearance - Theme Compatibility Keywords:
Cc:

Description

In the loop-single-reply.php file there are used div tags instead of ul and li tags.

As bbPress use a table less design I think it would be more familiar to use everywhere is needed.

Change History (3)

#1 @alexvorn2
12 years ago

AND to add <?php bbp_reply_class(); ?> in the first tag.

#2 @johnjamesjacoby
12 years ago

  • Component changed from General to Theme Compatability
  • Milestone changed from Awaiting Review to Future Release
  • Priority changed from normal to low
  • Type changed from defect to task
  • Version changed from 2.1 to 2.2.3

I think div's are fine here for now. The markup could be rearranged, but we can't easily move the call to bbp_reply_class() without shifting some associated CSS.

Moving to future release.

#3 @netweb
11 years ago

Classes were added in r4599

Note: See TracTickets for help on using tickets.