Page MenuHomePhabricator

Scribunto console sometimes added inside wikieditor
Closed, ResolvedPublic

Description

The console of Scribunto is always placed directly after the wpTextbox1, upon document ready. This means that there is a race condition with the setup of WikiEditor, which also manipulates the space around the textbox. Because of this sometimes, the entire console is placed inside the WikiEditor.

In this new async end of page loading times, the module will have to be aware of this potential clash and counter it.. somehow...

Event Timeline

TheDJ raised the priority of this task from to Needs Triage.
TheDJ updated the task description. (Show Details)
TheDJ added a subscriber: TheDJ.

See also: a screenshot of that condition

Screen Shot 2015-08-24 at 17.16.38.png (462×1 px, 81 KB)

Note potentially, CodeEditor exerts some influence here as well ?

The console of Scribunto is always placed directly after the wpTextbox1, upon document ready.

Not always: for most pages the EditPageBeforeEditChecks inserts a <div id="mw-scribunto-console"></div> which Scribunto uses in preference to looking for wpTextbox1.

For read-only pages, though, there isn't any equivalent hook that could be used (unless one has been added recently?), which is why it has the fallback to wpTextbox1.

Change 233603 had a related patch set uploaded (by TheDJ):
(Do Not Submit) Add mw-scribunto-console placeholder to view source page

https://gerrit.wikimedia.org/r/233603

Change 233603 merged by jenkins-bot:
Add mw-scribunto-console placeholder to view source page

https://gerrit.wikimedia.org/r/233603

Anomie assigned this task to TheDJ.
Anomie set Security to None.