Hey @Yaron_Koren, I've reopen this bug to add more efficient fix to make this function with more performance (yes, it's still slow for as). This fix is more tricky - is the cache key enough precise and not too narrow? It's work for my tests, but I'm not familiar with all the aspects of this args.

That's new to me! Thank you.

@Yaron_Koren I'm not sure how to edit the patch now, so I would write it here.

It's very late but if someone came to here as me and want to solve it without hacking the core, you can add simple js, in common.js or any other way

I'm not sure this is the right approach. We can also check if the special page is cargo related page, something like:
} elseif ( ( $title != null && $title->isSpecialPage() && strpos( $title->getText(), 'Cargo' ) !== FALSE && !$wgRequest->getCheck( 'wpRunQuery' ) )

That's my take how to implement this. Of course, if there is already a built in way to do it I would be happy to learn.

Yes, you right. editing (also in attached commit msg)

OK, done here: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/PageForms/+/771894/

By DRY concept we should set this chars group in one place (I'm assuming in config entry in extension.json), Then call it from two places in the code.
Do you want me to do it like this, or just add it in JS?

Sure thing. Done, is that good enough?

Yep, I owe you explanation, sorry.

@Yaron_Koren: what about this fix? Is this ever merged? Because autocomplete for unicode chars still not working.

Because table let me nice and tidy look out of the box,

Sorry, a lot of mass because of geritt, this is the relevant commit:
https://gerrit.wikimedia.org/r/c/mediawiki/extensions/PageForms/+/771897

A. well, "values from namesapce" is just the default way to grab pages form the wiki, the NS is just nice additional option, good to have - most of time it would be Main, sometimes files or categories.

@Yaron_Koren can we merge this commit?

The bug solved with other code

Here's suggestion for fix:
https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Cargo/+/697182

Fix suggestion:

Suggest to fix in gerrit:

Yes, it can. Thank you

Clarification: the suggested patch have two parts: one in VEForAll and one in PageForms, this parts working together.

A fix suggested here:

Trying to amend commit as asked results new commit:
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Duplicator/+/594970

I see now.
Yes, we still using PageForms 4.4.1.
We would check the new version, Thank you.

Maybe you could test it by make the parsoid response very slowly.

But sync event is not enough, because VEForAlll updated the textareas asynchronously.

Vedmaka - new approach is await in gerrit.

seems that mw.veForAll.Target had to inherit ve.init.mw.Target instead of ve.init.sa.Target

Temporary hack around it - interval

Branch on gerrit is here