136911 2014-09-18 01:12:33 -0700 Non-editable blocks at the beginning/end of contenteditable block break the selection 2016-11-21 03:15:40 -0800 1 1 1 Unclassified WebKit HTML Editing 528+ (Nightly build) Unspecified Unspecified NEW P2 Normal --- 1 pkoszulinski webkit-unassigned k.krzton oldest_to_newest 1035863 0 pkoszulinski 2014-09-18 01:12:33 -0700 Steps to reproduce the problem: 1. Open the attached sample. It contains editable divs with non-editable block islands at the beginning and the end of them. 2. Try steps explained in the sample - CTRL+A, select all from context menu, CTRL+end/home, setting selection programatically. What is the expected behavior? Selection should include all the contents of editable element from the first non-editable block to the last one. What went wrong? Selection is either totally lost (first case) or a collapsed selection is set (second case). I am aware that this is a tricky case but it works fine in all other browsers (except of course Blink). I tested the attached sample on Firefox and IE11 and both - visual part and selection API part - were working all right. This issue was reported multiple times on CKEditor's bug tracker - http://dev.ckeditor.com/ticket/11064. It is getting more and more important for adoption of CKEditor widgets (wrappers for non-editable content inside editor) which proved to be very useful and many users start using them. 1035864 1 238294 pkoszulinski 2014-09-18 01:13:44 -0700 Created attachment 238294 Sample to reproduce the issue 1035865 2 pkoszulinski 2014-09-18 01:14:33 -0700 Related bug on Blink's bug tracker - https://code.google.com/p/chromium/issues/detail?id=415474 1253217 3 k.krzton 2016-11-21 03:15:40 -0800 Is there any progress on this issue? It is still reproducible (same as Blink issue https://code.google.com/p/chromium/issues/detail?id=415474). 238294 2014-09-18 01:13:44 -0700 2014-09-18 01:13:44 -0700 Sample to reproduce the issue non-editable-blocks.html text/html 2611 pkoszulinski PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KCTxtZXRhIGNoYXJzZXQ9InV0Zi04Ij4KCTx0 aXRsZT5Gb28gZm9vITwvdGl0bGU+Cgk8c3R5bGU+CgkJZGl2IHsKCQkJcGFkZGluZzogMjBweDsK CQl9CgkJW2NvbnRlbnRlZGl0YWJsZT10cnVlXSB7CgkJCWJvcmRlcjogc29saWQgMnB4IGdyZWVu OwoJCX0KCQlbY29udGVudGVkaXRhYmxlPWZhbHNlXSB7CgkJCWJvcmRlcjogc29saWQgMnB4IHJl ZDsKCQl9CgkJW2NvbnRlbnRlZGl0YWJsZT1mYWxzZV0gPiAqIHsKCQkJYmFja2dyb3VuZDogI0VF RTsKCQl9Cgk8L3N0eWxlPgo8L2hlYWQ+Cjxib2R5PgoJPGgxPkJ1Z3M8L2gxPgoKCTx1bD4KCQk8 bGk+Q1RSTCtBIG9yICJzZWxlY3QgYWxsIiBmcm9tIGNvbnRleHQgbWVudSBkb2VzIG5vdCB3b3Jr IGF0IGFsbCAoc2VsZWN0aW9uIGlzIGxvc3QgaW4gMXN0IGNhc2Ugb3IgbW92ZWQgdG8gdGhlIGVu ZCBpbiAybmQgY2FzZSk8L2xpPgoJCTxsaT5DVFJMK0hvbWUsIENUUkwrRW5kIGtleXN0cm9rZXMg ZG8gbm90IHdvcmsgKHNob3VsZCBtb3ZlIHNlbGVjdGlvbiB0byBzdGFydC9lbmQgb2YgdGhlIGRv Y3VtZW50KTwvbGk+CgkJPGxpPkl0J3MgaW1wb3NzaWJsZSB0byBzZXQgdGhlIHNlbGVjdGlvbiBw cm9ncmFtYXRpY2FsbHkgYmVmb3JlIHRoZSBmaXJzdCBibG9jayBvciBhZnRlciB0aGUgc2Vjb25k IGJsb2NrPC9saT4KCTwvdWw+CgoJPGgxPjEuIE5vbi1lZGl0YWJsZSBibG9ja3MgYXQgYm90aCBl bmRzPC9oMT4KCgk8cD48YnV0dG9uIG9uY2xpY2s9InNlbGVjdEFsbCgwKSI+c2VsZWN0IGFsbDwv YnV0dG9uPjwvcD4KCgk8ZGl2IGNvbnRlbnRlZGl0YWJsZT0idHJ1ZSIgaWQ9ImVkaXRhYmxlMCI+ CgkJPGRpdiBjb250ZW50ZWRpdGFibGU9ImZhbHNlIj4KCQkJPHA+Rmlyc3QgYmxvY2s8L3A+CgkJ PC9kaXY+CgkJPHA+U29tZSBlZGl0YWJsZSB0ZXh0LiBTb21lIGVkaXRhYmxlIHRleHQuIFNvbWUg ZWRpdGFibGUgdGV4dC4gU29tZSBlZGl0YWJsZSB0ZXh0LiBTb21lIGVkaXRhYmxlIHRleHQuIFNv bWUgZWRpdGFibGUgdGV4dC48L3A+CgkJPGRpdiBjb250ZW50ZWRpdGFibGU9ImZhbHNlIj4KCQkJ PHA+U2Vjb25kIGJsb2NrPC9wPgoJCTwvZGl2PgoJPC9kaXY+CgoJPGgxPjIuIE5vbi1lZGl0YWJs ZSBibG9jayBvbmx5IGF0IHRoZSBiZWdpbm5pbmc8L2gxPgoKCTxwPjxidXR0b24gb25jbGljaz0i c2VsZWN0QWxsKDEpIj5zZWxlY3QgYWxsPC9idXR0b24+PC9wPgoKCTxkaXYgY29udGVudGVkaXRh YmxlPSJ0cnVlIiBpZD0iZWRpdGFibGUxIj4KCQk8ZGl2IGNvbnRlbnRlZGl0YWJsZT0iZmFsc2Ui PgoJCQk8cD5GaXJzdCBibG9jazwvcD4KCQk8L2Rpdj4KCQk8cD5Tb21lIGVkaXRhYmxlIHRleHQu IFNvbWUgZWRpdGFibGUgdGV4dC4gU29tZSBlZGl0YWJsZSB0ZXh0LiBTb21lIGVkaXRhYmxlIHRl eHQuIFNvbWUgZWRpdGFibGUgdGV4dC4gU29tZSBlZGl0YWJsZSB0ZXh0LjwvcD4KCTwvZGl2PgoK CTxoMT4zLiBPbmx5IGVkaXRhYmxlIHRleHQ8L2gxPgoKCTxwPjxidXR0b24gb25jbGljaz0ic2Vs ZWN0QWxsKDIpIj5zZWxlY3QgYWxsPC9idXR0b24+PC9wPgoKCTxkaXYgY29udGVudGVkaXRhYmxl PSJ0cnVlIiBpZD0iZWRpdGFibGUyIj4KCQk8cD5Tb21lIGVkaXRhYmxlIHRleHQuIFNvbWUgZWRp dGFibGUgdGV4dC4gU29tZSBlZGl0YWJsZSB0ZXh0LiBTb21lIGVkaXRhYmxlIHRleHQuIFNvbWUg ZWRpdGFibGUgdGV4dC4gU29tZSBlZGl0YWJsZSB0ZXh0LjwvcD4KCQk8cD5Tb21lIGVkaXRhYmxl IHRleHQuIFNvbWUgZWRpdGFibGUgdGV4dC4gU29tZSBlZGl0YWJsZSB0ZXh0LiBTb21lIGVkaXRh YmxlIHRleHQuIFNvbWUgZWRpdGFibGUgdGV4dC4gU29tZSBlZGl0YWJsZSB0ZXh0LjwvcD4KCTwv ZGl2PgoKCTxzY3JpcHQ+CgpmdW5jdGlvbiBzZWxlY3RBbGwoIGVkaXRhYmxlTnVtICkgewoJdmFy IHNlbCA9IGRvY3VtZW50LmdldFNlbGVjdGlvbigpLAoJCXJhbmdlID0gZG9jdW1lbnQuY3JlYXRl UmFuZ2UoKSwKCQllZGl0YWJsZSA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCAnZWRpdGFibGUn ICsgZWRpdGFibGVOdW0gKTsKCglyYW5nZS5zZXRTdGFydCggZWRpdGFibGUsIDAgKTsKCXJhbmdl LnNldEVuZCggZWRpdGFibGUsIGVkaXRhYmxlLmNoaWxkTm9kZXMubGVuZ3RoICk7CgoJc2VsLnJl bW92ZUFsbFJhbmdlcygpOwoJc2VsLmFkZFJhbmdlKCByYW5nZSApOwoKCWNvbnNvbGUubG9nKCAn UmFuZ2UgY291bnQnLCBzZWwucmFuZ2VDb3VudCApOwoJaWYgKCBzZWwucmFuZ2VDb3VudCApIHsK CQlyYW5nZSA9IHNlbC5nZXRSYW5nZUF0KCAwICk7CgkJY29uc29sZS5sb2coICdSYW5nZSBzdGFy dCcsIHJhbmdlLnN0YXJ0Q29udGFpbmVyLCByYW5nZS5zdGFydE9mZnNldCApOwoJCWNvbnNvbGUu bG9nKCAnUmFuZ2UgZW5kJywgcmFuZ2UuZW5kQ29udGFpbmVyLCByYW5nZS5lbmRPZmZzZXQgKTsK CQljb25zb2xlLmxvZyggJ1NlbCBhbmNob3InLCBzZWwuYW5jaG9yTm9kZSwgc2VsLmFuY2hvck9m ZnNldCApOwoJCWNvbnNvbGUubG9nKCAnU2VsIGZvY3VzJywgc2VsLmZvY3VzTm9kZSwgc2VsLmZv Y3VzT2Zmc2V0ICk7Cgl9Cn0KCgk8L3NjcmlwdD4KPC9ib2R5Pgo8L2h0bWw+Cg==