Opened 10 years ago
Closed 10 years ago
#2054 closed defect (bug) (fixed)
heading tag in reply-content
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.3 | Priority: | normal |
Severity: | normal | Version: | 2.2.2 |
Component: | Appearance - Included Themes | Keywords: | has-patch needs-testing |
Cc: |
Description
This code
<h1>This is heading 1</h1> <h2>This is heading 2</h2> <h3>This is heading 3</h3> <h4>This is heading 4</h4> <h5>This is heading 5</h5> <h6>This is heading 6</h6>
Attachments (7)
Change History (17)
#2
@
10 years ago
my theme is twentytwelve and use the default theme.
/wp-content/plugins/bbpress/templates/default
Theme Compatibility
don't built-in default theme into twentytwelve.
#3
@
10 years ago
- Milestone changed from Awaiting Review to 2.3
We could likely set more explicit stylings for these in 2.3.
#4
@
10 years ago
- Owner set to jaredatch
- Status changed from new to assigned
As per the 11/28 dev chat we will reset the margin/padding on header elements. Patch forthcoming.
#8
@
10 years ago
- Keywords has-patch needs-testing added; needs-patch removed
First pass at the patch.
Reset the padding and margin on all the headers.
Reset the list items (broken in TwentyTwelve).
Also added base styling for pre
and code
as I've found that a ton of themes don't style them.
Tested with TwentyTen/Eleven/Twelve. Could probably use some testing with some other themes, but I think it should be a good start.
#9
@
10 years ago
jaredatch, jjj. please test my css.
/* =User Pages -------------------------------------------------------------- */ #bbpress-forums h1, #bbpress-forums h2, #bbpress-forums h3, #bbpress-forums h4, #bbpress-forums h5, #bbpress-forums h6{ clear: none; }
and remove clear: none; below #bbpress-forums h1.
#bbpress-forums h1 { clear: none; font-size: 1.8em; line-height: 1em; padding-bottom: 10px; }
Thanks. Can you include details with your tickets please?