Opened 5 years ago

Last modified 4 years ago

#12915 confirmed Bug

Edit cell properties inside editable scrolls to top

Reported by: Matthew Popat Owned by:
Priority: Normal Milestone:
Component: General Version: 4.3
Keywords: Blink Cc:

Description

If you have a table inside a div with contenteditable="true", which is in turn inside a div with contenteditable="false" such as.

<div contenteditable="false">

<div contenteditable="true">

<table align="right" border="1" bordercolor="#ccc" cellpadding="5" cellspacing="0" style="border-collapse:collapse">

<tr><th scope="col">Position</th><th scope="col">Astronaut</th></tr> <tr><td>Commander</td><td>Neil A. Armstrong</td></tr>

</table>

</div>

</div>

and you change the properties of a cell. Then when you click 'Ok' on the cell properties dialog, the editor will scroll to the top, even if your table was not at the top of the page. This situation occurs when you have a widget with an editable section that contains a table. Reproduced in version 4.4.7 and on the CKEditor demo page.

Change History (3)

comment:1 Changed 5 years ago by Jakub Ś

Keywords: Blink added
Status: newconfirmed
Version: 4.3

I was able to reproduce this issue from CKEditor 4.3 in Blink browsers.

To reproduce I needed to allow the above code for simple box widget (modified allowedContent inside plugin file). After pasting the code into simple box and modifying the table cell, scrolling occurred.

comment:2 Changed 4 years ago by Matthew Popat

Version: 4.34.5.3

I have downloaded version 4.5.3 and this problem still occurs. Some preliminary debugging suggests it is caused by the dialogue plugin refocussing on the editor and not the nested editable when closed.

Last edited 4 years ago by Matthew Popat (previous) (diff)

comment:3 Changed 4 years ago by Jakub Ś

Version: 4.5.34.3

@matthew.popat Don't change the verison number.

It is used to indicate when problem have started and not last version where problem was spotted.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy