Download & Extend

Don't apply "display: inline-block" to .clearfix class, it's not compatible with themes

Project:Markdown Editor for BUEditor
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:major
Assigned:Unassigned
Status:needs review

Issue Summary

Hi!

Fusion is one of the very popular themes that gets screwed up by markdowneditor. It's used by 25 times more people than markdowneditor module. I'm sure there are other themes affected.

Solution: don't apply "display: inline-block" to .clearfix CSS class globally. Apply it as a child of some markdowneditor-specific class.

Comments

#1

This messes with the Omega 960 theme as well. Grrrrrrrr.

#2

FYI to all other google searches hitting this page: I added this to my local.css

/* Fix for markdowneditor css glitch */
.clearfix { display: block !important; }

#3

Status:active» needs work

Committed a fix in CVS. Please try the development snapshot and let me know if it fixes it. If you can help check for CSS layout issues in the major browsers with this fix in place I'd be very grateful.

#4

Works for me in:
ie7
chrome
ff 3.6

#5

Priority:normal» major
Status:needs work» needs review

This bug affects Zen 2 as well so I mark it as major.

Tested the fix in Safari 5 and Firefox 3.6 where it works well.

Looking forward to a 6.x-1.2 release.

#6

Status:needs review» needs work

There has been no patch posted, only a workaround (that should be placed into theme's css).

#7

lolmaus: A fix has been committed to 6-dev, see comment #3.

#8

Version:6.x-1.1» 6.x-1.x-dev
Status:needs work» needs review

2 frjo:

Ah! It's commited already. Please excuse my stupidity. :(

So, everyone please try the dev release to see whether .clearfix is no longer applied globally.

nobody click here