Skip to:
Content

Opened 4 years ago

Closed 3 years ago

#2833 closed enhancement (wontfix)

Adding a div wrapper around the loop-forums.php <ul> would greatly help styling

Reported by: risingfish Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Appearance - Included Themes Keywords:
Cc:

Description

The way the markup is currently built makes it difficult to put custom spacing between the forum list and the controls above it. Adding a div like the following will make it a lot easier to tweak:

<div class="bbp-forum-container">
	<ul id="forums-list-0" class="bbp-forums">
--- code ---
	</ul><!-- .forums-directory -->
</div>

There's no need to include pre-defined styles for it if you don't want to. It will just hangout there until some like myself chooses to use it.

Change History (5)

#1 @netweb
4 years ago

  • Component changed from General to Appearance - Included Themes
  • Keywords close added

Risingfish,

I'm pretty sure you will find that using bbp_before_main_content() and bbp_after_main_content() hooks should allow you do everything your thinking of, bbPress has many of these before/after hooks, the bbpress Visual Hooks plugin might be of interest to you to explore these.

#2 @netweb
4 years ago

  • Keywords reporter-feedback added

#3 @douglsmith
4 years ago

I may not be understanding what you're going for @risingfish, but what could you style on the div that you couldn't style on the ul? The same css could apply to either.

#4 @netweb
4 years ago

Here's an example plugin of what I was referring to above https://gist.github.com/ntwb/0f36597a6a0c5b76d72b

#5 @thebrandonallen
3 years ago

  • Keywords close reporter-feedback removed
  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Going to close this, as there seems to be existing solutions.

Note: See TracTickets for help on using tickets.