search

marcosamz / vaadin-ckeditor

Automatically exported from code.google.com/p/vaadin-ckeditor
0 stars 0 forks source link

Editor scroll down if you try to insert text content at runtime #3

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
Add a button on the page.
Into the keypress listener insert some code that insert data into the editor
by:
ckEditorTextField.setValue("Test");

At this moment, the editor scrolls down.

What is the expected output? What do you see instead?
The editor should replace the content

What version of the product are you using? On what operating system?
Last version

Please provide any additional information below.

Original issue reported on code.google.com by giovanni...@gmail.com on 1 Jun 2010 at 7:13

GoogleCodeExporter commented 9 years ago 
That is odd, indeed.  We hadn't ever changed the full text of the editor after 
it was created.  We'll see if this is something with Vaadin or CKEditor.

Original comment by yoz...@gmail.com on 15 Jun 2010 at 5:29

GoogleCodeExporter commented 9 years ago 
[deleted comment]
GoogleCodeExporter commented 9 years ago 
This appears to be related to the other bug about IE8 buttons causing the data 
to disappear.  This is either a Vaadin bug, or a bug in our code related to 
subclass TextField (perhaps that's not valid).

The default layout for a main window appears to be CssLayout, and the IE8 but 
continues to show up with both buttons if you don't do a refresh first.

Then I created a VerticalLayout that I added to the Window, and added the 
buttons and CKEditor to it, the IE8 reload did not cause the editor to 
disappear, but instead move down like your report.

In Firebug, the DIVs created look very similar, but a "height: 332px;" became 
"height: 602px", and an embedded DIV went from 280px to 550px (the same 270px 
bigger).  Of course, my code is not doing anything intentionally to cause the 
new size, so somehow the repaint is causing a new calculation.

I'll see if the Vaadin folks have any ideas or if I should really not attempt 
to subclass TextField for this editor.

Original comment by yoz...@gmail.com on 15 Jun 2010 at 6:44

GoogleCodeExporter commented 9 years ago 
Fixed in 0.4 release.  The patch is more a hack than a real solution in that we 
basically keep the DIV element "display: none;" on updates.  What is not clear 
is why that attribute goes away in the first place.

Also, the IE8 bug seems to persist in that if you load the page and click a 
button, all the editors disappear, but if you load the page, click refresh, 
then it works fine.

Original comment by yoz...@gmail.com on 16 Jun 2010 at 7:30

GoogleCodeExporter commented 9 years ago 
[deleted comment]
GoogleCodeExporter commented 9 years ago 
I solved the issue in the same way.

Thanks a lot.

Original comment by giovanni...@gmail.com on 1 Jul 2010 at 5:30

GoogleCodeExporter commented 9 years ago 
As an FYI here, the IE8 issue should be resolved now, too.  Was some sort of 
issue with CssLayout on the re-calcs that don't occur with VerticalLayout.

Original comment by yoz...@gmail.com on 1 Jul 2010 at 5:44

GoogleCodeExporter commented 9 years ago

Original comment by yoz...@gmail.com on 3 May 2011 at 9:56