Find is not highlighting the word in read-only blocks

[Satya Minnekanti]

To reproduce the defect:

1. copy the following code and paste it in to source

   <p >
   	first editable paragraph.</p>
   <p contenteditable = "false" >
   	first non editable paragraph.</p>
   <p contenteditable = "false" >
   	second non editable paragraph.</p>
   <p>
   	second editable paragraph.</p>
   

2. click on Find icon and enter the word non in find what field and click find button.

Expected Result:

non word in the first non editable paragraph should be highlighted.

Expected Result:

Find is not highlighting the non word in the first non editable but when we close the Find and Replace dialog you will see the word getting highlighted.

[Wiktor Walc]

As reported in #7049, using the following code:

<p contenteditable="false">inside read-only paragraph</p>

results in a javascript error:

Message: currentNode is null 
Line: 455
URI: /ckeditor/_source/plugins/styles/plugin.js

[typeof]

What do you think about adding a property called something like "appliedToReadOnly" to the CKEDITOR.style class? And then modify the style and find plugins accordingly?

[Garry Yao]

Find highlight is current done through styling in all browsers, while actually IE is the only browser that requires it (for not supporting selection mark without holding focus), highlight in the rest of browsers should be simplified by just making a plain selection, which will be free of the read-only styling limitation.

[Frederico Caldeira Knabben]

Replying to garry.yao:

Find highlight is current done through styling in all browsers, while actually IE is the only browser that requires it (for not supporting selection mark without holding focus), highlight in the rest of browsers should be simplified by just making a plain selection, which will be free of the read-only styling limitation.

How would we handle it on IE at this point?

[Garry Yao]

Vote for using also native selection on IE to present, the only sacrifice is that focus will be moved out of dialog temporarily when displaying the highlight.

[Garry Yao]

Ticket Test:
​run OR ​view source.

[Frederico Caldeira Knabben]

Replying to garry.yao:

Vote for using also native selection on IE to present, the only sacrifice is that focus will be moved out of dialog temporarily when displaying the highlight.

This is not an acceptable solution. It breaks the keyboard navigation on the dialog as well as bringing accessibility issues.

[Garry Yao]

Ticket tests updated.

[Garry Yao]

Fixed with [7211].