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 | 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;} }
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.