[
https://issues.apache.org/jira/browse/WAVE-285?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13051707#comment-13051707
]
Thomas Broyer commented on WAVE-285:
------------------------------------
There's an overflow:auto on .editor in the EditorHarness that's enough to make
it render correctly, but not enough for the editor to gain focus when clicking
within the editor boundaries (but outside the document's boundaries).
There's no overflow:auto otherwise, and you can reproduce the bug in WiaB:
inserting a floated div with your browser's developer tools, to simulate one
that'd be generated by a doodad, will make a "rendering bug" (where the floated
div overflows the editor); if you add an overflow:auto, it fixes rendering, but
then you can experience the "behavior bug" about focusing the editor).
I'll have to test with IE7+, but adding overflow:auto to the
ContentDocElement's <div> would probably fix it (I made a quick patch using the
<div><div>[content here]</div><br style="clear: both"></div> approach and it
works great, I'll try with the simpler overflow:auto fix)
> Editor should insert a <br style="clear: both"> at the end so it grows with
> floated elements (focus issue)
> ----------------------------------------------------------------------------------------------------------
>
> Key: WAVE-285
> URL: https://issues.apache.org/jira/browse/WAVE-285
> Project: Wave
> Issue Type: Bug
> Components: Extentions, Web Client
> Reporter: Thomas Broyer
> Priority: Minor
>
> When a doodad generates floated elements (e.g. style="float: left"), the
> editor surface doesn't grow with it. As a result, when clicking on the part
> on the side of the floated element, you might click outside the editor, and
> it thus fails to give it focus.
> Adding a <br style="clear: both"> or similar between the body element's </ul>
> and the ContentDocElement's </div> would fix it.
> Currently, the document <body><line/>Foo</body> is rendered as:
> <div>
> <ul>
> <div>Foo</div>
> </ul>
> </div>
> It should be made either:
> <div>
> <ul>
> <div>Foo</div>
> </ul>
> <br style="clear: both">
> </div>
> or (wrapping the current structure into an additional <div>, so it can be
> made the "autoAppendContainer")
> <div>
> <div>
> <ul>
> <div>Foo</div>
> </ul>
> </div>
> <br style="clear: both">
> </div>
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
