Opened 4 months ago
Closed 3 months ago
#23244 closed defect (bug) (fixed)
Twenty Twelve: add clear: both; to aligncenter class
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.6 |
| Component: | Bundled Theme | Version: | 3.5 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | DrewAPicture |
Description
img.aligncenter {
margin-bottom: 0.857143rem;
margin-top: 0.857143rem;
}
to
img.aligncenter {
margin-bottom: 0.857143rem;
margin-top: 0.857143rem;
clear: both;
}
Attachments (1)
Change History (9)
comment:1
DrewAPicture — 4 months ago
- Cc DrewAPicture added
comment:2
lancewillett — 4 months ago
I vote to close as wontfix unless alexvorn2 you can give a bit of the "why" it's needed.
See these tips for providing a useful bug report: http://codex.wordpress.org/Reporting_Bugs
because when adding two images: first one with float left and the second with float center you will get this - http://wpimpact.com/wp-content/uploads/2013/01/images_WPImpress_20130121_201840.png
previous themes: Twenty Eleven and Twenty Ten have clear: both;
I thought I will get understood very easily from what I posted before.
Clear: both help a little in arranging the element to fit in the column.
See what happens when I applied the fix: http://wpimpact.com/wp-content/uploads/2013/01/images_WPImpress_20130121_202720.png
comment:5
MikeHansenMe — 4 months ago
I was able to reproduce this as well. The patch is about as simple as it gets. +1
comment:6
follow-up:
↓ 7
lancewillett — 4 months ago
- Keywords has-patch added
- Milestone changed from Awaiting Review to 3.6
- Type changed from enhancement to defect (bug)
Screenshots and explanation are very helpful -- thanks. :)
In the patch we'd only apply to .aligncenter selector, though, probably.
Replying to lancewillett:
Screenshots and explanation are very helpful -- thanks. :)
In the patch we'd only apply to .aligncenter selector, though, probably.
We can apply to "alignnone" too.
comment:8
lancewillett — 3 months ago
- Owner set to lancewillett
- Resolution set to fixed
- Status changed from new to closed
In 23428:

@alexvorn2: Seems to be working fine without it. Wouldn't have a problem with testing it if you'd like to submit a patch.