Page link in TinyMCE editor instances is broken.

[tvdeyen]

          This broke the page link in TinyMCE editor instances.

Originally posted by @tvdeyen in #2439 (comment)

[tvdeyen]

We need to revert the change made in #2439. I totally forgot that we return all pages from the API on purpose. As already discussed in #2258

The actual issue was introduced in c98605a where we started to scope the pages to the language again. I will revert this change as well.

[dbwinger]

@tvdeyen my only concern with this is to make sure that everywhere we have a page selector, it is showing the site and language along with the page name/title, which I think is already the case. I have multiple sites mirroring the same content in different languages/regions, which means some page name duplication. I need to know which one I'm linking to.

[tvdeyen]

@dbwinger yes, this is the case. We have a jQuery plugin alchemyPageSelect that uses a Handlebars template to render the result. In that we display all necessary information. If not, please open a PR/issue.

[dbwinger]

Great, yes, I think that covers it all. Thanks!

…

On Fri, Jun 30, 2023 at 8:49 AM Thomas von Deyen ***@***.***> wrote: @dbwinger <https://github.com/dbwinger> yes, this is the case. We have a jQuery plugin alchemyPageSelect that uses a Handlebars template to render the result. It that we display all necessary information. If not, please open a PR/issue. — Reply to this email directly, view it on GitHub <#2500 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACTAF6DTXMFIQQJA3JQ2Y3XN3DNFANCNFSM6AAAAAAZYDRGR4> . You are receiving this because you were mentioned.Message ID: ***@***.***>