Page MenuHomePhabricator

Impossible to create pages when opted into Minerva skin on desktop
Closed, ResolvedPublic


I've opted into the desktop skin MinervaNeue on desktop
Upon trying to create a new page I got this screen and the editor never loaded:

There seems no way to bypass this except disabling JavaScript.


(This used to work)

Event Timeline

Deskana added a subscriber: Deskana.

I think I'm missing something here. That page exists now, so I tested this by navigated to another page that doesn't exist. If I click the edit pencil, then I can edit the page as normal, so I don't know how to reproduce this problem.

That said, you are right that that link doesn't work if your skin is MinervaNeue. I suspect that this is a problem with the skin, as that link works fine for me using Timeless, MonoBook, and Vector. Where is this link is exposed to users of the MinervaNeue skin?

^ hopefully that helps. Otherwise visit Special:Preferences and opt into the skin. You encounter this issue any time you follow a red link.
It seems some problematic JS is being loaded by VisualEditor without any fail safe.
Looks like there is something wrong in the logic here for isSupportedEditPage:
Or in the JS provided by VisualEditor that actually loads the editor.

This comment was removed by Esanders.
This comment was removed by Esanders.

I'll go back to my first guess. It is expected that once isSupportedEditPage returns true, that the page will have all the elements for initialising the editor in place. In this case the page doesn't have the #ca-edit link.

This is the offending line:

public function getPageActions() {
    return $this->isFallbackEditor() ? [] : $this->data['page_actions'];

This is intentional. Why would a editor have/need an edit icon? We only have a save button on that page.

It is assumed that you need to some 'edit' link to trigger and re-trigger the editor. Pinging @matmarex who wrote this code.

Change 462686 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] Remove #ca-edit from required skin elements

Change 462686 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Remove #ca-edit from required skin elements on edit pages

Deskana triaged this task as Medium priority.