1998404 - www.webmd.com - Find in page bar disappears when it scrolls down through the findings

Status: RESOLVED FIXED

(Firefox for Android :: Browser Engine, defect)

Description

[Calin Tanase]

Environment:
Operating system: Android 15
Firefox version: Firefox Mobile [phone]

Steps to reproduce:

1. Go to https://www.webmd.com/vitamins/ai/ingredientmono-138/milk-thistle
2. Dismiss the cookie banner.
3. Tap on the browser kebab menu and select "Find in page".
4. Type "milk" on the "Find in page" search bar.
5. Tap on the down arrow to go through the results. (at the time of testing it reproduced when reached result 8)

Expected Behavior:
Find in page bar is still visible and can go through other results.

Actual Behavior:
Find in page bar disappears.

Notes:

• Reproduces regardless of the status of ETP
• Reproduces in firefox-nightly, and firefox-release
• Does not reproduce in chrome

Created from https://github.com/webcompat/web-bugs/issues/186430

Comment 1

[Calin Tanase]

Attachment: Find in page - Nightly.mp4

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

Since nightly and release are affected, beta will likely be affected too.
For more information, please visit BugBot documentation.

Comment 3

[Ksenia Berezina [:ksenia]]

This is likely not a webcompat bug, but if it is, just move it back to our component.

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:boek, could you have a look please?

For more information, please visit BugBot documentation.

Comment 5

[Jeff Boek [:boek]]

It looks like when the find in page is scrolling the page the toolbar is getting presented (maybe also dismissing the find in page bar?)

Petru do you happen to know how this behavior cooperates?

Comment 6

[Petru-Mugurel Lingurar [:petru]]

This is interesting.
Taking it for investigation.

Comment 7

[Petru-Mugurel Lingurar [:petru]]

Attachment: Bug 1998404 - Ignore URL fragment changes for closing the find in page UX r=#android-reviewers

The "#" separator is already a reserved character for separating url fragments
and we can easily use it for cleaning the URL that is observed for knowing when
the user actually navigates away to a different website.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/de8aca3bd6a0
https://hg.mozilla.org/integration/autoland/rev/c40173b917a7
Ignore URL fragment changes for closing the find in page UX r=android-reviewers,moyin

Comment 9

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/c40173b917a7

Comment 10

[BugBot [:suhaib / :marco/ :calixte]]

Since nightly and release are affected, beta will likely be affected too.
For more information, please visit BugBot documentation.

Comment 11

[Lorand Janos]

Attachment: Recording_20251210_091936.mp4

I managed to reproduce the issue on an earlier build (147.0b1).

Verified as fixed on the latest Nightly build (148.0a1 from 2025-12-09T14:30:57).

Find in page bar is still visible and can go through other results.

Device used: Samsung Galaxy S25 Ultra (Android 15).

Marking the ticket as verified on 148.

Comment 12

[Phabricator Automation]

firefox-beta Uplift Approval Request

• User impact if declined: Very small fix for fixing the find in page functionality on websites using URL fragments - prevent the find in page UX abruptly ending when navigating through results.
• Code covered by automated testing: yes
• Fix verified in Nightly: yes
• Needs manual QE test: yes
• Steps to reproduce for manual QE testing: https://bugzilla.mozilla.org/show_bug.cgi?id=1998404#c0
• Risk associated with taking this patch: low
• Explanation of risk level: Very small targeted fix.
  Verified by QA.
• String changes made/needed: --
• Is Android affected?: yes

Comment 13

[Petru-Mugurel Lingurar [:petru]]

Attachment: Bug 1998404 - Ignore URL fragment changes for closing the find in page UX

The "#" separator is already a reserved character for separating url fragments
and we can easily use it for cleaning the URL that is observed for knowing when
the user actually navigates away to a different website.

Original Revision: https://phabricator.services.mozilla.com/D275416

Comment 14

[Phabricator Automation]

firefox-release Uplift Approval Request

• User impact if declined: Very small fix for fixing the find in page functionality on websites using URL fragments - prevent the find in page UX abruptly ending when navigating through results.
• Code covered by automated testing: yes
• Fix verified in Nightly: yes
• Needs manual QE test: yes
• Steps to reproduce for manual QE testing: https://bugzilla.mozilla.org/show_bug.cgi?id=1998404#c0
• Risk associated with taking this patch: low
• Explanation of risk level: Very small targeted fix.
  Verified by QA.
• String changes made/needed: --
• Is Android affected?: yes

Comment 15

[Petru-Mugurel Lingurar [:petru]]

Attachment: Bug 1998404 - Ignore URL fragment changes for closing the find in page UX

The "#" separator is already a reserved character for separating url fragments
and we can easily use it for cleaning the URL that is observed for knowing when
the user actually navigates away to a different website.

Original Revision: https://phabricator.services.mozilla.com/D275416

Comment 16

[Pulsebot]

https://github.com/mozilla-firefox/firefox/commit/9260ae689785
https://hg.mozilla.org/releases/mozilla-beta/rev/895b9142b911