Page MenuHomePhabricator

Fix eslint config, use kebab-case for Vue component tags
Closed, ResolvedPublic

Description

The library updater bot did a dependency update and rewrote some vue component markup while it was at it:
https://gerrit.wikimedia.org/r/c/mediawiki/extensions/QuickSurveys/+/782501/1/resources/ext.quicksurveys.lib/vue/QuickSurvey.vue

It turns out this was due to the eslint configuration, which should be corrected along with the component
From @Catrope

Using PascalCase tags like <WvuiButton> breaks in Vue templates in
MediaWiki, which is why our eslint rules require kebab-case tags like
<wvui-button>. However, this repo's eslint config was set up wrong and
accidentally enforced PascalCase instead, which led to buttons not
displaying