Extension talk:VisualEditor/Skin requirements

About this board

Previous discussion was archived at Extension talk:VisualEditor/Skin requirements/Archive 1 on 2019-05-16.

How to make visual editor load without reloading the page

8
Summary by Sophivorus

Fixed in recent versions of VisualEditor.

Sophivorus (talkcontribs)

Hi! As the developer of Skin:Poncho, I followed this guide to make it compatible with the visual editor. However, there's one detail that I couldn't manage. When I click Edit on the Vector skin, the visual editor loads without reloading the page. However, when I do so on the Poncho skin, the page reloads. See https://www.appropedia.org/Special:Random for a live site using Poncho. What am I missing? Thanks!

Matma Rex (talkcontribs)

It looks like there are two elements with id="ca-ve-edit" on your pages, one of which is invisible and unclickable, and VE is hooking to the invisible one.

Sophivorus (talkcontribs)

Right, I forgot about that. BTW, the reason why there're two elements with id "ca-ve-edit" is because Poncho outputs page actions as OOUI buttons rather than links, but VisualEditor assumes links and changes its content for plain text via JavaScript. Since I found no way to prevent this, I did this hack to trick VisualEditor. Perhaps when we update to MediaWiki 1.42+ I'll be able to use Codex or some other new figure this out.

Matma Rex (talkcontribs)
Sophivorus (talkcontribs)
Sophivorus (talkcontribs)

I resolved the conflicts but Jenkins now complains about a selenium test not passing. I shared my thoughts on the patchset. Cheers!

Matma Rex (talkcontribs)

I got it working and approved it. Thanks for doing most of the work!

Sophivorus (talkcontribs)

Thanks so much!! On the contrary, thanks for sharing your time and knowledge on this! I was able to remove the hack on Poncho and deployed all the new versions on Appropedia, yay!

Using Minerva, I get 'Your skin is incompatible with VisualEditor.'

1
84.111.224.227 (talkcontribs)

VisualEditor is supposed to support Minerva. Using 1.36.2, and latest VE (0.1.2) and Minerva (– (7203827) 17:04, 4 September 2021).

Reply to "Using Minerva, I get 'Your skin is incompatible with VisualEditor.'"

Any updates to this page for VE in MW 1.35 and up?

8
Revansx (talkcontribs)

Have the minimum requirements or recommendations changed now that VE is part of MW core?

Matma Rex (talkcontribs)

No, they didn't, as far as I know. I wonder what led to this question though, is there some context that I'm missing?

Revansx (talkcontribs)

@Matma Rex Hi Matma! Thanks for the quick response. The context is, I'm on MW 1.35 LTS and trying to use Skin:Metrolook branch REL1_35 and have discovered that it has lots of little CSS issues with VE integration. The Metrolook folks said [1] that they seek to be in compliance with the requirements of this page, but that is all they are willing to commit to for their purposes. But they also admitted that this page hasn't been updated for a while and so I got the sense that if this page had more requirements then it might motivate them to meet them.

[1] https://phabricator.wikimedia.org/T171480

Revansx (talkcontribs)

Interestingly.. I see now that you are the one who shared the VE requirements to the phabricator ticket T171480. Cool. In your response., you mentioned that if we needed help we could reach out to the VE folks. What's the best way to do that? In phabricator? Here? .. Eager to reach out. Thanks!

Matma Rex (talkcontribs)

On Phabricator is probably the most reliable way (or on IRC, if anyone happens to be around). To clarify, I'm also one of the VE folks :) I'm not willing to commit to help in fixing various CSS issues though, sorry – I think the responsibility on that falls on the skin, and it's probably far easier for the skin's author to resolve than us (although probably somewhat tedious).

Revansx (talkcontribs)
Matma Rex (talkcontribs)

Right. These are just the requirements that make it possible for the VE interface to appear when you click "Edit". But how it appears is mostly the skin's business.

Revansx (talkcontribs)

Understood. Thanks!

Reply to "Any updates to this page for VE in MW 1.35 and up?"
There are no older topics