Opened 14 months ago

Closed 13 months ago

#1800 closed defect (fixed)

Update bbp-forum-template CSS Class

Reported by: netweb Owned by:
Priority: normal Milestone: 2.1
Component: Actions/Filters Version: 2.0
Severity: minor Keywords: has-patch
Cc:

Description

Changeset 3735 modified some CSS Class names in bbpress.css
.bbp-forum-description was changed to .bbp-forum-content

(​http://bbpress.trac.wordpress.org/changeset/3735/branches/plugin/bbp-theme-compat/css/bbpress.css)

Update for /plugin/bbp-includes/bbp-forum-template.php change #L1816

FROM: 'before' => '<div class="bbp-template-notice info"><p class="bbp-forum-description">',

TO: 'before' => '<div class="bbp-template-notice info"><p class="bbp-forum-content">',

(​http://bbpress.trac.wordpress.org/browser/branches/plugin/bbp-includes/bbp-forum-template.php#L1816)

Attachments (2)

bbp-forum-template.php​ (75.5 KB) - added by netweb 14 months ago.
bbPress.1800.diff​ (543 bytes) - added by netweb 14 months ago.

Download all attachments as: .zip

Change History (11)

netweb — 14 months ago

comment:1   netweb — 14 months ago

Be Gentle, my first ever patch submitted :)

Sadly I was hoping the patch would show up as a 'diff' oh well, had hoped for ...ticket/1800/1800.diff, maybe next time, at least its accurate (Line #1816)!

Will read more trac docs... Have only just started using GitHub? for VCS! Grokking slowly...

comment:2   alexvorn2 — 14 months ago

please upload the diff, can help you to generate the file if it is hard for you...

netweb — 14 months ago

comment:3 follow-up: ↓ 4   netweb — 14 months ago

  • Severity changed from normal to minor

Ok... I think I got it this time :)

comment:4 in reply to: ↑ 3   johnjamesjacoby — 14 months ago

Replying to netweb:

Ok... I think I got it this time :)

Nicely done! Thanks!

comment:5   johnjamesjacoby — 14 months ago

  • Milestone changed from Awaiting Review to 2.1

comment:6   johnjamesjacoby — 14 months ago

(In [3834]) Add missing styling to bbp-forum-content in theme compat. See #1800.

comment:7   johnjamesjacoby — 14 months ago

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

(In [3835]) Tweak bbp-forum-content and bbp-forums-list styling. Fixes #1800.

comment:8 follow-up: ↓ 9   netweb — 13 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Patch was not applied :)

comment:9 in reply to: ↑ 8   johnjamesjacoby — 13 months ago

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

Replying to netweb:

Patch was not applied :)

It didn't need to be. Changing the class in that one place is not enough to fix the problem.

bbp-forum-content is the wrapper class for the actual content area of a forum. bbp-forum-description is used in two places: index/subforums, and the header description.

Note: See TracTickets for help on using tickets.