Making WordPress.org

Opened 5 years ago

Closed 5 years ago

#4428 closed defect (bug) (fixed)

Support Forums: Add sub-forum support to the front page archive view

Reported by: clorith's profile Clorith Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: Priority: normal
Component: Support Forums Keywords: has-patch has-screenshots
Cc:

Description

There comes a time, in every forums life, when sub-forums become a necessity.

We had some sub forums previously, but we saw no further need for these with the forum migration, and so they were never included in the front page archive styles we created at that time.

After some discussions with other teams, we see a need for these at this time, in particular so that the mobile forums can be included, and they have a need to separate between some of the platforms.

The attached patch checks if a forum has sub-forums, and lists them in the same style as the "View forum" link does today.

It also checks if the parent is a category or not, if it's not a category then it won't show the "View forum" link, if it's a usable forum with sub-forums (thinking ahead, in case it ever comes up) it will append the view link after the sub-forum links.

The intent here is not to have a massive amount of sub-forums going many levels deep, but on a top level there is a legitimate need at times.

Attachments (2)

sub-forums.PNG (39.8 KB) - added by Clorith 5 years ago.
4428.patch (1.6 KB) - added by Clorith 5 years ago.

Download all attachments as: .zip

Change History (5)

@Clorith
5 years ago

@Clorith
5 years ago

#1 @SergeyBiryukov
5 years ago

  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

Previously: #1959, #2042, #2191.

This ticket was mentioned in Slack in #meta by clorith. View the logs.


5 years ago

#3 @SergeyBiryukov
5 years ago

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

In 9377:

Support Theme: Add subforum support to the front page archive view.

Props Clorith.
Fixes #4428.

Note: See TracTickets for help on using tickets.