Switching palette causes scroll position jump on Firefox

[sisp]

Context

No response

Bug description

The scroll position jumps upwards upon clicking the palette toggle on Firefox (v117.0, Ubuntu 20.04). For instance, if you scroll to the bottom of a page, then clicking the palette toggle several times will move the scroll position to the top of the page eventually. This is certainly not desired behavior.

It works fine for me on Chromium.

Related links

• Reporting a bug

Reproduction

9.2.6-palette-toggle-scroll-position.zip

Steps to reproduce

1. Open any page of the docs in the reproduction.
2. Scroll down, e.g. to the bottom of the page.
3. Click the palette toggle and observe how the scroll position jumps upwards upon every click of the toggle.

Browser

Firefox

Before submitting

• I have read and followed the bug reporting guidelines.
• I have attached links to the documentation, and possibly related issues and discussions.
• I assure that I have removed all customizations before submitting this bug report.
• I have attached a .zip file with a minimal reproduction.

[nejch]

Just giving extra context:

• Fedora 38 amd64
• Firefox 116 I couldn't reproduce this.
• Firefox 117, after upgrade, I also get this behavior.

---

Screencast.from.2023-09-01.11-43-45.webm

[squidfunk]

Simply follow the reproduction steps below, no need for a self-contained reproduction artifact.

Thanks for reporting. However, please provide a minimal reproduction. It is necessary, because it allows us to fix the bug much quicker. Otherwise we have to create the reproduction first.

[sisp]

Okay, I've update the original post with a minimal reproduction.

[squidfunk]

Fixed in e79ffab. Here's my analysis:

mkdocs-material/src/assets/stylesheets/main/components/_header.scss

Lines 244 to 250 in e79ffab

	// Hack: Firefox 117 introduces a bug where the browser scrolls the page by
	// a small amount to the top every time the header button is focused. After
	// investigating, we're confident that it seems to be caused by the input
	// field being too close to the border - see https://t.ly/APO8l
	> input {
	bottom: 0;
	}

[squidfunk]

Released as part of 9.2.7.