Skip to:
Content

bbPress.org

Opened 3 years ago

Closed 3 years ago

#3443 closed regression (fixed)

"bbpress" body class appearing on all pages

Reported by: johnjamesjacoby's profile johnjamesjacoby Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.6.8 Priority: high
Severity: normal Version: 2.6.7
Component: Appearance Keywords: commit
Cc:

Description

A regression was discovered in 2.6.7 that is causing the bbpress body class to unintentionally appear on all pages, even non-bbPress ones.

Introduced in r7190.

Fix imminent.

Change History (2)

#1 @johnjamesjacoby
3 years ago

In 7220:

Appearance: only add no-js body class on bbPress pages.

This change addresses a regression introduced in r7190, causing the bbpress body class to unintentionally be added to all pages triggering layout issues in some WordPress themes.

This commit makes sure that the bbpress body class is only added when intended, and moves the no-js body class into the same conditional check.

In trunk for 2.7. See #3443.

#2 @johnjamesjacoby
3 years ago

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

In 7221:

Appearance: only add no-js body class on bbPress pages.

This change addresses a regression introduced in r7190, causing the bbpress body class to unintentionally be added to all pages triggering layout issues in some WordPress themes.

This commit makes sure that the bbpress body class is only added when intended, and moves the no-js body class into the same conditional check.

In branches/2.6 for 2.6.8. Fixes #3443.

Note: See TracTickets for help on using tickets.