Link dialog shows anchors from entire document in inline and div-based editors

[Piotrek Koszuliński]

1. Add <a name="xxx" id="xxx"></a> somewhere in the non-editable part of divarea and inlineall samples.
2. See that link dialog shows this anchor in the anchors drop down.

Only anchors from the editor's content should be displayed. This behaviour may be configurable, because in some cases, especially in inline editors, it may make sense to link to the surrounding anchors.

[Piotrek Koszuliński]

It looks that we need to discuss this first. Some of us said that in inline editors all anchors should be shown.

[Marek Lewandowski]

Definitely we should take care of that, since it messes (especialy id-based dropdown) with anchors created used especially for layout of page where CKEditor is placed.

[Olek Nowodziński]

As for me, inline editors should consider surrounding anchors because they are supposed to integrate as much as possible. Still, div-based editors should act just like framed editor in that case, detecting anchors from editor's content only.

[Jakub Ś]

I agree div based editor should only be limited to anchors in editor while inline editor should allow accessing anchors in whole page.

[Piotrek Koszuliński]

The question that remains is what should be the default behaviour in each type of editor?

• framed - only content,
• inline - entire document or content?
• divarea - entire document or content? in this case the more general question may be - do we understand divarea as inline editing or is it a lightweight editing in separate context (as framed)?

[Jakub Ś]

To my understanding:

• framed - only content,
• inline - entire document
• divarea - only content and we understand divarea a lightweight editing in separate context (as framed). If it was to be treated as inline editor what sense would there be to have it? What would the difference be between inline and dive area (except for elements path)?

[Olek Nowodziński]

Replying to Reinmar:

The question that remains is what should be the default behaviour in each type of editor?

• framed - only content,
• inline - entire document or content?
• divarea - entire document or content? in this case the more general question may be - do we understand divarea as inline editing or is it a lightweight editing in separate context (as framed)?

• framed - only content
• inline - entire document
• divarea - only content

Divarea is basically framed editing without iframe-ish limitations. Something like Framed++. In fact, this is why we created it and we should follow that path.

[Frederico Caldeira Knabben]

Replying to a.nowodzinski:

• framed - only content
• inline - entire document
• divarea - only content

I agree.

[Olek Nowodziński]

​t/11359 branch unifies the discovery of anchors according to comment:10.

(+corresponding tests)

[Piotrek Koszuliński]

1. No <meta charset> in test file.
2. Instead of itemsAreSame I would use string comparison (after join(',')). YUI's array assertions do not give clear error messages.
3. You're not testing empty anchors (which have to be discovered differently, because their representation differs).
4. I would expose logic retrieving the scope to CKEDITOR.plugins.link. We didn't make it configurable, so let's make it overridable.

[Piotrek Koszuliński]

* @returns {Array} An array of anchor elements.

May be changed to:

* @returns {CKEDITOR.dom.element[]} An array of anchor elements.

[Olek Nowodziński]

​git:05f125f landed in master (c7219cb tests).