The service MobileFrontend.Context created by https://gerrit.wikimedia.org/r/c/mediawiki/extensions/MobileFrontend/ /305212 (rEMFRf3cf72ba636f: Add MobileFrontend.Context service) is not a valid service because it depends on RequestContext::getMain().
In https://gerrit.wikimedia.org/g/mediawiki/core/ /80064365ac0c0809b0932bfde77f082727aced8c/includes/ServiceWiring.php#15 stands:

Services MUST NOT vary their behaviour on global state, especially not WebRequest, RequestContext (T218555).

In T366792#9873346, @Nardog hat geschrieben:

How is this different from T363630: Score audio cannot be played if the page has other embedded media?

Formally the removal of the parameter onlyroot is a breaking change, but the impact is just one expanded level of subcategories.

In T290062#9760939, @Krinkle wrote:

What cost are we trading?
What is the benefit?

According to what commit is it deprecated?

https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CollapsibleVector/ /638100 uses the hook 'wikipage.categories' to trigger the collapsing again. This is bad, because this hook gets also triggered on every live preview. Therefor https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CollapsibleVector/ /1021554 reverts this change.

The bug description is about a specific bug which was accidental introduced by change rECTRaec8d75570fd: Replace WebRequest::getVal by ::getRawVal or ::getText. This bug gets fixed by https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CategoryTree/ /991038 PS1. But you blocked this change.

@Jdlrobson: Is this still a regression?

This bug was introduced by the change rECTRaec8d75570fd: Replace WebRequest::getVal by ::getRawVal or ::getText.

I tested with a iPhone SE3 with iOS 16.7.2, 30 MBit/s downstream and with mobile version and desktop version of the page. I can see the starting and stopping of loading animation, which takes a few 100 ms I guess.

The behavior on https://patchdemo.wmflabs.org/wikis/274381f11d/wiki/Hymne_der_Sowjetunion with a iPhone Safari is now slightly different:

This issue is independent to T350369, because rEPHN365e80b9f580: Use static integers instead of OO.ui.Keys (https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Phonos/ /973387/) is now deployed, but issue still exist.

modules/ext.phonos.init.js uses OO.ui.Keys. OO.ui.Keys is defined in module oojs-ui-core. In extension.json there is no dependency on this module. Therefor error can happen if the module in not loaded yet.

Issue is solved. Verified on Wikipedia.

The URL parameters maxage and smaxage of api.php ignore now negative integers.

No more hits: https://codesearch.wmcloud.org/search/?q=AjaxUploadDestCheck

rMW95a4189814e2: Prepare for deprecation of Skin::bottomScripts is already included in MediaWiki 1.39.

rMW11e0fba3b0cd: Deprecate global function wfReportTime() didn't introduce a duplicate call of SkinAfterBottomScripts. The call was just moved to the beginning of getBottomScripts to include the execution time of this call in the elapsed time calculation.

The issues on skins Timeless and Splash are solved. The issue with the green markers is tracked in T342445.

The green markers are generated by the gadget https://en.wikipedia.org/wiki/MediaWiki:Gadget-WatchlistGreenIndicators.css and not by the skin Timeless. Therefor the change described in T342445#9037143 should adapt the gadget to the new HTML code.

@Trappist_the_monk Is this the problem described in T342445: Watchlist isn't showing unread triangle (multiple change) markers in green?

As someone who uses Timeless on small screens, this is a pretty significant negative change to watchlist behavior. Did you test small resolutions here?

1.41.0-wmf.19 is now deployed on https://en.wiktionary.org/wiki/hasOwnProperty and there are no JavaScript errors anymore.

A change from

Background is the change https://gerrit.wikimedia.org/r/c/mediawiki/extensions/GrowthExperiments/ /912356

I don't see a security problem here. https://www.mediawiki.org/wiki/Security_for_developers#Cross-site_scripting_(XSS) recommends:

Escape as close to the output as possible, so that the reviewer can easily verify that it was done.

Therefor the developer should use

$element.text( mw.message( key ).text() )

or

$element.text( message )

instead of

$element.html( mw.message( key ).escaped() )

or

$element.html( message )

The most common use case is the unescaped mw.message( key ).text() because escaping should be done as close to the output as possible. And mw.msg( key ) is a useful shortcut for the most common use case.

Before rMW103eb48ac20e: ResourceLoader: Set module state "error" if request fails on network the screen gets black.
After rMW103eb48ac20e: ResourceLoader: Set module state "error" if request fails on network there is a flash of a black screen.

The problematic line-height: 0 was introduced in rMWaf2aed8ea9a9: Suppress line-height inside thumbs by @Arlolra.

The wrong URL for ajax-loader.gif was probably introduced by rEKAR5bf8b81ee483: Kartographer: convert to packageFiles 3 years ago.

A simple fix would be to use the link

https://www.mediawiki.org/w/index.php?title=Special:Log&page=Compatibility&wpFormIdentifier=logeventslist

instead of

https://www.mediawiki.org/w/index.php?title=Special:Log&page=Compatibility

on https://www.mediawiki.org/w/index.php?title=Compatibility&action=history but there are also other links which use the URL parameter page.

The field Target is not filled with the value of the URL parameter page since rMW72913f4cc3f4: Call setFormIdentifier() on LogEventsList form.

The user option variant also contains a language code which can be a deprecated language code after T117845: Rename the language codes sr-ec and sr-el to the BCP 47 conform codes sr-Cyrl and sr-Latn or other changes of language codes.