Page MenuHomePhabricator

Decide whether to enable new termbox by default in Wikibase
Closed, ResolvedPublic


Currently, the new termbox, aka Termbox v2, aka mobile termbox, is disabled by default in Wikibase, but enabled in the example settings, meaning it’s effectively default-enabled in most Wikibase installations (including the release pipeline docker images). As we want to get rid of the example settings, we need to make a decision: should we properly enable termbox by default, or disable it (with a notice in the release notes that it’s no longer enabled by the example settings)?

Event Timeline

This is a product decision, so this is up to @Lydia_Pintscher or @Samantha_Alipio_WMDE, I assume (not sure which of them).

(Sam's decision but asking question for clarification.)

This is about enabling it for the mobile view only, right? For desktop we are definitely not there yet.

Only on mobile, I think. (I use the example settings and get the new termbox on mobile but not on desktop.)

Hi @Lucas_Werkmeister_WMDE @Michael, sorry for taking so long to respond.
Yes, let's enable the termbox on mobile by default, because:

  • the example settings for the Wikibase Extension already enable the termbox on Mobile, and it seems like the vast majority of people uses the example settings (as this is encouraged in all the documentation), so the vast majority seems to be seeing the termbox already
  • On Wikidata, we use server side rendering. However, this is not part of our Wikibase release packages. People without javascript will not see any terms on mobile, if the new termbox is enabled by default. However, ever since the termbox was enabled in the example settings more than 2 years ago, we have not had complaints of not-existing termboxes.
  • Taking this out of the settings will make the installation process easier.

Change 726873 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/Wikibase@master] Enable new / mobile Termbox (v2) by default

Change 726873 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Enable new / mobile Termbox (v2) by default