Skip to:
Content

bbPress.org

Opened 10 years ago

Closed 10 years ago

#2661 closed defect (bug) (duplicate)

Responsive css needs more rules to handle show_lead_topic mode

Reported by: wpdennis's profile wpdennis Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Appearance - Theme Compatibility Keywords:
Cc:

Description

If bbp_show_lead_topic() is enabled the topic has own css classes (with topic instead of reply). The responsive css part covers only reply classes (bbp-reply-author and so on).

I temporarely fixed it by adding the following rules:

#bbpress-forums .bbp-body div.bbp-topic-author
same as #bbpress-forums .bbp-body div.bbp-reply-author BUT margin-top has to be 0.

#bbpress-forums div.bbp-topic-author img.avatar
same as #bbpress-forums div.bbp-reply-author img.avatar BUT top has to be 0.

And the following rules identical to their reply counterparts:

#bbpress-forums div.bbp-topic-author a.bbp-author-name
#bbpress-forums div.bbp-topic-author .bbp-author-role
#bbpress-forums .bbp-body div.bbp-topic-content
#bbpress-forums div.topic

As a side note: On smaller devices were we don't split replies in 2 columns (Author|Content), there's no reason to show the header with 2 columns. One option would be to remove the author column (only tested with show_lead_topic() activated):

@media only screen and (max-width: 480px) {
    .bbp-replies .bbp-header .bbp-reply-author {display:none;}
}

Change History (1)

#1 @netweb
10 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed

Thanks, I am pretty sure I have most of this covered in the most recent patch in #2486.

If you could check out the latest patch there and any feedback for what may be missing that would be appreciated.

Note: See TracTickets for help on using tickets.