In T314925#8146288, @awight wrote:

The marker icons seem to be broken because protocol-relative links are generated, but to an external site: "//api.tiles.mapbox.com/mapbox.js/v3.3.1/images/marker-icon.png" . It would need to be "https://". However, we should be pulling the icon from the local kartotherian service either way, so we'll need to override the marker URL entirely. The image should be something like https://maps-beta.wmflabs.org/v4/marker/pin-l-marker+f00.png

The URL is coming from leaflet/src/layer/marker/Icon.Default.js and it looks like it's designed to be overridden by an Icon constructor option.

You should be able to disable the moved line detection by setting ini_set( 'wikidiff2.moved_paragraph_detection_cutoff', 0 ); in your LocalSettings.php

In T307993#8145779, @ECohen_WMDE wrote:

I would keep it simple for now then and add the aria-label. Is this stalled because it was waiting for my review or is it something else?

FYI: Not working on the issue itself but on some impact on one of our tools.

Fine for me. Thanks!

Very good questions.

From some brief looks into the uk fails:

• I first checked superset if request might mainly come from direct calls to the API but that seemed not to be the case
• Then I looked at the pages that are associated with the revids in the logs:

In all cases I checked I saw the following, when just looking at the page using the title:
Template/file changes in this version are pending review. The stable version was checked on 21 June 2022.
So there's a stable version, that's also the most recent version, but there are still Pending changes because some template dependencies are unreviewed. Also the user will always be redirected to the Pending changes tab by default it seems. [1]

Note: This is a behavior we found weird but are not sure if we should fix it or leave it like that. UX feedback needed. :-)
P.S.: Fix is already there so the effort is kind of zero to fix it.

Checked all cases and they seem fixed.

The patches merged should fix the issue for cases with and without the sticky header. We're not introducing a dynamic calculation for now but the main user facing issues should be gone.

This is fixed on master with the current changes to scroll to highlighted parameters while respecting the sticky header height.

Fixed with https://gerrit.wikimedia.org/r/c/mediawiki/extensions/VisualEditor/+/819516/

I used Firefox. In Chrome it seems to work just fine. 🤔

In T312926#8119085, @gerritbot wrote:

Change 819046 had a related patch set uploaded (by WMDE-Fisch; author: WMDE-Fisch):

[mediawiki/extensions/VisualEditor@master] Tweak the hard coded top scroll padding for the parameter list

https://gerrit.wikimedia.org/r/819046

In T312926#8086886, @awight wrote:

Some of us can still reproduce the issue, so leaving it open.

In T314065#8118497, @Manuel wrote:

Hi @WMDE-Fisch, thx for looking into this! Could you please elaborate a bit?

In T314065#8113146, @Manuel wrote:

Thank you for reporting this! Pinging wdwb-tech about this.

I tried to find a way to block that default scroll. But did not succeed. There seems to be none? o.O

What happens here:

Seems also fixed with https://gerrit.wikimedia.org/r/c/mediawiki/extensions/VisualEditor/+/813631