Page MenuHomePhabricator

Vector 2022: Table of contents intersection observer doesn't account for browser scaling (uses pixels)
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  • open Firefox and zoom via the browser, for exemple at 120%
  • open a random page on a wiki with Vector 2022 skin, for exemple this one
  • scroll down

What happens?:

  • the active section of the content doesn't match the highlighted section of the TOC

What should have happened instead?:

  • the highlighted section in the TOC should be the active one

Software version (skip for WMF-hosted wikis like Wikipedia):

Other information (browser name/version, screenshots, etc.):
(reported by a user on MediaWiki)
configuration of the user: firefox-esr 102.8.0esr, Debian 11, screen resolution 1920x1200

tested also with Chrome Version 110.0.5481.105 (Build officiel) (64 bits) and Firefox version 110 (64 bits), Windows 10 Home, screen resolution 2560x1440, same result

Event Timeline

Probably related to https://phabricator.wikimedia.org/T317661 but perhaps this needs a specific solution

Jdlrobson renamed this task from Vector 2022: zooming in Firefox makes the ToC highlighting not fit to the text to Vector 2022: Table of contents intersection observer doesn't account for browser scaling (uses pixels).Feb 27 2023, 7:36 PM
ovasileva triaged this task as Medium priority.Mar 1 2023, 10:51 PM
ovasileva moved this task from Incoming to Current Quarter on the Web-Team-Backlog-Archived board.

@Patafisik Thank you for the report. I tried replicating this on my Mac and Window (via Browserstack), but haven't been able to replicate this problem. Can you please check my recording at https://app.screencast.com/KfbS8Q0V1Uxay?conversation=2E9AB8aetLIJcobL533a0s, and let me know if I need to do something different to replicate it?

@nray thank you for looking into this. Actually I can't reproduce it and I see the same thing you see, so I asked the user for a new feedback. However, if no technical changes have resolved this task meanwhile, T331109 may explain why both me and the user have had the impression of a mismatch.

Jdlrobson claimed this task.
Jdlrobson subscribed.

This is working for me as expected as well. Please feel free to reopen if something got missed.