Skip to:
Content

bbPress.org

Opened 14 years ago

Last modified 13 years ago

#2244 new task (blessed)

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

Reported by: alexvorn2 Owned by:
Priority: low Milestone: Future Release
Component: Appearance - Theme Compatibility Version: 2.2.3
Severity: minor 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
14 years ago

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

#2 @johnjamesjacoby
14 years ago

  • Component General → Theme Compatability
  • Milestone Awaiting Review → Future Release
  • Priority normal → low
  • Type defect → task
  • Version 2.1 → 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
13 years ago

Classes were added in r4599

Note: See TracTickets for help on using tickets.