Suspect this is because our setSelection calls setFocus AFTER moving the cursor. Googling suggests you should always focus before moving the cursor.
https://stackoverflow.com/questions/33394855/how-to-set-cursor-position-in-codemirror-editor
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Focus before setting the cursor/selection | mediawiki/extensions/CodeMirror | master | +1 -1 |
I was just debugging this. The <ref></ref> link is put there by ChartInsert, right?
In one of my own user scripts, I only had to change the way I was getting the cursor position (diff) so that it knew where to set it. Maybe not the same issue. The gadget on enwiki appears to use charinsert-core.js, and I can't find the relevant code there.
It's not the same issue I think. I had breakpoints on all the textSelection functions and this setSelection is the last in the sequence, and I just verified with codemirror.doc.getCursor() in the console that the setFocus() causes the move of the cursor.
Change 440750 had a related patch set uploaded (by TheDJ; owner: TheDJ):
[mediawiki/extensions/CodeMirror@master] Focus before setting the cursor/selection
Change 440750 merged by jenkins-bot:
[mediawiki/extensions/CodeMirror@master] Focus before setting the cursor/selection