View jumps to the top of the page when the search string does not match any word on the page

NEW
Unassigned

Status

()

6 years ago
2 years ago

People

(Reporter: AdrianT, Unassigned)

Tracking

Trunk
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox20 affected, firefox21 affected, firefox22 affected, firefox23 affected, firefox28 affected, firefox29 affected, firefox30 affected, firefox31 affected, fennec-)

Details

(Reporter)

Description

6 years ago
Firefox Mobile 15 beta 6 build 1/ Nightly 17.0a1 2012-08-21
Device: Asus Transformer (Android 4.0.3)

Steps to reproduce:
1. Load planet.mozilla.org
2. Open "Find in page" type "firefo" and advance through a few results to position the view somewhere lower in the page
3. Add a "c" to the search field so the search string is "firefoc"

Expected results:
When the search string does not exist on the page no word is highlighted

Actual results:
No word is highlighted but the view jumps to the top of the page regardless of where the page was positioned before.

Note: 
On the stock Android Browser the highlight is just removed but the scroll position is not changed
This sounds working as expected to me.
Ian what is the expected behavior here?
tracking-fennec: ? → -
I talked with Ian on IRC and here are the conclusions:

1. ibarlow Xti: Hm, yeah I don't think we should jump to the top. We should maintain the most recent scroll position, and turn off the highlight on the last matched word

A suggestion was that every new search should start from the beginning.
2. ibarlow: I agree with that behaviour, that a new search would start from the top of the page

I guess that all can be fixed through to this bug.
(Reporter)

Updated

6 years ago
status-firefox15: affected → ---
status-firefox16: affected → ---
status-firefox18: --- → affected
status-firefox19: --- → affected
status-firefox17: affected → ---
status-firefox18: affected → ---
status-firefox19: affected → ---
status-firefox20: --- → affected
status-firefox21: --- → affected
status-firefox22: --- → affected
status-firefox23: --- → affected

Updated

5 years ago
status-firefox28: --- → affected
status-firefox29: --- → affected
status-firefox30: --- → affected
status-firefox31: --- → affected
You need to log in before you can comment on or make changes to this bug.